Another round of findbug and checkstyle fixes.

Every time I pick up changes from upstream I seem to spend 20 minutes
fixing things that can be easily caught by static code analysis.
adding compareTo to a bean without equals and hashCode *will* lead to problems. That's why findbugs flags it.