Modernize type names in the compiler#26895
Conversation
jroesch
force-pushed
the
modernize-typeck-names
branch
from
e62e5f8 to
754aaea
Compare
|
|
|
@arielb1 totally okay with bike shedding or removal. I'll remove it and tack another commit on here. |
There was a problem hiding this comment.
I hate to ask for this, but I'd rather have ty::TypeError::CyclicTy (i.e., not pub use all the names from the enum)
There was a problem hiding this comment.
Yeah I wanted to do it, but wasn't sure how much pain it was to fix. I'll bite the bullet.
There was a problem hiding this comment.
would you be okay with just TypeError::Variant? I think it is slightly less verbose and just as clear.
|
lgtm |
|
r=me whenever you feel you've suffered enough :) |
|
+1, this always tripped me up. |
|
@bors r=nikomatsakis |
|
📌 Commit 19218ee has been approved by |
This PR modernizes some names in the type checker. The only remaining snake_case name in ty.rs is `ctxt` which should be resolved by @eddyb's pending refactor. We can bike shed over the names, it would just be nice to bring the type checker inline with modern Rust. r? @eddyb cc @nikomatsakis
6 participants
This PR modernizes some names in the type checker. The only remaining snake_case name in ty.rs is
ctxtwhich should be resolved by @eddyb's pending refactor. We can bike shed over the names, it would just be nice to bring the type checker inline with modern Rust.r? @eddyb
cc @nikomatsakis