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 the pooled allocator for performance reasons #503

Merged
merged 1 commit into from
Feb 2, 2014
Merged

Use the pooled allocator for performance reasons #503

merged 1 commit into from
Feb 2, 2014

Conversation

normanmaurer
Copy link

Enjoy...

@codefromthecrypt
Copy link

Build fail is unrelated.

@normanmaurer Thanks! Have you yet sent a contributor agreement so that we can distribute your contribution? I know.. one line :)

1 similar comment
@codefromthecrypt
Copy link

Build fail is unrelated.

@normanmaurer Thanks! Have you yet sent a contributor agreement so that we can distribute your contribution? I know.. one line :)

@normanmaurer
Copy link
Author

Nope not yet... Let me check once at home and sign it ;)

Am 01.02.2014 um 20:35 schrieb Adrian Cole notifications@github.com:

Build fail is unrelated.

@normanmaurer Thanks! Have you yet sent a contributor agreement so that we can distribute your contribution? I know.. one line :)


Reply to this email directly or view it on GitHub.

@swankjesse
Copy link
Member

@normanmaurer thanks!

Naive question: why isn't fast allocator on by default?

@normanmaurer
Copy link
Author

@swankjesse basically when we released 4.0.0 we was not sure it is stable and now we don't wanted to change the default in a bugfix release. It will be default in 4.1.x and 5.x

@swankjesse
Copy link
Member

@normanmaurer sounds good to me.

@normanmaurer
Copy link
Author

@swankjesse @adriancole CLA signed..

codefromthecrypt pushed a commit that referenced this pull request Feb 2, 2014
Use the pooled allocator for performance reasons
@codefromthecrypt codefromthecrypt merged commit a12d6a6 into square:master Feb 2, 2014
@codefromthecrypt
Copy link

@normanmaurer awesome and… welcome!

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