Repository navigation
Complete the various renamings of terms belonging to feature checked_type_aliases - #164528
Conversation
|
r? @chenyukang rustbot has assigned @chenyukang. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
|
||
| if !usage_spans.is_empty() { | ||
| // First, check if the ADT/LTA is (probably) cyclical. We say probably here, since we're | ||
| // First, check if the ADT is (probably) cyclical. We say probably here, since we're |
There was a problem hiding this comment.
Since #141030 we no longer consider CTAs here.
|
with file name changed, the comment also need to updated? tests/ui/checked-type-aliases/constrained-late-bound-regions.rs:2 // Weak alias types constrain late-bound regions ...
tests/ui/checked-type-aliases/unconstrained-late-bound-regions.rs:1 // Weak alias types only constrain ...
tests/ui/coherence/orphan-check-free-aliases-covering.rs:1 // Weak aliases cover type parameters ...
tests/ui/coherence/orphan-check-free-aliases-not-covering.rs:1 // Weak aliases might not cover ... |
| // | ||
| // Listed below are ill-formed type system entities which we don't reject since they appear inside | ||
| // the definition of (eager) type aliases. These type aliases are intentionally not referenced from | ||
| // anywhere to prevent the eagerly expanded / instantiated aliased types from getting wfchecked | ||
| // the definition of (unchcked) type aliases. These type aliases are intentionally not referenced |
There was a problem hiding this comment.
| // the definition of (unchcked) type aliases. These type aliases are intentionally not referenced | |
| // the definition of (unchecked) type aliases. These type aliases are intentionally not referenced |
|
r=me after fixed issues in comment. |
Oops, I only did a case sensitive ripgrep for weak, eager & lazy ^^'. Gonna use |
And "eager (type alias)" with "unchecked (type alias)".
|
@bors r=chenyukang rollup |
in accordance with the corresponding feature name, `checked_type_aliases`.
Complete the various renamings of terms belonging to feature `checked_type_aliases` Various terms used in the context of feature `checked_type_aliases` have undergone renamings. See PRs rust-lang#140249 and rust-lang#158758. However, these PRs didn't update every single occurrence. This PR rectifies that. For context, the renamings are 1. weak (alias (type)) → free (alias (type)) 2. lazy (type alias) → checked (type alias), LTA → CTA 3. eager (type alias) → unchecked (type alias) Moreover, I've renamed `tests/ui/checked-type-alias/` to `tests/ui/checked-type-aliases/` in accordance with the name of the feature, `checked_type_aliases`. <sub>(No LLM was or will be used by me during the entire creation process of this PR)</sub>
Rollup of 9 pull requests Successful merges: - #164792 (rustc_codegen_cranelift subtree update) - #163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run) - #164781 (Bump dist-x86_64-illumos to Ubuntu 22.04) - #160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments) - #163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700) - #164167 (Account for `use foo as bar;` on resolution error suggestions) - #164438 (Make async drop shim proxy pinned) - #164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`) - #164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
Rollup merge of #164528 - fmease:cta-renamings, r=chenyukang Complete the various renamings of terms belonging to feature `checked_type_aliases` Various terms used in the context of feature `checked_type_aliases` have undergone renamings. See PRs #140249 and #158758. However, these PRs didn't update every single occurrence. This PR rectifies that. For context, the renamings are 1. weak (alias (type)) → free (alias (type)) 2. lazy (type alias) → checked (type alias), LTA → CTA 3. eager (type alias) → unchecked (type alias) Moreover, I've renamed `tests/ui/checked-type-alias/` to `tests/ui/checked-type-aliases/` in accordance with the name of the feature, `checked_type_aliases`. <sub>(No LLM was or will be used by me during the entire creation process of this PR)</sub>
Rollup of 9 pull requests Successful merges: - rust-lang/rust#164792 (rustc_codegen_cranelift subtree update) - rust-lang/rust#163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run) - rust-lang/rust#164781 (Bump dist-x86_64-illumos to Ubuntu 22.04) - rust-lang/rust#160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments) - rust-lang/rust#163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700) - rust-lang/rust#164167 (Account for `use foo as bar;` on resolution error suggestions) - rust-lang/rust#164438 (Make async drop shim proxy pinned) - rust-lang/rust#164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`) - rust-lang/rust#164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
Rollup of 9 pull requests Successful merges: - rust-lang/rust#164792 (rustc_codegen_cranelift subtree update) - rust-lang/rust#163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run) - rust-lang/rust#164781 (Bump dist-x86_64-illumos to Ubuntu 22.04) - rust-lang/rust#160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments) - rust-lang/rust#163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700) - rust-lang/rust#164167 (Account for `use foo as bar;` on resolution error suggestions) - rust-lang/rust#164438 (Make async drop shim proxy pinned) - rust-lang/rust#164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`) - rust-lang/rust#164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
|
Note This PR was benchmarked as part of triage of its containing rollup: triage URL. Finished benchmarking commit (575f569): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)This perf run didn't have relevant results for this metric. CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. Artifact size: 406.24 MiB -> 406.29 MiB (0.01%) |
Various terms used in the context of feature
checked_type_aliaseshave undergone renamings. See PRs #140249 and #158758. However, these PRs didn't update every single occurrence. This PR rectifies that.For context, the renamings are
Moreover, I've renamed
tests/ui/checked-type-alias/totests/ui/checked-type-aliases/in accordance with the name of the feature,checked_type_aliases.(No LLM was or will be used by me during the entire creation process of this PR)