Remove unnecessary semicolons (#4535)

FasterXML · May 17, 2024 · f2045ca · f2045ca
1 parent ac7b179
commit f2045ca
Show file tree

Hide file tree

Showing 20 changed files with 15 additions and 33 deletions.
diff --git a/src/main/java/com/fasterxml/jackson/databind/AnnotationIntrospector.java b/src/main/java/com/fasterxml/jackson/databind/AnnotationIntrospector.java
@@ -68,7 +68,6 @@ public enum Type {
              * {@link com.fasterxml.jackson.annotation.JsonBackReference}
              */
             BACK_REFERENCE
-            ;
         }
 
         private final Type _type;
@@ -1111,7 +1110,7 @@ public String[] findEnumValues(MapperConfig<?> config, AnnotatedClass annotatedC
      */
     @Deprecated
     public void findEnumAliases(Class<?> enumType, Enum<?>[] enumValues, String[][] aliases) {
-        ;
+        ; // do nothing
     }
 
     /**

diff --git a/src/main/java/com/fasterxml/jackson/databind/JsonNode.java b/src/main/java/com/fasterxml/jackson/databind/JsonNode.java
@@ -79,7 +79,7 @@ public enum OverwriteMode {
          * Mode in which all incompatible node types may be replaced, including
          * Array and Object nodes where necessary.
          */
-        ALL;
+        ALL
     }
 
     /*

diff --git a/src/main/java/com/fasterxml/jackson/databind/ObjectWriter.java b/src/main/java/com/fasterxml/jackson/databind/ObjectWriter.java
@@ -1543,7 +1543,6 @@ public Prefetch forRootType(ObjectWriter parent, JavaType newType) {
                     return new Prefetch(newType, ser, null);
                 } catch (DatabindException e) {
                     // need to swallow?
-                    ;
                 }
             }
             return new Prefetch(newType, null, typeSerializer);

diff --git a/src/main/java/com/fasterxml/jackson/databind/annotation/JsonSerialize.java b/src/main/java/com/fasterxml/jackson/databind/annotation/JsonSerialize.java
@@ -240,7 +240,6 @@ public enum Inclusion
          * @since 2.3
          */
         DEFAULT_INCLUSION
-        ;
     }
 
     /**
@@ -269,6 +268,5 @@ public enum Typing
          * @since 2.3
          */
         DEFAULT_TYPING
-        ;
     }
 }
diff --git a/src/main/java/com/fasterxml/jackson/databind/cfg/CoercionAction.java b/src/main/java/com/fasterxml/jackson/databind/cfg/CoercionAction.java
@@ -37,6 +37,5 @@ public enum CoercionAction
      * empty collection; for POJOs instance configured with default constructor
      * and so on.
      */
-    AsEmpty;
-    ;
+    AsEmpty
 }
diff --git a/src/main/java/com/fasterxml/jackson/databind/cfg/CoercionInputShape.java b/src/main/java/com/fasterxml/jackson/databind/cfg/CoercionInputShape.java
@@ -92,5 +92,4 @@ public enum CoercionInputShape
      */
     EmptyString
 
-    ;
 }
