Merge pull request #1027 from hazendaz/master

Move rest of 'def' to actual object types

Author: hazendaz

src/main/groovy/org/codehaus/mojo/spotbugs/BaseViolationCheckMojo.groovy

@@ -125,7 +125,7 @@ abstract class BaseViolationCheckMojo extends AbstractMojo {
 
         Node xml = new XmlParser().parse(outputFile)
 
-        def bugs = xml.BugInstance
+        NodeList bugs = xml.BugInstance
         int bugCount = bugs.size()
         log.info("BugInstance size is ${bugCount}")
 
@@ -151,7 +151,7 @@ abstract class BaseViolationCheckMojo extends AbstractMojo {
 
         int bugCountAboveThreshold = 0
         for (i in 0..bugCount-1) {
-            def bug = bugs[i]
+            Node bug = bugs[i]
             int priorityNum = bug.'@priority' as Integer
             String priorityName = SpotBugsInfo.spotbugsPriority[priorityNum]
             String logMsg = priorityName + ': ' + bug.LongMessage.text() + SpotBugsInfo.BLANK +
@@ -195,9 +195,9 @@ abstract class BaseViolationCheckMojo extends AbstractMojo {
         !sourceFiles.isEmpty()
     }
 
-    private void printBugs(int total, def bugs) {
+    private void printBugs(int total, NodeList bugs) {
         for (i in 0..total - 1) {
-            def bug = bugs[i]
+            Node bug = bugs[i]
             log.error(bug.LongMessage.text() + SpotBugsInfo.BLANK + bug.SourceLine.'@classname' + SpotBugsInfo.BLANK +
                 bug.SourceLine.Message.text() + SpotBugsInfo.BLANK + bug.'@type')
         }