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

Michael Niedermayer michaelni
Mon May 21 18:48:46 CEST 2007


Hi

On Mon, May 21, 2007 at 05:59:26PM +0200, Thijs Vermeir wrote:
> 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.

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070521/0da464fa/attachment.pgp>



More information about the ffmpeg-devel mailing list