General code quality fix-4

src/main/java/com/sonyericsson/jenkins/plugins/bfa/db/Semaphore.java

@@ -30,13 +30,13 @@
  */
 public class Semaphore {
 
-    private java.util.concurrent.Semaphore semaphore;
+    private java.util.concurrent.Semaphore semaphoreLock;
 
     /**
      * Standard constructor.
      */
     public Semaphore() {
-        semaphore = new java.util.concurrent.Semaphore(0);
+        semaphoreLock = new java.util.concurrent.Semaphore(0);
     }
 
     /**
@@ -46,19 +46,19 @@ public Semaphore() {
      */
     public void acquire() throws InterruptedException {
         int take;
-        if (semaphore.availablePermits() < 1) {
+        if (semaphoreLock.availablePermits() < 1) {
             take = 1;
         } else {
-            take = semaphore.availablePermits();
+            take = semaphoreLock.availablePermits();
         }
-        semaphore.acquire(take);
+        semaphoreLock.acquire(take);
     }
 
     /**
      * @see java.util.concurrent.Semaphore#release()
      * Releases a permit, returning it to the semaphore.
      */
     public void release() {
-        semaphore.release();
+        semaphoreLock.release();
     }
 }

src/main/java/com/sonyericsson/jenkins/plugins/bfa/graphs/ComputerGraphAction.java

@@ -117,7 +117,7 @@ private boolean isSlave() {
         if (computer == null) {
             return false;
         }
-        return (computer.getNode() instanceof Slave);
+        return computer.getNode() instanceof Slave;
     }
 
     @Override

src/main/java/com/sonyericsson/jenkins/plugins/bfa/model/indication/FoundIndication.java

@@ -50,6 +50,9 @@ public class FoundIndication {
      */
     protected static final String FILE_ENCODING = System.getProperty("file.encoding");
     private String matchingFile;
+    /**
+     * @deprecated, kept for backwards compatibility.
+     */
     @Deprecated
     private transient Integer matchingLine;
     private String pattern;

src/main/java/com/sonyericsson/jenkins/plugins/bfa/statistics/Statistics.java

@@ -198,7 +198,7 @@ public Statistics(@JsonProperty("projectName")    String projectName,
     }
 
         /**
-         * Deprecated, kept for backwards compatibility.
+         * @deprecated, kept for backwards compatibility.
          * @param projectName the project name.
          * @param buildNumber the build number.
          * @param startingTime the starting time.

src/test/java/com/sonyericsson/jenkins/plugins/bfa/db/SemaphoreTest.java

@@ -43,7 +43,7 @@ public class SemaphoreTest {
     @Test(timeout = 20000)
     public void testAcquireAndRelease() throws Exception {
         Semaphore semaphore = new Semaphore();
-        java.util.concurrent.Semaphore innerSemaphore = Whitebox.getInternalState(semaphore, "semaphore");
+        java.util.concurrent.Semaphore innerSemaphore = Whitebox.getInternalState(semaphore, "semaphoreLock");
         assertEquals("The semaphore should have no available permits to start with", 0,
                 innerSemaphore.availablePermits());
         semaphore.release();