Github user TheRealHaui commented on a diff in the pull request:

    https://github.com/apache/commons-text/pull/63#discussion_r140526276
  
    --- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
    @@ -614,13 +615,11 @@ public static String swapCase(final String str) {
             for (int index = 0; index < strLen;) {
                 final int oldCodepoint = str.codePointAt(index);
                 final int newCodePoint;
    -            if (Character.isUpperCase(oldCodepoint)) {
    +            if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint) ) {
                     newCodePoint = Character.toLowerCase(oldCodepoint);
                     whitespace = false;
    -            } else if (Character.isTitleCase(oldCodepoint)) {
    -                newCodePoint = Character.toLowerCase(oldCodepoint);
    -                whitespace = false;
    -            } else if (Character.isLowerCase(oldCodepoint)) {
    +            }
    +            else if (Character.isLowerCase(oldCodepoint)) {
    --- End diff --
    
    No, you're right.
    My mistake.
    
    Corrected and commited the correction.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to