SpotBugs violations

src/main/java/hudson/plugins/jacoco/JacocoPublisher.java

@@ -1,6 +1,5 @@
 package hudson.plugins.jacoco;
 
-import edu.umd.cs.findbugs.annotations.NonNull;
 import hudson.*;
 import hudson.model.Result;
 import hudson.model.AbstractBuild;
@@ -51,18 +50,18 @@ public class JacocoPublisher extends Recorder implements SimpleBuildStep {
      * <p>
      * TODO: define a configuration mechanism.
      */
-    public Rule rule;
+    // public Rule rule;
     @Deprecated
     public transient String includes;
     @Deprecated
     public transient int moduleNum;
     /**
      * {@link hudson.model.HealthReport} thresholds to apply.
      */
-    public JacocoHealthReportThresholds healthReports;
+    private JacocoHealthReportThresholds healthReports;
 
     // Delta coverage thresholds to apply
-    public JacocoHealthReportDeltaThresholds deltaHealthReport;
+    JacocoHealthReportDeltaThresholds deltaHealthReport;
 
 
     /**
@@ -799,27 +798,8 @@ public BuildStepMonitor getRequiredMonitorService() {
         return BuildStepMonitor.NONE;
     }
 
-    @Override
-    public BuildStepDescriptor<Publisher> getDescriptor() {
-        return (BuildStepDescriptor<Publisher>)super.getDescriptor();
-    }
-
-    /**
-     * @deprecated
-     *      use injection via {@link Jenkins#getInjector()}
-     */
-    public static /*final*/ BuildStepDescriptor<Publisher> DESCRIPTOR;
-
-    private static void setDescriptor(BuildStepDescriptor<Publisher> descriptor) {
-        DESCRIPTOR = descriptor;
-    }
-
     @Extension @Symbol("jacoco")
     public static class DescriptorImpl extends BuildStepDescriptor<Publisher> {
-        public DescriptorImpl() {
-            super(JacocoPublisher.class);
-            setDescriptor(this);
-        }
 
 		@NonNull
         @Override

src/main/java/hudson/plugins/jacoco/model/CoverageObject.java

@@ -66,12 +66,12 @@ public abstract class CoverageObject<SELF extends CoverageObject<SELF>> {
 	/**
 	 * Variables used to store which child has to highest coverage for each coverage type.
 	 */
-	public int maxClazz=1;
-	public int maxMethod=1;
-	public int maxLine=1;
-	public int maxComplexity=1;
-	public int maxInstruction=1;
-	public int maxBranch=1;
+	private int maxClazz=1;
+	private int maxMethod=1;
+	private int maxLine=1;
+	private int maxComplexity=1;
+	private int maxInstruction=1;
+	private int maxBranch=1;
 
 	private volatile boolean failed = false;
 

src/main/java/hudson/plugins/jacoco/report/CoverageReport.java

@@ -51,7 +51,7 @@ private CoverageReport(JacocoBuildAction action) {
 //	private String complexityColor;
 //	private String lineColor;
 //	private String methodColor;
-	public JacocoHealthReportThresholds healthReports;
+	private JacocoHealthReportThresholds healthReports;
 
 	/**
 	 * Loads the exec files using JaCoCo API. Creates the reporting objects and the report tree.