[FFmpeg-devel] [PATCH] Fix warning about decode_mb_skip (in libavcodec/h264.h)

Måns Rullgård mans
Fri Jul 2 00:56:32 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Tue, Jun 29, 2010 at 10:06:12PM -0700, Eli Friedman wrote:
>> Patch attached; fixes warning "'decode_mb_skip' defined but not used".
>>  As far as I can tell, doesn't have any significant effect on the
>> generated code, since the function in question gets inlined either
>> way.
>
> feel free to add av_unused

I'd rather you did one of the following:

1.  If the intent is to inline the function, mark it inline.
2.  If not, move it to a .c file of your choosing and make it extern.
3.  If only used from one .c file, move it there.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list