mirror of
https://github.com/mpv-player/mpv
synced 2024-12-24 07:33:46 +01:00
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
28 lines
548 B
C
28 lines
548 B
C
|
|
// prototypes:
|
|
//static ao_info_t info;
|
|
static int control(int cmd, void *arg);
|
|
static int init(int rate,int channels,int format,int flags);
|
|
static void uninit(int immed);
|
|
static void reset(void);
|
|
static int get_space(void);
|
|
static int play(void* data,int len,int flags);
|
|
static float get_delay(void);
|
|
static void audio_pause(void);
|
|
static void audio_resume(void);
|
|
|
|
#define LIBAO_EXTERN(x) ao_functions_t audio_out_##x =\
|
|
{\
|
|
&info,\
|
|
control,\
|
|
init,\
|
|
uninit,\
|
|
reset,\
|
|
get_space,\
|
|
play,\
|
|
get_delay,\
|
|
audio_pause,\
|
|
audio_resume\
|
|
};
|
|
|