Skip to content

sw_isr_table: Add spaces to fix compiler warning #90880

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.

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

Merged
merged 1 commit into from
May 31, 2025

Conversation

thughes
Copy link
Contributor

@thughes thughes commented May 30, 2025

When using clang to build a C++ file that includes this file, it warns:

include/zephyr/sw_isr_table.h:195:28: error: invalid suffix on literal;
C++11 requires a space between literal and identifier
[-Wreserved-user-defined-literal]
         "." Z_STRINGIFY(prefix)"."file"." Z_STRINGIFY(counter)

@thughes thughes force-pushed the push-upkvvossurmn branch from 4de80a3 to 3666255 Compare May 30, 2025 17:36
When using clang to build a C++ file that includes this file, it warns:

include/zephyr/sw_isr_table.h:195:28: error: invalid suffix on literal;
C++11 requires a space between literal and identifier
[-Wreserved-user-defined-literal]
         "." Z_STRINGIFY(prefix)"."file"." Z_STRINGIFY(counter)

Signed-off-by: Tom Hughes <tomhughes@chromium.org>
@thughes thughes force-pushed the push-upkvvossurmn branch from 3666255 to 7e19c4b Compare May 30, 2025 18:26
@thughes thughes changed the title sw_isr_table: Add space to fix compiler warning sw_isr_table: Add spaces to fix compiler warning May 30, 2025
Copy link

@thughes
Copy link
Contributor Author

thughes commented May 30, 2025

@ycsin @kartben No reviewers or assignees were assigned. Could you help me find a reviewer? (I can't modify assignee/reviewer.)

@thughes
Copy link
Contributor Author

thughes commented May 30, 2025

cc @fabiobaltieri

@kartben kartben added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label May 31, 2025
@nashif nashif merged commit db726c4 into zephyrproject-rtos:main May 31, 2025
33 checks passed
@thughes thughes deleted the push-upkvvossurmn branch June 2, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Interrupt Controller Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants