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

Jump to bottom

MISRA C - Avoid nested // and /* in comments #10270

Closed
ceolin opened this issue Sep 27, 2018 · 0 comments · Fixed by #11423
Closed

MISRA C - Avoid nested // and /* in comments #10270

ceolin opened this issue Sep 27, 2018 · 0 comments · Fixed by #11423
Labels
area: MISRA-C Feature A planned feature with a milestone priority: medium Medium impact/importance bug
Milestone
v1.14.0

Comments

@ceolin
Copy link
Member

ceolin commented Sep 27, 2018

C99 does not allow nested comment using /* */ and use nested // can be trick and lead to errors. In Zephyr we have few occurrences of this, mostly in the following pattern:

/* blah blah blah https://www.zephyrproject.org/ */

change it seems wrong, so we have to create a deviation for these cases and fix the others.

part of: #9552

MISRA C rule 3.1
@ceolin ceolin added Feature A planned feature with a milestone priority: medium Medium impact/importance bug area: MISRA-C labels Sep 27, 2018
@ceolin ceolin added this to the v1.14.0 milestone Sep 27, 2018
@pfl pfl mentioned this issue Nov 15, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: MISRA-C Feature A planned feature with a milestone priority: medium Medium impact/importance bug
Projects
None yet
Milestone
v1.14.0
Development

Successfully merging a pull request may close this issue.

Avoid nested C/C++ comments pfl/zephyr
1 participant
@ceolin