commit 894403557c776649a8eea2b6f39e987c5c748636
author Nandakumar Vadivelu <nanda@apache.org> Wed Dec 13 21:45:49 2023 +0530
committer GitHub <noreply@github.com> Wed Dec 13 08:15:49 2023 -0800
tree 27d2d3e5ff6575f67f80d4a9f950e3de217ee86e
parent e7eed11a941e149bdf6f4e9b5037f950998c5793

RATIS-1649. Fix SF_SWITCH_NO_DEFAULT in PureJavaCrc32C. (#991)

ratis-common/src/main/java/org/apache/ratis/util/PureJavaCrc32C.java

diff --git a/ratis-common/src/main/java/org/apache/ratis/util/PureJavaCrc32C.java b/ratis-common/src/main/java/org/apache/ratis/util/PureJavaCrc32C.java
index bdb0002..26345fb 100644
--- a/ratis-common/src/main/java/org/apache/ratis/util/PureJavaCrc32C.java
+++ b/ratis-common/src/main/java/org/apache/ratis/util/PureJavaCrc32C.java
@@ -20,8 +20,6 @@
  */
 package org.apache.ratis.util;
 
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;
 import java.util.zip.Checksum;
@@ -53,7 +51,6 @@
   }
 
   @Override
-  @SuppressFBWarnings("SF_SWITCH_NO_DEFAULT")
   public void update(byte[] b, int off, int len) {
     int localCrc = crc;
 
@@ -86,6 +83,7 @@
       case 3: localCrc = (localCrc >>> 8) ^ T[T8_0_START + ((localCrc ^ b[off++]) & 0xff)];
       case 2: localCrc = (localCrc >>> 8) ^ T[T8_0_START + ((localCrc ^ b[off++]) & 0xff)];
       case 1: localCrc = (localCrc >>> 8) ^ T[T8_0_START + ((localCrc ^ b[off++]) & 0xff)];
+      break;
       default:
         /* nothing */
     }
@@ -93,7 +91,6 @@
     crc = localCrc;
   }
 
-  @SuppressFBWarnings("SF_SWITCH_NO_DEFAULT")
   public void update(ByteBuffer b) {
     int localCrc = crc;