[Ffmpeg-devel] Re: [PATCH] portability fixes (ILP32 vs LP64)
Dave Dodge
dododge
Wed Jan 25 21:57:48 CET 2006
On Tue, Jan 24, 2006 at 07:25:53PM +0000, Dieter wrote:
> warning: no previous prototype for 'ff_h264_idct_add_mmx2'
I haven't looked at the code in question to see if any of this applies,
but _usually_ this particular warning indicates one of the following
scenarios:
- the function is not called directly from outside of the
translation unit, and therefore can be marked static. This
warning is not normally issued for static functions.
- the function is intended for use by other translation units, but a
prototype is missing from the appropriate header file.
- the header file (for example foo.h) does contains a prototype, but
the file that defines the function (for example foo.c) did not
include that header. It's usually a good idea for foo.c to
include foo.h even if it doesn't really need the stuff from foo.h,
because it allows the compiler to check that the prototype matches
the definition.
-Dave Dodge
More information about the ffmpeg-devel
mailing list