commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From TheRealHaui <...@git.apache.org>
Subject [GitHub] commons-text pull request #63: small-code-quality-improvements
Date Fri, 22 Sep 2017 15:35:03 GMT
Github user TheRealHaui commented on a diff in the pull request:

    https://github.com/apache/commons-text/pull/63#discussion_r140525860
  
    --- 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)
) {
    --- End diff --
    
    The corresponding lines as the whole method are already completely covered by existing
Unit Tests.
    ![selection_050](https://user-images.githubusercontent.com/6312834/30752358-5a0d0094-9fbc-11e7-9ada-822cb344cefb.png)



---

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


Mime
View raw message