Skip to content

Inlining methods/functions in core. #4467

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 13, 2013
Merged

Inlining methods/functions in core. #4467

merged 1 commit into from
Jan 13, 2013

Conversation

gifnksm
Copy link
Contributor

@gifnksm gifnksm commented Jan 13, 2013

Inlining small methods/functions in core.
Also inlining some functions which take functions as arguments.

Also inlining some functions which take functions as arguments.
@catamorphism
Copy link
Contributor

Seems harmless. I'm a bit worried about the amount of clutter, but I filed a separate issue on that - #4472 ; this is probably the best we can do right now.

catamorphism added a commit that referenced this pull request Jan 13, 2013
Inlining methods/functions in core.
@catamorphism catamorphism merged commit 50cd3c1 into rust-lang:incoming Jan 13, 2013
@catamorphism
Copy link
Contributor

Thanks!

@gifnksm gifnksm deleted the inlining-core branch January 14, 2013 01:38
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 17, 2024
…et,GuillaumeGomez

Allow more attributes in `clippy::useless_attribute`

Fixes rust-lang#12753
Fixes rust-lang#4467
Fixes rust-lang#11595
Fixes rust-lang#10878

changelog: [`useless_attribute`]: Attributes allowed on `use` items now include `ambiguous_glob_exports`, `hidden_glob_reexports`, `dead_code`, `unused_braces`, and `clippy::disallowed_types`.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants