Skip to content

Fix specialized std::vector<bool> handling#23

Merged
SemaiCZE merged 1 commit into
SemaiCZE:masterfrom
alexey-pelykh:fix/std_vector_bool
Apr 23, 2024
Merged

Fix specialized std::vector<bool> handling#23
SemaiCZE merged 1 commit into
SemaiCZE:masterfrom
alexey-pelykh:fix/std_vector_bool

Conversation

@alexey-pelykh

Copy link
Copy Markdown
Contributor

No description provided.

@SemaiCZE

Copy link
Copy Markdown
Owner

Hi,

as you can see this breaks all GCC builds. I know that vector<bool> is tricky because of multiple different platform specific implementations, but I won't merge it this way. I'd rather have a failing just MacOS than Mac and Linux on the most popular compiler.

Thanks

@alexey-pelykh

Copy link
Copy Markdown
Contributor Author

I'll look closer, I don't anticipate it to be impossible at all.

@alexey-pelykh

Copy link
Copy Markdown
Contributor Author

That's one way to tackle the issue, checked on latest GCC and Clang

@alexey-pelykh

Copy link
Copy Markdown
Contributor Author

Cool, so it passes the tests. Obviously g++-9: command not found isn't related

@SemaiCZE SemaiCZE merged commit eddf900 into SemaiCZE:master Apr 23, 2024
@alexey-pelykh alexey-pelykh deleted the fix/std_vector_bool branch April 23, 2024 17:09
@SemaiCZE

Copy link
Copy Markdown
Owner

Thanks for the PR 👍. I updated ubuntu and mac versions in the CI

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.

2 participants