Lots of fixes

[josephcsible]

• Stop calling the postInit method, since we don't actually do anything in it.
• Remove ServerProxy, and just use CommonProxy on the server.
• Stop overriding getRenderType, since the superclass method is the same as ours.
• Remove some unnecessary variables.
• Remove an unnecessary size() check before an iteration.
• Avoid mixing if-else with ?:.
• Remove an unnecessary return statement.
• Just return if we know our return value, instead of setting it then
• immediately returning what we set.
• Remove more unnecessary casts.
• Simplify some unnecessarily complicated logic.
• Remove some unused methods.
• Use Java's iteration-style for instead of C-style for where possible.
• Throw an exception if we add an enum value but forget to add a case for it.
• Rewrite ByteUtil to use overloading, remove unnecessary operations, and generally be much cleaner.
• Fix a typo in our error message.

[frodare]

Changes look good, however please update the ByteUtilTest class to reflect your changes.

[frodare]

There are some failing tests to investigate. The testCopyArray test can be removed if the copy array method is not being used.