Reading NBT data on the render function is very bad and should be fixed

[flashgnash]

Currently, in order to figure out whether to display the oxygen HUD, needsOxygen is called (method to determine whether the player is an android currently)

This function works by reading a player's NBT data, which as I understand is an expensive operation (on the server this is stored in memory, I'm unsure how it's accessed on the client)

There may be a solution in moving further up the call chain and stopping the render overlay handler being called in the first place if needsOxygen returns false (render overlay is only responsible for displaying info on whether a player is suffocating/how much oxygen they have left, which is irrelevant to androids who don't need it anymore)