]> git.sesse.net Git - ffmpeg/commit
avcodec/cbs: Avoid leaving the ... out in calls to variadic macros
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sun, 22 Mar 2020 19:34:21 +0000 (20:34 +0100)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sun, 12 Apr 2020 21:23:49 +0000 (23:23 +0200)
commit14dd0a9057019e97ff9438f6cc1502f6922acb85
tree68e4f03323b8b503a8ce6dcc5ce7fa3a5d4ee9db
parentb7d9507bb8c4d1b8bf99158d6859a5b2ecd73298
avcodec/cbs: Avoid leaving the ... out in calls to variadic macros

According to C99, there has to be at least one argument for every ...
in a variadic function-like macro. In practice most (all?) compilers also
allow to leave it completely out, but it is nevertheless required: In a
variadic macro "there shall be more arguments in the invocation than there
are parameters in the macro definition (excluding the ...)." (C99,
6.10.3.4).

CBS (not the framework itself, but the macros used in the
cbs_*_syntax_template.c files) relies on the compiler allowing to leave
a variadic macro argument out. This leads to warnings when compiling in
-pedantic mode, e.g. "warning: must specify at least one argument for
'...' parameter of variadic macro [-Wgnu-zero-variadic-macro-arguments]"
from Clang.

Most of these warnings can be easily avoided: The syntax_templates
mostly contain helper macros that expand to more complex variadic macros
and these helper macros often omit an argument for the .... Modifying
them to always expand to complex macros with an empty argument for the
... at the end fixes most of these warnings: The number of warnings went
down from 400 to 0 for cbs_av1, from 1114 to 32 for cbs_h2645, from 38 to
0 for cbs_jpeg, from 166 to 0 for cbs_mpeg2 and from 110 to 8 for cbs_vp9.

These eight remaining warnings for cbs_vp9 have been fixed by switching
to another macro in cbs_vp9_syntax_template: The fixed values for the
sync bytes as well as the trailing bits for byte-alignment are now read
via the fixed() macro (this also adds a check to ensure that trailing
bits are indeed zero as they have to be).

Reviewed-by: Mark Thompson <sw@jkqxz.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavcodec/cbs_av1.c
libavcodec/cbs_h2645.c
libavcodec/cbs_jpeg.c
libavcodec/cbs_mpeg2.c
libavcodec/cbs_vp9.c
libavcodec/cbs_vp9_syntax_template.c