[streaming] [Fwd: Re: VoD with authentication]
Son Minh Tran
son-minh.tran at int-evry.fr
Wed Mar 8 12:46:13 CET 2006
Dear Jean-Paul Saman,
Thank you for your review. Please find in the attachment the diff file
generated by svn. Hopefully now the code is compliant to vlc convention
REgards
Son
Jean-Paul Saman a écrit :
> 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.
>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: vod.patch
Url: http://mailman.videolan.org/pipermail/streaming/attachments/20060308/4ef1eb4d/attachment.txt
More information about the streaming
mailing list