[FFmpeg-devel] [PATCH] Add rtsp options support ( added option)

[Thijs Vermeir]

Hi
On Mon, 2007-05-21 at 17:19 +0200, Michael Niedermayer wrote:
> Hi
>
> On Mon, May 21, 2007 at 05:13:35PM +0200, Thijs Vermeir wrote:
> > Hi
> > >[...]
> > Thanks for your review,
> > In the patches applied this issues should be solved.
>
> [...]
>
> > +        int length_of_path = option_list++ - path;
> > +        path[length_of_path] = 0;
>
> no this is even more obfuscated
> its just a plain and simple
> *option_list++ = 0;
Oeps, this shoud be fixed now.

Gr,
Thijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rtsp_options_1_of_2.patch
Type: text/x-patch
Size: 1278 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070521/06099672/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rtsp_options_2_of_2.patch
Type: text/x-patch
Size: 1437 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070521/06099672/attachment-0001.bin>