-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Fix misspelling of "constraint" as "contraint" #98113
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
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
@@ -982,7 +982,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> { | |||
} | |||
} | |||
|
|||
fn assoc_ty_contraint_param_error_emit(&self, data: &ParenthesizedArgs) -> () { | |||
fn assoc_ty_constraint_param_error_emit(&self, data: &ParenthesizedArgs) -> () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you rename this to something starting with emit_
? Perhaps emit_bad_parenthesized_trait_in_assoc_ty
? It's more consistent with other functions that emit errors.
Also, please remove the -> ()
, because those are redundant.
a644feb
to
c8b411e
Compare
@bors r+ rollup |
📌 Commit c8b411e has been approved by |
Rollup of 7 pull requests Successful merges: - rust-lang#97822 (Filter out intrinsics if we have other import candidates to suggest) - rust-lang#98026 (Move some tests to more reasonable directories) - rust-lang#98067 (compiler: remove unused deps) - rust-lang#98078 (Use unchecked mul to compute slice sizes) - rust-lang#98083 (Rename rustc_serialize::opaque::Encoder as MemEncoder.) - rust-lang#98087 (Suggest adding a `#[macro_export]` to a private macro) - rust-lang#98113 (Fix misspelling of "constraint" as "contraint") Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
I misspelled a function name a while back