[streaming] Re: [Son Minh Tran <son-minh.tran at int-evry.fr>] [Fwd: Re: VoD with authentication]

Jean-Paul Saman jean-paul.saman at planet.nl
Tue Mar 7 23:40:55 CET 2006


Son,

Again thank you for your contribution.

I had a quick look at the code and have some remarks for readibility 
improvements:

1) your addition could be easier to review by us and yourself if you 
would use GNU diff. It prints a difference to the original file with 
only your changes in it.

Usage:
diff -urN vlc-trunk/modules/misc/rtsp.c vlc-vod/modules/misc/rtsp.c > 
vod.patch

Or use: svn diff > vod.patch

2) there is still a lot of your original debug information around, like: 
"//SON". Please remove them

3) use VLC coding style. I suggest to read this webpage 
http://developers.videolan.org/vlc/vlc/doc/developer/html/ch01.html#id2847643

I'll send a more detailed description to you personally.

Kind greetings,
Jean-Paul Saman.

-- 
This is the streaming mailing-list, see http://www.videolan.org/streaming/
To unsubscribe, please read http://www.videolan.org/support/lists.html



More information about the streaming mailing list