[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