Simplify

FasterXML · Aug 26, 2023 · 78b17fb · 78b17fb
1 parent d48f9f6
commit 78b17fb
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 17 deletions.
diff --git a/src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializerFactory.java b/src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializerFactory.java
@@ -802,15 +802,9 @@ protected void removeIgnorableTypes(SerializationConfig config, BeanDescription
     protected void removeSetterlessGetters(SerializationConfig config, BeanDescription beanDesc,
             List<BeanPropertyDefinition> properties)
     {
-        Iterator<BeanPropertyDefinition> it = properties.iterator();
-        while (it.hasNext()) {
-            BeanPropertyDefinition property = it.next();
-            // one caveat: only remove implicit properties;
-            // explicitly annotated ones should remain
-            if (!property.couldDeserialize() && !property.isExplicitlyIncluded()) {
-                it.remove();
-            }
-        }
+        // one caveat: only remove implicit properties;
+        // explicitly annotated ones should remain
+        properties.removeIf(property -> !property.couldDeserialize() && !property.isExplicitlyIncluded());
     }
 
     /**

diff --git a/src/main/java/com/fasterxml/jackson/databind/ser/std/IterableSerializer.java b/src/main/java/com/fasterxml/jackson/databind/ser/std/IterableSerializer.java
@@ -50,9 +50,7 @@ public boolean hasSingleElement(Iterable<?> value) {
             Iterator<?> it = value.iterator();
             if (it.hasNext()) {
                 it.next();
-                if (!it.hasNext()) {
-                    return true;
-                }
+                return !it.hasNext();
             }
         }
         return false;

diff --git a/src/main/java/com/fasterxml/jackson/databind/ser/std/UUIDSerializer.java b/src/main/java/com/fasterxml/jackson/databind/ser/std/UUIDSerializer.java
@@ -51,11 +51,8 @@ protected UUIDSerializer(Boolean asBinary) {
     public boolean isEmpty(SerializerProvider prov, UUID value)
     {
         // Null UUID is empty, so...
-        if (value.getLeastSignificantBits() == 0L
-                && value.getMostSignificantBits() == 0L) {
-            return true;
-        }
-        return false;
+        return value.getLeastSignificantBits() == 0L
+                && value.getMostSignificantBits() == 0L;
     }
 
     @Override