Skip to content

Conversation

firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

This fixes the issue but I am not convinced this is the full fix. It seems there is a parseDefine() call which should not be happening since a macro is being re-defined.

@firewave
Copy link
Collaborator Author

This fixes the issue but I am not convinced this is the full fix. It seems there is a parseDefine() call which should not be happening since a macro is being re-defined.

The GNU compilers both only show a warning, so this fix should be fine.

a.cpp:2:9: warning: 'e' macro redefined [-Wmacro-redefined]
    2 | #define e
      |         ^
a.cpp:1:9: note: previous definition is here
    1 | #define e(...)__VA_OPT__()
      |         ^
a.cpp:2:9: warning: ‘e’ redefined
    2 | #define e
      |         ^
a.cpp:1:9: note: this is the location of the previous definition
    1 | #define e(...)__VA_OPT__()
      |         ^

@firewave firewave marked this pull request as ready for review August 30, 2025 19:04
@firewave firewave merged commit 7f59eec into danmar:master Sep 25, 2025
14 checks passed
@firewave firewave deleted the leak-xx branch September 25, 2025 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants