General Fixup

Replaced all unnecessary for loops with foreach

Removed unnecessary throw statement on end of method

Removed unnecessary checking of when variable is null even when the fact
is already established

Fixed bug where player object was being checked to see if it equaled
another string (GuiKitMenu:147)

Fixed when StringBuilder's extend method was being passed in a string
that was concatenated completely removing the performance enhancement of
ever using a string builder

Simplified Return statements (SEMANTICS)

Change from using the Bukkit scheduler call method to an inner class
(Bukkit Runnable) (SEMANTICS)

Renamed all empty catch blocks exception variables to, "ignored", so as
to be clear that the exception is ignored. (SEMANTICS)