You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/09/12 13:47:50 UTC
(commons-jxpath) 11/27: Merge if clauses
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-jxpath.git
commit 68d19c3f871e5ad0f6f44759c2fa58c5953e0dbd
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Sep 12 09:43:58 2024 -0400
Merge if clauses
---
.../apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java b/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
index 8ae0583..fd95a23 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
@@ -108,13 +108,10 @@ public class JDOMModelTest extends XMLModelTestCase {
buffer.append("</");
buffer.append(tag);
buffer.append(">");
- }
- else if (object instanceof Text || object instanceof CDATA) {
- if (text) {
- String string = ((Text) object).getText();
- string = string.replace('\n', '=');
- buffer.append(string);
- }
+ } else if ((object instanceof Text || object instanceof CDATA) && text) {
+ String string = ((Text) object).getText();
+ string = string.replace('\n', '=');
+ buffer.append(string);
}
}