Skip to content

Feature/mixin quantifiers - #2643

Open
LlamaLad7 wants to merge 8 commits into
minecraft-dev:devfrom
LlamaLad7:feature/mixin-quantifiers
Open

Feature/mixin quantifiers#2643
LlamaLad7 wants to merge 8 commits into
minecraft-dev:devfrom
LlamaLad7:feature/mixin-quantifiers

Conversation

@LlamaLad7

Copy link
Copy Markdown
Contributor

Basic support for Mixin quantifiers, along with related bugfixes. Commits best consumed invididually.

Closes #2518

Mixin MemberInfo strings are only tangentially related to concrete MemberReferences. We also introduce a MemberMatcher superinterface for resolving things.
Really nothing is "ambiguous" (except perhaps to the reader), it's just that Mixin's default behaviour of matching the first method in a given class is unexpected, but I'll leave the terminology for now. Adding any explicit quantifier suppresses this inspection, including `{1}` if that's really what's desired.

Additionally, adding a wildcard is almost never the correct fix. Choosing one of the available descriptors is what should be offered instead, but I'll leave that too for now.
@LlamaLad7
LlamaLad7 force-pushed the feature/mixin-quantifiers branch from 81a1c74 to ffdb662 Compare September 3, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mixin wildcard * is not properly accounted for

2 participants