Bump com.github.spotbugs:spotbugs-maven-plugin from 4.7.3.6 to 4.8.1.0 (

#127)

* Bump com.github.spotbugs:spotbugs-maven-plugin from 4.7.3.6 to 4.8.1.0

Bumps [com.github.spotbugs:spotbugs-maven-plugin](https://github.com/spotbugs/spotbugs-maven-plugin) from 4.7.3.6 to 4.8.1.0.
- [Release notes](https://github.com/spotbugs/spotbugs-maven-plugin/releases)
- [Commits](spotbugs/spotbugs-maven-plugin@spotbugs-maven-plugin-4.7.3.6...spotbugs-maven-plugin-4.8.1.0)

---
updated-dependencies:
- dependency-name: com.github.spotbugs:spotbugs-maven-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Marking final as recommended in https://wiki.sei.cmu.edu/confluence/display/java/OBJ11-J.+Be+wary+of+letting+constructors+throw+exceptions

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rishi Baldawa <baldawar@amazon.com>

pom.xml

@@ -40,7 +40,7 @@
     <gpg.plugin.version>3.1.0</gpg.plugin.version>
     <staging.plugin.version>1.6.13</staging.plugin.version>
     <checkstyle.plugin.version>3.3.1</checkstyle.plugin.version>
-    <spotbugs.plugin.version>4.7.3.6</spotbugs.plugin.version>
+    <spotbugs.plugin.version>4.8.1.0</spotbugs.plugin.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
   </properties>
 

src/main/software/amazon/event/ruler/Event.java

@@ -46,7 +46,7 @@
 // TODO: Improve unit-test coverage, there are surprising gaps
 @Immutable
 @ThreadSafe
-class Event {
+final class Event {
 
     private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper();
     private static final JsonFactory JSON_FACTORY = new JsonFactory();

src/main/software/amazon/event/ruler/IntIntMap.java

@@ -7,7 +7,7 @@
 /**
  * A fast primitive int-int map implementation. Keys and values may only be positive.
  */
-class IntIntMap implements Cloneable {
+final class IntIntMap implements Cloneable {
 
     // taken from FastUtil
     private static final int INT_PHI = 0x9E3779B9;

src/main/software/amazon/event/ruler/Range.java

@@ -8,7 +8,7 @@
  * "Numeric" means that the character repertoire is "digits"; initially, either 0-9 or 0-9a-f. In the current
  *  implementation, the number of digits in the top and bottom of the range is the same.
  */
-public class Range extends Patterns {
+public final class Range extends Patterns {
     /**
      * Bottom and top of the range. openBottom true means we're looking for > bottom, false means >=
      *  Similarly, openTop true means we're looking for < top, false means <= top.

src/main/software/amazon/event/ruler/input/MultiByte.java

@@ -9,7 +9,7 @@
  * A grouping of multiple bytes. This can be used to represent a character that has a UTF-8 representation requiring
  * multiple bytes.
  */
-public class MultiByte {
+public final class MultiByte {
 
     public static final byte MIN_FIRST_BYTE_FOR_ONE_BYTE_CHAR = (byte) 0x00;
     public static final byte MAX_FIRST_BYTE_FOR_ONE_BYTE_CHAR = (byte) 0x7F;