Commit Graph
2 Commits
Author SHA1 Message Date
riking fe6ce01037 Define hashCode() for TimeUnit as it is being used in a HashMap
TimeUnit is being used as the key to a map in TimeDiffUtil, so we must define a hashCode() consistent with its equals() method.

The implementation is the same as java.lang.Long.hashCode().
2013-08-30 18:45:57 -07:00
riking 5c6cd57a4d Use a MutableBoolean to synchronize on instead of a Boolean
The code synchronizes on a boxed primitive constant, such as an Boolean.

Since there normally exist only two Boolean objects, this code could be synchronizing on the same object as other, unrelated code, leading to unresponsiveness and possible deadlock.

This commit uses the MutableBoolean class from commons-lang, which is included in bukkit.jar.
2013-08-30 18:40:48 -07:00