Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bugfix] Handle gcc unsupported __VA_OPT___ macro (#1880) #1881

Merged
merged 9 commits into from
Sep 18, 2022
Merged

[Bugfix] Handle gcc unsupported __VA_OPT___ macro (#1880) #1881

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
6d3dbd2
[Bugfix] Handle gcc unsupported __VA_OPT___ macro
tonhuisman Sep 16, 2022
7a050af
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, fix linter complaints
tonhuisman Sep 16, 2022
aaeb6f1
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, improved detection…
tonhuisman Sep 17, 2022
349f1fb
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, fix doxygen warnings
tonhuisman Sep 17, 2022
5a5e02f
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, fix linter complaints
tonhuisman Sep 17, 2022
923e5ff
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, fix linter contrad…
tonhuisman Sep 17, 2022
6229d15
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, fix linter contrad…
tonhuisman Sep 17, 2022
aaab233
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, link directly to s…
tonhuisman Sep 17, 2022
0fef273
Minor commenting style tweak
crankyoldgit Sep 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[Bugfix] Handle gcc unsupported __VA_OPT___ macro, improved detection… 
… macro
  • Loading branch information
@tonhuisman
tonhuisman committed Sep 17, 2022
commit aaeb6f1f5ba3b55f3aa9e3f6e87ffad7cff836b9
32 changes: 10 additions & 22 deletions src/IRmacros.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,29 +6,16 @@
/// @file IRmacros.h

/**
* PP_NARG() macro get number of arguments provided
* Source:
* http://jonjagger.blogspot.com/2010/11/c-macro-magic-ppnarg.html
* ('base' version)
* VA_OPT_SUPPORTED macro to check if __VA_OPT__ is supported
* Source: https://stackoverflow.com/questions/48045470/
* portably-detect-va-opt-support
* (answer by cpplearner)
*/
/// @cond TEST
#ifndef __VA_OPT__
#define __VA_OPT__(...)
#endif
#define PP_NARG(...) \
PP_NARG_(__VA_ARGS__, PP_RSEQ_N())

#define PP_NARG_(...) \
PP_ARG_N(__VA_ARGS__)

#define PP_ARG_N( \
_1, _2, _3, _4, _5, _6, N, ...) (N)

#define PP_RSEQ_N() \
6, 5, 4, 3, 2, 1, 0
/// @endcond
#define PP_THIRD_ARG(a, b, c, ...) c
#define VA_OPT_SUPPORTED_I(...) PP_THIRD_ARG(__VA_OPT__(, ), true, false, )
#define VA_OPT_SUPPORTED VA_OPT_SUPPORTED_I(?)
/**
* PP_NARG() end
* VA_OPT_SUPPORTED end
*/

/**
Expand All @@ -43,7 +30,8 @@
* NB: If __VA_OPT__ macro not supported, the <true_result> will be expanded!
*/
/// @cond TEST
#if PP_NARG(__VA_OPT__(, )) != 2
#if !VA_OPT_SUPPORTED
// #pragma message("Compiler without __VA_OPT__ support")
#define COND(cond, a, b) a
#else
#define NOTHING
Expand Down