commit 67ab9cb33df455bc6559cf40693cb6928ace2dc1
author Dan Klco <dklco@apache.org> Mon Jan 11 23:02:34 2021 -0500
committer Dan Klco <dklco@apache.org> Mon Jan 11 23:02:34 2021 -0500
tree d697ee36dce17136aaa7daa5a4f5c375c0ac3cfb
parent 052371e0b505d82d90e80a2180cc36725406bae1

Fixing some code smells

src/main/java/org/apache/sling/rewriter/impl/components/Html5Serializer.java

diff --git a/src/main/java/org/apache/sling/rewriter/impl/components/Html5Serializer.java b/src/main/java/org/apache/sling/rewriter/impl/components/Html5Serializer.java
index b9a81e4..ad14116 100644
--- a/src/main/java/org/apache/sling/rewriter/impl/components/Html5Serializer.java
+++ b/src/main/java/org/apache/sling/rewriter/impl/components/Html5Serializer.java
@@ -176,10 +176,7 @@
         if ("br".equals(localName) && ("clear".equals(atts.getLocalName(i)))) {
             return true;
         }
-        if (atts.getValue(i) == null) {
-            return true;
-        }
-        return false;
+        return atts.getValue(i) == null;
     }
 
     @Override
@@ -190,7 +187,7 @@
     /**
      * Encode and write a specific part of an array of characters.
      */
-    private void writeEncoded(char data[], int start, int length) throws SAXException {
+    private void writeEncoded(char[] data, int start, int length) throws SAXException {
         int end = start + length;
 
         if (data == null) {