mirror of https://github.com/mpv-player/mpv
e7db4ccf1a
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 git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@17567 b3059339-0415-0410-9bf9-f77b7e298cf2 |
||
|---|---|---|
| .. | ||
| .cvsignore | ||
| Makefile | ||
| a52.h | ||
| a52_internal.h | ||
| bit_allocate.c | ||
| bitstream.c | ||
| bitstream.h | ||
| compare.c | ||
| config.h | ||
| crc.c | ||
| downmix.c | ||
| imdct.c | ||
| imdct_3dnow.h | ||
| imdct_mlib.c | ||
| liba52.txt | ||
| liba52_amd64_changes.diff | ||
| liba52_changes.diff | ||
| mangle.h | ||
| mm_accel.h | ||
| parse.c | ||
| resample.c | ||
| resample_altivec.c | ||
| resample_c.c | ||
| resample_mmx.c | ||
| srfftp.h | ||
| srfftp_3dnow.h | ||
| tables.h | ||
| test.c | ||