[FFmpeg-devel] [PATCH 1/7] opt: Remove dead code

Carl Eugen Hoyos cehoyos at ag.or.at
Fri Feb 6 10:27:57 CET 2015


Timothy Gu <timothygu99 <at> gmail.com> writes:

>          if (!i || !*val)
>              return 0;
>      }
> -
> -    return 0;
>  }

I fear that some (broken) compilers will emit 
a warning when this gets applied that we 
request to be interpreted as an error.
Same for 3, 4 and 5.
I don't remember if an assert helps.

The img2 patch is ok.

Carl Eugen



More information about the ffmpeg-devel mailing list