Fixed some Law of Demeter violations in DockController.
ship.addUpgrade() now accepts ShieldType instead of Shield.
Added pmd to gradle build. -x pmdMain to build without running pmd.
PMD'd the project. Added findbugs to the gradle build. -x findbugsMain to exclude. Changed to generate html reports because they're more useful without the findbugs gui.
Merging master into jack/dev while working on pmd/findbugs
Merge branch 'master' of https://github.com/gem-5/last-adventures

Conflicts:
	src/main/java/edu/gatech/gem5/game/controllers/DockController.java
Added javadocs to fields in Data class.
Refactored AbstractNPC to remove duplicate code in its subclasses.
Fixed no whitespace b/t {} error in Reader files.
Cleaned up unused imports, still like 500 to go...
Only 450 imports left.
Findbugs'd most of the project, still many errors that can't be gotten rid of right now/ever.
Made pirate view file a member of Pirate and included mising import in Reader to merge.
Reverted the leftover code from the fix that tried to get rid of writing to static field Stage in LastAdventures.
"Fixed Findbugs error in StarType exposing internal."
Did Findbugs fixes on files that needed to use StringBuilder instead of String.
Fixed some more random casting and privacy bugs Findbugs complained about.
Fixed Findbugs Unread fields erros. Removed the useless fields.
Fixed some dead stores that Findbugs complained about.
Merge remote-tracking branch 'origin/master' into james/dev