AdoptOpenJDK/jitwatch — pull request #64 by neomatrix369 (2014-04-28 22:04:12+00:00)
Title: Sonar major warnings others

A number of Major warnings have been removed in this pull request, these include types like:

* Anon Inner Length
* Ncss Method Count (executable lines count)
* Simplify Boolean Expression
* Avoid Rethrowing Exception (redundant in this ...
* Switch statement found where default case is ...
* Malicious code vulnerability - Field should be …
* Avoid Print Stack Trace
* Method Names
* Local Variable Name
* Local Final Variable Name
* Final Class
* Hide Utility Class
* Parameter Assignment

A number of methods and classes have been refactored improving size and naming.

If the names given to them are not meaningful enough, please raise them in the respective places.

Not everything has been refactored - but as much as Boy Scouting possible has been done. Still a number of areas can be refactored.

As sanity check, please run JitWatch in both headless and UI modes to see, if it runs as expected.

These changes are the last of the Major warnings - a number of others have been omitted for the moment, some being false positives, others will be looked into at a later stage.