Skip to content

AP Fixes#591

Closed
LlamaLad7 wants to merge 2 commits intoSpongePowered:masterfrom
LlamaLad7:sponge
Closed

AP Fixes#591
LlamaLad7 wants to merge 2 commits intoSpongePowered:masterfrom
LlamaLad7:sponge

Conversation

@LlamaLad7
Copy link
Copy Markdown
Contributor

@LlamaLad7 LlamaLad7 commented Jul 24, 2022

This fixes 2 issues:

  1. Even when lambdas and such have mappings, they are not remapped due to Mirror being prioritised over ASM.
  2. Intersection types are not resolved properly by the AP and as such do not get remapped.

This PR fixes those and builds a fully working version of FabricAPI, with all lambdas properly in the refmaps.
Closes #590 and closes #507

@LlamaLad7 LlamaLad7 changed the title AP: Fix remapping of synthetic members in resolvable classes. AP Fixes Jul 24, 2022
@LlamaLad7
Copy link
Copy Markdown
Contributor Author

(Note to self that this may require some changes regarding code assuming that TypeHandles will have TypeMirrors)

@LlamaLad7 LlamaLad7 mentioned this pull request Aug 12, 2023
@LlamaLad7 LlamaLad7 closed this Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Synthetic members in resolvable classes are not remapped. Generic '&' operator / fabric

1 participant