Named opt-in trait impls
Nobody has claimed this yet.
- Dominant language
- Markdown
- Stars
- 6.6k
- Forks
- 1.7k
- Avg merge
- 16h 14m
- Merged PRs (30d)
- 1
Description
We all know that from inference perspective, all impls are "global" if not excluded with coherence. So this is actually part of the "inference API" of a crate. (But not yet well documented by tools like rustdoc to this day, sadly)
I've seen many tragedies that are straightforward outcome of this.
But i feel we should have patching tools that move ourselves forward, if this is not solvable problem in the near future.
What if we could opt-out an impl from the implicit parts of inference?
Strawman proposal: Named opt-in trait impls.
#[optin_impl(u8_idx)]
impl Index<u8> for [T] {
// impl
}
fn foo1() {
dbg!([1,2,3][3u8]); // Doesn't compile, because no impls is selectable
dbg!([1,2,3][3usize]); // Works fine as always, with `usize` as idx,
dbg!([1,2,3][3]); // Works fine as always, with `usize` as idx,
}
#[use_optin_impls(u8_idx, ...)]
fn foo2() {
dbg!([1,2,3][3u8]); // Works fine, with `u8` as idx,
dbg!([1,2,3][3usize]); // Works fine, with `usize` as idx,
dbg!([1,2,3][3]); // Doesn't compile, because two impls are both in effect
}
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names no files, tests, or implementation entry points. Start by reviewing the proposed Rust syntax and its interaction with trait coherence and inference, then determine the design and RFC work needed; done would require an agreed, implementable proposal rather than a small code change.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100