[PATCH v2 05/14] drivers/bus: use portable variadic macros
    David Marchand 
    david.marchand at redhat.com
       
    Mon Dec 23 09:43:38 CET 2024
    
    
  
On Wed, Dec 11, 2024 at 11:09 PM Andre Muezerie
<andremue at linux.microsoft.com> wrote:
>
> Many places are using a GCC extension related to variadic macros,
> where a name prepends the ellipsis. This results in a warning like
> the one below when compiling the code with MSVC:
>
> app\test-pmd\testpmd.h(1314): error C2608:
>     invalid token '...' in macro parameter list
>
> Variadic macros became a standard part of the C language with C99.
> GCC, Clang and MSVC handle them properly.
>
> The fix is to remove the prefix name (args... becomes ...) and use
> __VA_ARGS__.
>
> Signed-off-by: Andre Muezerie <andremue at linux.microsoft.com>
There is one missed occurence:
$ git grep 'define.*[^([:space:]]\.\.\..*)' -- drivers/bus/
drivers/bus/fslmc/qbman/include/compat.h:#define pr_debug(fmt, args...) {}
This does not require a new revision, I'll fix it myself when applying.
-- 
David Marchand
    
    
More information about the dev
mailing list