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

C89/C90 standard (and variants) treated as unknown #365

Closed
firewave opened this issue Sep 2, 2024 · 0 comments
Closed

C89/C90 standard (and variants) treated as unknown #365

firewave opened this issue Sep 2, 2024 · 0 comments
Assignees
@firewave
Labels
bug

Comments

@firewave
Copy link
Collaborator

firewave commented Sep 2, 2024 

$ ./simplecpp -std=c89 test.cpp

test.cpp:1: dui error: unknown standard specified: 'c89'

The problem is that these standards have no value for __STDC_VERSION__ so simplecpp::getCStdString() returns an empty string which therefore treats it as unknown in simplecpp::preprocess().
@firewave firewave mentioned this issue Sep 2, 2024
Merged
@firewave firewave self-assigned this Sep 2, 2024
@firewave firewave added the bug label Sep 2, 2024
@firewave firewave changed the title C89/C90 standards (and variants) treated as unknown C89/C90 standard (and variants) treated as unknown Sep 2, 2024
@firewave firewave mentioned this issue Sep 9, 2024
Merged
firewave added a commit to firewave/simplecpp that referenced this issue Sep 9, 2024
@firewave
fixed danmar#365 - do not treat standards with empty define string as…
a209e14 
… unknown
firewave added a commit to firewave/simplecpp that referenced this issue Sep 9, 2024
@firewave
fixed danmar#365 - do not treat standards with empty define string as…
4df1d6c 
… unknown
@danmar danmar closed this as completed in 2c66d08 Sep 9, 2024
@danmar danmar modified the milestone: 1.2.0 Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@firewave firewave

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danmar @firewave