Global PacketExistingPlayer::name [16]

SpadesX commits before 0493af85e126ab66e056ff667b1884b328c28f8e allowed player names 16 bytes and one NULL terminator long, instead of 15 bytes and one NULL terminator. @betterspadesbug BetterSpades commits before [fc7ab028345306391081953ec9383580b835f391](fc7ab028345306391081953ec9383580b835f391) did not properly handle player names which were too long, potentially causing crashes or funkiness after the actual name.

Struct PacketOrientationData

SpadesX copied and pasted the same code used for piqueserver, resulting in the NaN bug being copied with it.