[FFmpeg-devel] [PATCH 7/8] Split away the "found" case in ff_h264_find_frame_end().

Guillaume POIRIER poirierg
Thu Oct 2 21:25:22 CEST 2008


Hello,

On Thu, Oct 2, 2008 at 7:59 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Oct 02, 2008 at 04:39:57PM +0200, Diego 'Flameeyes' Petten? wrote:
>> With the original code, ICC issued this warning:
>>
>> libavcodec/h264_parser.c(77): warning #589: transfer of control
>> bypasses initialization of:
>>             variable "v" (declared at line 58)
>>                   goto found;
>>
>> since the found branch was part of another branch where a variable v
>> was declared; since the branch is a returning branch, independent of
>> all the remaining variables, split it at the end of the function.
>
> ok, though thats more because its more readable IMHO than because
> of the warning.

Patch committed.

Guillaume
-- 
One should not give up hope on imbeciles. With a little training, you
can make them into soldiers.
 -- Pierre Desproges



More information about the ffmpeg-devel mailing list