[FFmpeg-devel] FFMPEG RTSP parameters support

Ronald S. Bultje rsbultje
Mon Nov 15 15:58:15 CET 2010


Hi,

On Mon, Nov 15, 2010 at 2:19 PM, Martin Storsj? <martin at martin.st> wrote:
> On Mon, 15 Nov 2010, Stas Oskin wrote:
>> > Stas, can you test the patch with the actual server where this was needed?
>> > You could also remove the else branch from the addition in rtspdec.c and
>> > see if that is enough for it to work.
>> >
>> >
>>
>> Patch works great, both originally and with the else section removed:
>> + ? ?} else {
>> + ? ? ? ?/* Remove the query parameters from the control uri for subsequent
>> + ? ? ? ? * requests. */
>> + ? ? ? ?char *ptr = strchr(rt->control_uri, '?');
>> + ? ? ? ?if (ptr)
>> + ? ? ? ? ? ?*ptr = '\0';
>> + ? ?}
>>
>> Thanks for the fix, hope it get committed to tree.
>
> Great, thanks for testing.
>
> Ronald, Luca B, ok to commit without that else branch?

Yes, thanks :-).

Ronald



More information about the ffmpeg-devel mailing list