search

Chocohead / Fabric-ASM

Just what Asie never wanted
Mozilla Public License 2.0
159 stars 19 forks source link

Fix/avoid re-entrance #24

Open LlamaLad7 opened 1 year ago

LlamaLad7 commented 1 year ago

Based on my other PR since I can't build otherwise. Mixin strictly disallows re-entrance during transformation so we should avoid evaluating args suppliers at transformation-time, instead storing the supplier and evaluating it at runtime. Currently if an arg supplier causes a classload for a class that has a mixin it will crash.

MrNavaStar commented 1 year ago

Running into this issue, merge would be great! (Thanks for the awesome library)