Fixes #74

[TheBusyBiscuit]

Short Description

This pull request fixes Issue #74 by adding a break statement after a Food Synthesizer has been located in the inventory, this will also benefit the performance as it reduces unnecessary iterations through the inventory.

Additions/Changes/Removals

• Refactored GlowEnchant to use a HashSet
• Added some Nonnull and Nullable annotations
• Removed the registration of the LiteXpansion category (Categories are automatically registered once they have at least one item)

Related Issues

Resolves #74

Checklist

• [ ] I have fully tested the proposed changes and promise that they will not break everything into chaos.
• [ ] I have also tested the proposed changes in combination with base Slimefun and made sure nothing breaks/unexpected happens.
• [x] I followed the existing code standards and didn't mess up the formatting.
• [ ] I did my best to add documentation to any public classes or methods I added which may not be obvious to maintainers.
• [x] I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values

[J3fftw1]

Thanks for adding this btw ❤️

[TheBusyBiscuit]

You're welcome :D I wanted to fix at least one or two bugs in LX this month anyway, but but I decided to also include that Glow Enchant refactoring I've been calling out for weeks XD

[TheBusyBiscuit]

And I only now realized I worked of off master :NotLikeThis: Oh well

[J3fftw1]

let me merge it for you