[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