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.
the missing break's *have* to be a bug. Or are the various web hooks really are additive?