Made style changes

According to Clang's fugly ass
suggestions.

Author: J-manLans

pom.xml

@@ -55,6 +55,12 @@
             <artifactId>commons-collections4</artifactId>
             <version>4.5.0</version>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <version>4.9.6</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>
@@ -202,62 +208,4 @@
             </plugin>
         </plugins>
     </build>
-
-    <profiles>
-        <profile>
-            <id>jolla-dev</id>
-            <build>
-                <plugins>
-                    <!-- Surefire -->
-                    <plugin>
-                        <artifactId>maven-surefire-plugin</artifactId>
-                        <configuration>
-                            <includes>
-                                <include>com/thealgorithms/datastructures/trees/**/*Test.java</include>
-                            </includes>
-                        </configuration>
-                    </plugin>
-
-                    <!-- Pitest -->
-                    <plugin>
-                        <groupId>org.pitest</groupId>
-                        <artifactId>pitest-maven</artifactId>
-                        <configuration>
-                            <targetClasses>com.thealgorithms.datastructures.trees.LCA</targetClasses>
-                            <targetTests>com.thealgorithms.datastructures.trees.LCATest</targetTests>
-                        </configuration>
-                        <executions>
-                            <execution>
-                                <id>pitest</id>
-                                <phase>verify</phase> <!-- Runs only during verify -->
-                                <goals>
-                                    <goal>mutationCoverage</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                    </plugin>
-
-                    <!-- JaCoCo -->
-                    <plugin>
-                        <groupId>org.jacoco</groupId>
-                        <artifactId>jacoco-maven-plugin</artifactId>
-                        <executions>
-                            <execution>
-                                <goals>
-                                    <goal>prepare-agent</goal>
-                                </goals>
-                            </execution>
-                            <execution>
-                                <id>generate-code-coverage-report</id>
-                                <phase>verify</phase> <!-- Runs only during verify -->
-                                <goals>
-                                    <goal>report</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
-    </profiles>
 </project>

src/main/java/com/thealgorithms/datastructures/trees/LCA.java

@@ -8,13 +8,13 @@ private LCA() {
     }
 
     public static void main(String[] args) {
-        Scanner SCANNER = new Scanner(System.in);
+        Scanner scanner = new Scanner(System.in);
 
         // The adjacency list representation of a tree:
         ArrayList<ArrayList<Integer>> adj = new ArrayList<>();
 
         // v is the number of vertices and e is the number of edges
-        int v = SCANNER.nextInt();
+        int v = scanner.nextInt();
         int e = v - 1;
 
         for (int i = 0; i < v; i++) {
@@ -25,8 +25,8 @@ public static void main(String[] args) {
         int to;
         int from;
         for (int i = 0; i < e; i++) {
-            to = SCANNER.nextInt();
-            from = SCANNER.nextInt();
+            to = scanner.nextInt();
+            from = scanner.nextInt();
 
             adj.get(to).add(from);
             adj.get(from).add(to);
@@ -42,8 +42,8 @@ public static void main(String[] args) {
         dfs(adj, 0, -1, parent, depth);
 
         // Inputting the two vertices whose LCA is to be calculated
-        int v1 = SCANNER.nextInt();
-        int v2 = SCANNER.nextInt();
+        int v1 = scanner.nextInt();
+        int v2 = scanner.nextInt();
 
         // Outputting the LCA
         System.out.println(getLCA(v1, v2, depth, parent));

src/test/java/com/thealgorithms/ciphers/MyPlayfairCipherTest.java

@@ -7,16 +7,16 @@
 
 public class MyPlayfairCipherTest {
     private PlayfairCipher playfair;
-    private final String keyword = "KEYWORD";
+    private static final String KEYWORD = "KEYWORD";
 
     @BeforeEach
     public void setup() {
-        playfair = new PlayfairCipher(keyword);
+        playfair = new PlayfairCipher(KEYWORD);
     }
 
     @Test
     void shouldEncryptAndDecryptDuringSameRowDigraph() {
-        String plaintext = keyword.substring(0, 2);
+        String plaintext = KEYWORD.substring(0, 2);
 
         String encrypted = playfair.encrypt(plaintext);
         String decrypted = playfair.decrypt(encrypted);

src/test/java/com/thealgorithms/datastructures/trees/LCATest.java

@@ -24,7 +24,7 @@ public class LCATest {
      *        9   8
      * <pre>
      */
-    private final static String TREE = "10\n0\n1\n0\n2\n1\n5\n5\n6\n2\n4\n2\n3\n3\n7\n7\n9\n7\n8\n";
+    private static final String TREE = "10\n0\n1\n0\n2\n1\n5\n5\n6\n2\n4\n2\n3\n3\n7\n7\n9\n7\n8\n";
 
     /* ========================
         Tests

src/test/java/com/thealgorithms/maths/RomanNumeralUtilConstructorTest.java

@@ -4,9 +4,11 @@
 
 import java.lang.reflect.Constructor;
 import org.junit.jupiter.api.Test;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 // Covers the private constructor for code coverage tools
 public class RomanNumeralUtilConstructorTest {
+    @SuppressFBWarnings("RFI_SET_ACCESSIBLE")
     @Test
     void shouldInvokePrivateConstructor() throws Exception {
         Constructor<RomanNumeralUtil> constructor = RomanNumeralUtil.class.getDeclaredConstructor();