PatchworkMC / patchwork-patcher

Patches Forge mod jars, allowing them to be loaded by Fabric
GNU Lesser General Public License v3.0
316 stars 33 forks source link

Remove typetools todo and add a few comments to genInstanceRegistrars #79

Closed coderbot16 closed 3 years ago

coderbot16 commented 3 years ago

Based on my code inspection, the methods that we are calling already hit the non-typetools path, since we're already explicitly specifying the event class. TypeTools is only used for the methods where an explicit event class is not specified, but we do not generate calls to those methods.

I've also added a few comments in the nearby code that hopefully allow it to make a little more sense.