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 invalid static_cast. #11

Closed
wants to merge 1 commit into from

Conversation

BenWiederhake
Copy link

This fixes the issue described in https://svn.boost.org/trac/boost/ticket/11673 ("Invalid static_cast"). All regression tests pass. In case you want a PR against master, see my branch "master".

@MarcelRaad
Copy link
Contributor

This change is also included in #9.

@BenWiederhake
Copy link
Author

Indeed! I'm closing this in favor of the better explained and executed #9. (This PR is kind of hacked together.)

@artyom-beilis
Copy link
Member

No #9 isn't good fix it also removes "map" instread of unordered map support (there is a good reason to keep it - no details there)

#11 is actually better one. It can be pushed

@artyom-beilis
Copy link
Member

Ok... the problem is that the code that used unordered_map had never actually was used due to a simple fact that version.hpp wasn't included and it never got to be compiled.

I consider removing entire unordered part as it was never properly tested

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