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

Use arx::std namespace #5

Conversation

matthijskooijman
Copy link
Contributor

This a companion to hideakitai/ArxTypeTraits#2. It updates the embedded ArxTypeTraits to that version and renames all std:: references that are provided by ArxTypeTraits (everything except for the containers) to arx::std::.

This also contains some related changes in ArxContainer that should be extracted eventually, but I just made them here for ease of review.

@matthijskooijman matthijskooijman changed the title Use arx std namespace Use arx::std namespace Sep 23, 2020
@matthijskooijman matthijskooijman force-pushed the use_arx_std_namespace branch 2 times, most recently from 6e1dbf9 to 1b2c8dc Compare September 24, 2020 09:49
@hideakitai
Copy link
Owner

Can you add MsgPack/util/ArxTypeTraits/ArxTypeTraits/replace_minmax_macros.h?

This fixes compatibility with libraries such as ArduinoSTL which make
the standard header available on platforms that do not normally have it.
@matthijskooijman
Copy link
Contributor Author

w00ps, done :-)

@matthijskooijman
Copy link
Contributor Author

I think this PR is no longer relevant, all changes have been superseded by changes to ArxTypeTraits, so I'm closing this one.

@matthijskooijman matthijskooijman deleted the use_arx_std_namespace branch September 25, 2020 21:16
@matthijskooijman matthijskooijman restored the use_arx_std_namespace branch September 25, 2020 21:16
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