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

Patch to support pre-c++11 compiler #18

Open
GoogleCodeExporter opened this issue Oct 6, 2015 · 5 comments
Open

Patch to support pre-c++11 compiler #18

GoogleCodeExporter opened this issue Oct 6, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

Hi, All, I made some changes for old gcc:

* Add a btree_config.h
* Config type_traits according to whether c++11 is support and enabled by 
compiler.
* Allow user define size_type to be size_t, which make the size() interface of 
btree containers compatible with STL totally.
* Remove dependency of the __COUNTER__ macro in bench test, which was 
introduced since gcc 4.3, make cpp-btree compile with older compiler.

Tested on gcc 4.1.2 & 4.7.2, w/ & w/o c++11 enabled.

Original issue reported on code.google.com by chen3feng on 8 Apr 2013 at 5:00

Attachments:

@GoogleCodeExporter
Copy link
Author

[deleted comment]

@GoogleCodeExporter
Copy link
Author

And fixed all warnings of strict-aliasing.

Original comment by chen3feng on 8 Apr 2013 at 5:06

@GoogleCodeExporter
Copy link
Author

[deleted comment]

@GoogleCodeExporter
Copy link
Author

Your new file "btree_config.h" does not have a license listed at the top. Is 
your intent to make that AL2? I'm very interested in this patch, but cannot use 
it as is.

Thanks.

Original comment by sor...@gmail.com on 8 Dec 2013 at 10:40

@GoogleCodeExporter
Copy link
Author

I declare: I accept make the new file btree_config.h follow AL2,so you are
welcome to change it as your wish, thanks.
2013-12-9 AM6:40于 <cpp-btree@googlecode.com>写道:

compiler

Is your intent to make that AL2? I'm very interested in this patch, but
cannot use it as is.

Original comment by chen3feng on 9 Dec 2013 at 12:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant