minor: make private class final with default constructor (#2740)

vavr-io · Aug 19, 2024 · cc091d9 · cc091d9
1 parent a75090d
commit cc091d9
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/vavr/src/test/java/io/vavr/collection/BitSetTest.java b/vavr/src/test/java/io/vavr/collection/BitSetTest.java
@@ -569,7 +569,7 @@ public void shouldReturnTailOfNonEmptyHavingReversedOrder() {
 
     // -- classes
 
-    private static class Mapper<T> implements Serializable {
+    private static final class Mapper<T> implements Serializable {
 
         private static final long serialVersionUID = 1L;
 

diff --git a/vavr/src/test/java/io/vavr/collection/HashArrayMappedTrieTest.java b/vavr/src/test/java/io/vavr/collection/HashArrayMappedTrieTest.java
@@ -191,7 +191,7 @@ public int compareTo(WeakInteger other) {
         }
     }
 
-    private class Comparator<K, V> {
+    private final class Comparator<K, V> {
         private final java.util.Map<K, V> classic = new java.util.HashMap<>();
         private Map<K, V> hamt = HashMap.empty();
 

diff --git a/vavr/src/test/java/io/vavr/collection/euler/Euler17Test.java b/vavr/src/test/java/io/vavr/collection/euler/Euler17Test.java
@@ -121,7 +121,7 @@ default int letterCount(Seq<Integer> range) {
     /**
      * Solution using Vavr Pattern Matching.
      */
-    private static class SolutionA implements SolutionProblem17 {
+    private static final class SolutionA implements SolutionProblem17 {
         @Override
         public int letterCount(int num) {
             return Match(num).of( /*@formatter:off*/
@@ -143,7 +143,7 @@ private static int length(int number) {
     /**
      * A more general solution using functionality of the Vavr Collections.
      */
-    private static class SolutionB implements SolutionProblem17 {
+    private static final class SolutionB implements SolutionProblem17 {
         @Override
         public int letterCount(int number) {
             return asText(number).length();