diff --git a/src/main/java/com/fasterxml/jackson/databind/cfg/ConstructorDetector.java b/src/main/java/com/fasterxml/jackson/databind/cfg/ConstructorDetector.java
@@ -55,7 +55,7 @@ public enum SingleArgConstructor {
          * {@link com.fasterxml.jackson.databind.exc.InvalidDefinitionException}
          * in ambiguous case.
          */
-        REQUIRE_MODE;
+        REQUIRE_MODE
     }
 
     /*

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/BasicDeserializerFactory.java b/src/main/java/com/fasterxml/jackson/databind/deser/BasicDeserializerFactory.java
@@ -282,7 +282,6 @@ protected ValueInstantiator _constructDefaultValueInstantiator(DeserializationCo
             final boolean isNonStaticInnerClass = beanDesc.isNonStaticInnerClass();
             if (isNonStaticInnerClass) {
                 // TODO: look for `@JsonCreator` annotated ones, throw explicit exception?
-                ;
             } else {
                 // 18-Sep-2020, tatu: Although by default implicit introspection is allowed, 2.12
                 //   has settings to prevent that either generally, or at least for JDK types
@@ -964,7 +963,6 @@ protected void _addExplicitAnyCreator(DeserializationContext ctxt,
                 if (!useProps) {
                     // Otherwise, `@JsonValue` suggests delegation
                     if (beanDesc.findJsonValueAccessor() != null) {
-                        ;
                     } else if (injectId != null) {
                         // But Injection suggests property-based (for legacy reasons?)
                         useProps = true;

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializer.java b/src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializer.java
@@ -1044,9 +1044,7 @@ protected Object deserializeUsingPropertyBasedWithExternalTypeId(JsonParser p,
                 // first: let's check to see if this might be part of value with external type id:
                 // 11-Sep-2015, tatu: Important; do NOT pass buffer as last arg, but null,
                 //   since it is not the bean
-                if (ext.handlePropertyValue(p, ctxt, propName, null)) {
-                    ;
-                } else {
+                if (!ext.handlePropertyValue(p, ctxt, propName, null)) {
                     // Last creator property to set?
                     if (buffer.assignParameter(creatorProp,
                             _deserializeWithErrorWrapping(p, ctxt, creatorProp))) {

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializerFactory.java b/src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializerFactory.java
@@ -594,7 +594,7 @@ protected void addBeanProps(DeserializationContext ctxt,
                         // 23-Jan-2018, tatu: As per [databind#1805], need to ensure we don't
                         //   accidentally sneak in getter-as-setter for `READ_ONLY` properties
                         if (builder.hasIgnorable(propDef.getName())) {
-                            ;
+                            ; // skip
                         } else {
                             prop = constructSetterlessProperty(ctxt, beanDesc, propDef);
                         }

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/SettableBeanProperty.java b/src/main/java/com/fasterxml/jackson/databind/deser/SettableBeanProperty.java
@@ -330,14 +330,12 @@ public void assignIndex(int index) {
      *
      * @since 2.8.3
      */
-    public void fixAccess(DeserializationConfig config) {
-        ;
-    }
+    public void fixAccess(DeserializationConfig config) { /* nop */ }
 
     /**
      * @since 2.9.4
      */
-    public void markAsIgnorable() { }
+    public void markAsIgnorable() { /* nop */ }
 
     /**
      * @since 2.9.4

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/impl/CreatorCollector.java b/src/main/java/com/fasterxml/jackson/databind/deser/impl/CreatorCollector.java
@@ -320,9 +320,7 @@ protected boolean verifyNonDup(AnnotatedWithParams newOne, int typeIndex, boolea
                     if (_isEnumValueOf(newOne)) {
                         return false; // ignore
                     }
-                    if (_isEnumValueOf(oldOne)) {
-                        ;
-                    } else {
+                    if (!_isEnumValueOf(oldOne)) {
                         _reportDuplicateCreator(typeIndex, explicit, oldOne, newOne);
                     }
                 }
@@ -332,7 +330,6 @@ else if (newType.isAssignableFrom(oldType)) {
                     return false;
                 } else if (oldType.isAssignableFrom(newType)) {
                     // new type more specific, use it
-                    ;
                     // 23-Feb-2021, tatu: due to [databind#3062], backwards-compatibility,
                     //   let's allow "primitive/Wrapper" case and tie-break in favor
                     //   of PRIMITIVE argument (null would never map to scalar creators,

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/std/EnumDeserializer.java b/src/main/java/com/fasterxml/jackson/databind/deser/std/EnumDeserializer.java
@@ -419,7 +419,7 @@ private final Object _deserializeAltString(JsonParser p, DeserializationContext
                     // index yet (might need combination of "Does format have Numbers"
                     // (XML does not f.ex) and new `EnumFeature`. But can disallow "001" etc.
                     if (c == '0' && name.length() > 1) {
-                        ;
+                        ; // skip
                     } else {
                         try {
                             int index = Integer.parseInt(name);

diff --git a/src/main/java/com/fasterxml/jackson/databind/deser/std/JsonNodeDeserializer.java b/src/main/java/com/fasterxml/jackson/databind/deser/std/JsonNodeDeserializer.java
@@ -823,7 +823,6 @@ protected final JsonNode _fromBigDecimal(DeserializationContext ctxt,
                 bigDec = bigDec.stripTrailingZeros();
             } catch (ArithmeticException e) {
                 // If we can't, we can't...
-                ;
             }
         }
         return nodeFactory.numberNode(bigDec);

diff --git a/src/main/java/com/fasterxml/jackson/databind/jsontype/PolymorphicTypeValidator.java b/src/main/java/com/fasterxml/jackson/databind/jsontype/PolymorphicTypeValidator.java
@@ -74,7 +74,6 @@ public enum Validity {
          * determination will be {@code DENIED}, for safety reasons.
          */
         INDETERMINATE
