[vlc-devel] [PATCH 00/13] I took `modules/access/rtsp` (aka. "realrtsp") to the hospital

[Filip Roséen]

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.

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).


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).

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!


Thanks,
F

------------------------------------------------------------------------------

On 16/02/25 17:22, Rémi Denis-Courmont wrote:

> Le 2016-02-25 11:12, Filip Roséen a écrit :
> >I accidentally stumbled upon the module *realrtsp* 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).
> 
> This is well-known; the plugin has been disabled for years.
> 
> IMHO, that plugin should just be deleted.
> 
> -- 
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20160225/f86299a3/attachment.html>