Little Logistics

Little Logistics

12M Downloads

Remove unused boilerplate from ShippingMod.java

MuradAkh opened this issue ยท 1 comments

commented

Reported by Valhelsia team:

Should probably remove the setup(...), enqueueIMC(...), processIMC(...) methods, and the RegistryEvents class from here: https://github.com/MuradAkh/LittleLogistics/blob/main-1.18/src/main/java/dev/murad/shipping/ShippingMod.java since that just adds some log spam (such as the "HELLO from [blah]" text) - it's all example code from Forge, not needed for anything, just meant to teach you how to do a few things. Make sure to remove them from the constructor too so they don't get added as event listeners.

In the 1.16 branches there's a few other things too, such as onServerStarting(...) that are unused other than just spamming the log, too. It's not the worst set of issues in the world, but small improvements like that go a long way in large packs (every line in the log that I can remove safely makes it much easier to troubleshoot actual issues).

commented

Resolved