This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git

commit 57aba9817498082fc7b56d146e0733beaf6deb22
Author: Gary Gregory <gardgreg...@gmail.com>
AuthorDate: Mon Feb 22 15:01:12 2021 -0500

    Combine nested 'if' statement in 'else' block to 'else if'.
---
 src/main/java/org/apache/commons/io/FilenameUtils.java         |  8 +++-----
 .../java/org/apache/commons/io/output/XmlStreamWriter.java     | 10 ++++------
 2 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/src/main/java/org/apache/commons/io/FilenameUtils.java 
b/src/main/java/org/apache/commons/io/FilenameUtils.java
index c14ad08..3a99731 100644
--- a/src/main/java/org/apache/commons/io/FilenameUtils.java
+++ b/src/main/java/org/apache/commons/io/FilenameUtils.java
@@ -1427,12 +1427,10 @@ public class FilenameUtils {
                         if (repeat >= 0) {
                             backtrack.push(new int[] {wcsIdx, repeat});
                         }
-                    } else {
+                    } else if (!caseSensitivity.checkRegionMatches(fileName, 
textIdx, wcs[wcsIdx])) {
                         // matching from current position
-                        if (!caseSensitivity.checkRegionMatches(fileName, 
textIdx, wcs[wcsIdx])) {
-                            // couldn't match token
-                            break;
-                        }
+                        // couldn't match token
+                        break;
                     }
 
                     // matched text token, move text index to end of matched 
token
diff --git a/src/main/java/org/apache/commons/io/output/XmlStreamWriter.java 
b/src/main/java/org/apache/commons/io/output/XmlStreamWriter.java
index 59849f7..881f1d7 100644
--- a/src/main/java/org/apache/commons/io/output/XmlStreamWriter.java
+++ b/src/main/java/org/apache/commons/io/output/XmlStreamWriter.java
@@ -178,12 +178,10 @@ public class XmlStreamWriter extends Writer {
                         // encoding
                         encoding = defaultEncoding;
                     }
-                } else {
-                    if (xmlProlog.length() >= BUFFER_SIZE) {
-                        // no encoding found in first characters: using default
-                        // encoding
-                        encoding = defaultEncoding;
-                    }
+                } else if (xmlProlog.length() >= BUFFER_SIZE) {
+                    // no encoding found in first characters: using default
+                    // encoding
+                    encoding = defaultEncoding;
                 }
             } else {
                 // no XML prolog: using default encoding

Reply via email to