LANG-1474 documentation correction #442
Conversation
kinow
changed the title
There was a problem hiding this comment.
Testing using master branch, the docs appear to be correct for me @verhas. Could you double check if you get the same using master, please? I've also kicked Travis as the error in one of the builds is most likely due to some glitch in their env.
Thanks for the pull request.
Bruno
| @@ -3369,7 +3369,7 @@ public static boolean isAlphaSpace(final CharSequence cs) { | |||
| * <p>Whitespace is defined by {@link Character#isWhitespace(char)}.</p> | |||
| * | |||
| * <pre> | |||
| * StringUtils.isAnyBlank((String) null) = true | |||
| * StringUtils.isAnyBlank((String) null) = false | |||
There was a problem hiding this comment.
I just tried System.out.println(StringUtils.isAnyBlank((String) null)); using master branch and it printed true. Have you tried this code too?
| @@ -3402,7 +3402,7 @@ public static boolean isAnyBlank(final CharSequence... css) { | |||
| * <p>Checks if any of the CharSequences are empty ("") or null.</p> | |||
| * | |||
| * <pre> | |||
| * StringUtils.isAnyEmpty((String) null) = true | |||
| * StringUtils.isAnyEmpty((String) null) = false | |||
| @@ -3575,7 +3575,7 @@ public static boolean isMixedCase(final CharSequence cs) { | |||
| * <p>Whitespace is defined by {@link Character#isWhitespace(char)}.</p> | |||
| * | |||
| * <pre> | |||
| * StringUtils.isNoneBlank((String) null) = false | |||
| * StringUtils.isNoneBlank((String) null) = true | |||
There was a problem hiding this comment.
This one gave me false, as before.
| @@ -3600,7 +3600,7 @@ public static boolean isNoneBlank(final CharSequence... css) { | |||
| * <p>Checks if none of the CharSequences are empty ("") or null.</p> | |||
| * | |||
| * <pre> | |||
| * StringUtils.isNoneEmpty((String) null) = false | |||
| * StringUtils.isNoneEmpty((String) null) = true | |||
There was a problem hiding this comment.
Same as this one, false not true.
|
+1 |
|
@Stzx do you think the documentation changes are correct, or did you get the same as I did testing it? (sorry, couldn't tell if +1 was for my comment or for the PR change :) |
|
@kinow Sorry. |
|
Thanks for testing, it was my bad too for not understanding it. Let's wait @verhas to comment now 👍 |
|
@verhas tested again today, and same behavior in master. Closing the PR, but in case you can confirm it's indeed an issue, feel free to re-open and reply with some step to confirm/reproduce it. Thanks! |
No description provided.