-        ;
     }
 
     /**

diff --git a/src/main/java/com/fasterxml/jackson/databind/jsontype/impl/SubTypeValidator.java b/src/main/java/com/fasterxml/jackson/databind/jsontype/impl/SubTypeValidator.java
@@ -274,7 +274,7 @@ public void validateSubType(DeserializationContext ctxt, JavaType type,
             //    for some Spring framework types
             // 05-Jan-2017, tatu: ... also, only applies to classes, not interfaces
             if (raw.isInterface()) {
-                ;
+                ; // skip
             } else if (full.startsWith(PREFIX_SPRING)) {
                 for (Class<?> cls = raw; (cls != null) && (cls != Object.class); cls = cls.getSuperclass()){
                     String name = cls.getSimpleName();

diff --git a/src/main/java/com/fasterxml/jackson/databind/ser/PropertyBuilder.java b/src/main/java/com/fasterxml/jackson/databind/ser/PropertyBuilder.java
@@ -284,7 +284,7 @@ protected JavaType findSerializationType(Annotated a, boolean useStaticTyping, J
             // Must be a super type to be usable
             Class<?> rawDeclared = declaredType.getRawClass();
             if (serClass.isAssignableFrom(rawDeclared)) {
-                ; // fine as is
+                // fine as is
             } else {
                 /* 18-Nov-2010, tatu: Related to fixing [JACKSON-416], an issue with such
                  *   check is that for deserialization more specific type makes sense;

diff --git a/src/main/java/com/fasterxml/jackson/databind/util/AccessPattern.java b/src/main/java/com/fasterxml/jackson/databind/util/AccessPattern.java
@@ -27,5 +27,4 @@ public enum AccessPattern {
      * needed.
      */
     DYNAMIC
-    ;
 }
diff --git a/src/main/java/com/fasterxml/jackson/databind/util/ClassUtil.java b/src/main/java/com/fasterxml/jackson/databind/util/ClassUtil.java
@@ -583,7 +583,7 @@ public static <T> Constructor<T> findConstructor(Class<T> cls, boolean forceAcce
             }
             return ctor;
         } catch (NoSuchMethodException e) {
-            ;
+            return null;
         } catch (Exception e) {
             ClassUtil.unwrapAndThrowAsIAE(e, "Failed to find default constructor of class "+cls.getName()+", problem: "+e.getMessage());
         }

diff --git a/src/main/java/com/fasterxml/jackson/databind/util/ISO8601DateFormat.java b/src/main/java/com/fasterxml/jackson/databind/util/ISO8601DateFormat.java
@@ -20,8 +20,8 @@ public class ISO8601DateFormat extends DateFormat
     private static final long serialVersionUID = 1L;
 
     public ISO8601DateFormat() {
-        this.numberFormat = new DecimalFormat();;
-        this.calendar = new GregorianCalendar();;
+        this.numberFormat = new DecimalFormat();
+        this.calendar = new GregorianCalendar();
     }
 
     @Override