[FFmpeg-devel] [PATCH] remove transport concatenation dead code
Ronald S. Bultje
rsbultje
Tue Sep 2 14:56:07 CEST 2008
Hi,
On Sun, Aug 31, 2008 at 7:48 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> at some point (long ago), transports were flags and it was apparently
> planned to ask rtsp servers for multiple transports and just use
> whatever worked for them. That code never did what it was supposed to
> and nowadays, we don't use the flags anymore (see
> make_setup_request()). However, the code for transport concatenation
> is still there, it looks like this:
>
> for(..) {
> char transport[];
> transport[0]=0;
>
> if (transport_type == UDP){
> if(transport[0]!=0)...
> snprintf(transport+strlen(transport), ...);
> } else if (transport_type == TCP){
> if(transport[0]!=0)...
> snprintf(transport+strlen(transport), ...);
> } else ... etc.
> }
>
> All code assuming the transport is already filled in is useless
> because only one of the conditions is true at any given time, not more
> (hence the if/else if/else if, not if/if/if). Attached patch removes
> the dead code.
This one also needs review. It's kind of obvious so I could just apply
it in 2-3 days but I'd prefer a review anyway, rookie here.
Ronald
More information about the ffmpeg-devel
mailing list