Skip to content

fixed #11917 (Default to native platform in Windows builds)#5428

Merged
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:native
Sep 11, 2023
Merged

fixed #11917 (Default to native platform in Windows builds)#5428
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:native

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@danmar danmar merged commit 64cd09a into cppcheck-opensource:main Sep 11, 2023
@firewave firewave deleted the native branch September 11, 2023 11:00
@chrchr-github
Copy link
Copy Markdown
Collaborator

chrchr-github commented Sep 18, 2023

What is the native platform supposed to represent? We have simplifyMicrosoftStringFunctions(), which does not run for native.
Edit: We should probably adjust isWindows().

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 18, 2023

The native was meant to match the compiler that built the cppcheck binary. So.. the sizes determined at compile time.

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.

3 participants