• Stefan Huehner's avatar
    Patch by Stefan Huehner / stefan % huehner ! org \ · 9b2283cc
    Stefan Huehner authored
    patch replaces '()' for the correct '(void)' in function
    declarations/prototypes which have no parameters. The '()' syntax tell
    thats there is a variable list of arguments, so that the compiler cannot
    check this. The extra CFLAG '-Wstrict-declarations' shows those cases.
    
    Comments about a similar patch applied to ffmpeg:
    
    That in C++ these mean the same, but in ANSI C the semantics are
    different; function() is an (obsolete) K&R C style forward declaration,
    it basically means that the function can have any number and any types
    of parameters, effectively completely preventing the compiler from doing
    any sort of type checking. -- Erik Slagter
    
    Defining functions with unspecified arguments is allowed but bad.
    With arguments unspecified the compiler can't report an error/warning
    if the function is called with incorrect arguments. -- Måns Rullgård
    
    Originally committed as revision 17567 to svn://svn.mplayerhq.hu/mplayer/trunk/postproc
    9b2283cc
swscale.c 75.9 KB