Metallurgy 4: Reforged

Describe the bug:
A new ArrayList is created for every player every tick

Line 66 in 20f1df9

    
           List<ItemStack> list = Lists.newArrayList(pl.getArmorInventoryList().iterator());

From a quick check, this is doubled depending on OsmiumLutetium availability?

To Reproduce:
Profiling & Code review

Expected behavior:
Avoid memory allocation in event handlers.

Versions:

Metallurgy: 1.0.2
Conflicting mod (if regarding mod integration): none
Other mods you think could cause issues: none
Forge: 14.23.5.2854
Minecraft: 1.12.2
Modpack (if available): https://www.curseforge.com/minecraft/modpacks/fsg-arcania-craft/files/3025855

Crash Report (if available):
n/a

Latest Log (if necessary):
n/a

Screenshots (if available):
n/a

Additional context (optional):
n/a

oof, that code has been there for quite a while, thanks for reporting it