-
Notifications
You must be signed in to change notification settings - Fork 126
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
[build][Suggestion] Enable C++11 support; #198
Comments
I'm game, if nothing else just to get familiar with it. What are the On Mon, Feb 16, 2015 at 12:16 PM, Gregory King [email protected]
|
There are couple of features that I find useful:
(2) Disadvantages: There a few other features (but I haven't used them much) and so I will decline to comment (for example, R-Value references and std::move ...). |
How much porting would be involved or will the current C++ just work? I.e. On Mon, Feb 16, 2015 at 2:16 PM, Gregory King [email protected]
|
On Mon, Feb 16, 2015 at 4:00 PM, Lawrence Love [email protected] wrote:
|
I'll try and answer the question: C++11 is a nearly a superset of C++98/03 (it has a few language extension/new keywords, so using certain features of C++11 can be guaranteed to not compile if forced into C++98/C++03 compliant compiler, I haven't come across much C++98/C++03 (primarily collisions with new extensions/other features) code that doesn't properly compile when using C++11 - further there are a few deprecated features, including To enable using C++11: you can add In addition clang/llvm provides tools to migrate to using C++11 idioms.clang-moderize. Documentation of -std compliance mode for clang. Edit: small corrections; |
Thanks, that helps. I had looked at that Stackoverflow link but your On Mon, Feb 16, 2015 at 5:19 PM, Gregory King [email protected]
|
@kingtaurus, there are even more goodies from C++11. If we would move to C++11 as the default, we can loose the pthread dependency. This means that the |
I was hoping to see if others want to enable using C++11 standard during compilation. Its trivial to enable, however I don't want to start using C++11 features to update the code base until others are okay with it.
The text was updated successfully, but these errors were encountered: