Github user chtompki commented on a diff in the pull request:
https://github.com/apache/commons-text/pull/63#discussion_r140545049
--- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
@@ -614,10 +614,7 @@ 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)) {
- newCodePoint = Character.toLowerCase(oldCodepoint);
- whitespace = false;
- } else if (Character.isTitleCase(oldCodepoint)) {
+ if (Character.isUpperCase(oldCodepoint) ||
Character.isTitleCase(oldCodepoint) ) {
--- End diff --
Checkstyle want's to see this as
```java
if (Character.isUpperCase(oldCodepoint) ||
Character.isTitleCase(oldCodepoint)) {
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
