<p>I certainly agree with you; the module is a mess and should probably be rewritten/removed in the future–especially if the equivalent functionality is available elsewhere.</p>
<p>Truthfully I get bored easily and I thought that I could just as well clean/fix the module during my spare time (since it is still part of the codebase).</p>
<p>I understand that it might be hard to get someone to review and merge it into master given the reputation of the module, but if it makes it inside I would at least rest a little bit easier at night (knowing that the pile of problems in the module is not as big any more).</p>
<p>With the goal of becoming more familiar with the codebase, analyzing/working on different modules (even the poor ones) might help me reach the finish line sooner. If you got some other piece of advice on the matter I would be very thankful!</p>
<p>Thanks, F</p>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<p>On 16/02/25 17:22, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>Le 2016-02-25 11:12, Filip Roséen a écrit :</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>I accidentally stumbled upon the module <em>realrtsp</em> and quickly realized that the poor fella was indeed quite sick. I have spent a few hours writing patches for the most serious implications that the current code have, and focus has been on making the module more stable (ie. not crash on invalid input).</p>
</blockquote>
<p>This is well-known; the plugin has been disabled for years.</p>
<p>IMHO, that plugin should just be deleted.</p>
<p>– Rémi Denis-Courmont http://www.remlab.net/ _______________________________________________ vlc-devel mailing list To unsubscribe or modify your subscription options: https://mailman.videolan.org/listinfo/vlc-devel</p>
</blockquote>