GeyserMC / MCProtocolLib

A library for communication with a Minecraft client/server.
MIT License
714 stars 201 forks source link

Use built-in ResourceLocation type #811

Closed AlexProgrammerDE closed 1 month ago

Camotoy commented 1 month ago

I'm neutral on this and will delegate it to Bas or someone else to merge. I absolutely know that Keys are easier to use instead of loads of string splitting, but I did some testing and they do take up more memory, even with the minecraft prefix shared across all Key instances. So it's a tradeoff, and it's not one I'd prefer to push the merge button on.