Skip to content

libextra: Introduce typed arenas. #11358

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

Merged
merged 1 commit into from
Jan 8, 2014
Merged

Conversation

pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Jan 7, 2014

A typed arena is a type of arena that can only allocate objects of one
type. It is 3x faster than the existing arena and 13x faster than malloc
on Mac.

r? @brson

@emberian
Copy link
Member

emberian commented Jan 7, 2014

this is awesome

/// Creates a new arena with preallocated space for the given number of
/// objects.
///
/// TODO(pcwalton): `contains_managed` check and bailout.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not just go ahead and add the fail!()?

A typed arena is a type of arena that can only allocate objects of one
type. It is 3x faster than the existing arena and 13x faster than malloc
on Mac.
bors added a commit that referenced this pull request Jan 8, 2014
A typed arena is a type of arena that can only allocate objects of one
type. It is 3x faster than the existing arena and 13x faster than malloc
on Mac.

r? @brson
@bors bors closed this Jan 8, 2014
@bors bors merged commit b7ff9c1 into rust-lang:master Jan 8, 2014
flip1995 pushed a commit to flip1995/rust that referenced this pull request Sep 12, 2023
…r=Jarcho

Rename incorrect_impls to non_canonical_impls, move them to warn by default

The wording/category of these feel too strong to me, I would expect most of the time it's linting the implementations aren't going to be *incorrect*, just unnecessary

changelog: rename `incorrect_clone_impl_on_copy_type` to [`non_canonical_clone_impl`]
changelog: rename `incorrect_partial_ord_impl_on_ord_type` to [`non_canonical_partial_ord_impl`]
changelog: Move [`non_canonical_clone_impl`], [`non_canonical_partial_ord_impl`] to suspicious
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.

6 participants