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.

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

Fix MISRA C++ 2023 rule 12.3.1 "The union keyword shall not be used" #633

Closed
mohammedRafeeque opened this issue Jun 14, 2024 · 1 comment
Assignees
Labels
bug Something isn't working c++ C++ language generator
Milestone

Comments

@mohammedRafeeque
Copy link

mohammedRafeeque commented Jun 14, 2024

Zserio version and language
Zserio: latest from master
Commit date: 6 days ago (6/4/2024 7:12:28 PM)
Commit hash: b8cc6c0

Describe the bug
1 Misra warning getting reported

  1. union UntypedHolder
<style> </style>
SNo Rule Severity Message Filename Linenumber
1677 MISRACPP2023-12_3_1-a 2 The 'union' keyword shall not be used /zserio_runtime/include/roadgraph_reliable/zserio_runtime/zserio/AnyHolder.h 296

Please check possibility of fixing the same.

Steps to reproduce the behavior:
SCA run using MISRA 2023 configuration.

Expected behavior
No SCA warnings for MISRA 2023 ruleset

Additional context
N.A

@Roland-Homeier @mikir FYI.

@mohammedRafeeque mohammedRafeeque added the bug Something isn't working label Jun 14, 2024
@mikir mikir changed the title Fix MISRA C++ 2023 rule MISRACPP2023-12_3_1-a The 'union' keyword shall not be used Fix MISRA C++ 2023 rule 12.3.1 "The union keyword shall not be used" Jun 18, 2024
@mikir mikir added the c++ C++ language generator label Jun 18, 2024
@mikir mikir added this to the 2.15 milestone Jun 18, 2024
@Mi-La
Copy link
Contributor

Mi-La commented Aug 8, 2024

The sonar rule M23_158 is already enabled and we have a single accepted issue as an exception:

Module Line Command Link
AnyHolder.h 296 union UntypedHolder SonarCloud

@Mi-La Mi-La self-assigned this Aug 8, 2024
@Mi-La Mi-La closed this as completed Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working c++ C++ language generator
Projects
None yet
Development

No branches or pull requests

3 participants