[Ffmpeg-devel] Re: [PATCH] portability fixes (ILP32 vs LP64)
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
- 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
More information about the ffmpeg-devel