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-text.git
commit 045159b0fa05d5e0e08bdf2b2daf5fa13c815ca0 Author: Gary Gregory <[email protected]> AuthorDate: Sat Jul 18 17:14:00 2020 -0400 Use empty tests instead of size tests. --- src/main/java/org/apache/commons/text/TextStringBuilder.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/commons/text/TextStringBuilder.java b/src/main/java/org/apache/commons/text/TextStringBuilder.java index 10e6230..c9b1eec 100644 --- a/src/main/java/org/apache/commons/text/TextStringBuilder.java +++ b/src/main/java/org/apache/commons/text/TextStringBuilder.java @@ -1187,7 +1187,7 @@ public class TextStringBuilder implements CharSequence, Appendable, Serializable * @return this, to enable chaining */ public TextStringBuilder appendSeparator(final char separator) { - if (size() > 0) { + if (isNotEmpty()) { append(separator); } return this; @@ -1204,10 +1204,10 @@ public class TextStringBuilder implements CharSequence, Appendable, Serializable * @return this, to enable chaining */ public TextStringBuilder appendSeparator(final char standard, final char defaultIfEmpty) { - if (size() > 0) { - append(standard); - } else { + if (isEmpty()) { append(defaultIfEmpty); + } else { + append(standard); } return this; }
