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
The following commit(s) were added to refs/heads/master by this push: new d1ddd69 Remove noisy inline comments d1ddd69 is described below commit d1ddd692cd8c242e9f37ba5e1a6c5920dd90eed4 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Wed Jan 31 10:23:59 2024 -0500 Remove noisy inline comments --- src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java | 1 - src/main/java/org/apache/commons/jxpath/util/ClassLoaderUtil.java | 1 - src/main/java/org/apache/commons/jxpath/util/ValueUtils.java | 1 - src/test/java/org/apache/commons/jxpath/TestBean.java | 2 -- .../java/org/apache/commons/jxpath/ri/model/MixedModelTest.java | 6 ------ .../java/org/apache/commons/jxpath/ri/model/XMLModelTestCase.java | 2 -- 6 files changed, 13 deletions(-) diff --git a/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java b/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java index bf1961f..253be98 100644 --- a/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java +++ b/src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java @@ -124,7 +124,6 @@ public abstract class JXPathContextFactory { JXPathContext parentContext, Object contextBean); - // -------------------- private methods -------------------- // This code is duplicated in all factories. // Keep it in sync or move it to a common place // Because it's small probably it's easier to keep it here diff --git a/src/main/java/org/apache/commons/jxpath/util/ClassLoaderUtil.java b/src/main/java/org/apache/commons/jxpath/util/ClassLoaderUtil.java index eb6eabc..0bacef6 100644 --- a/src/main/java/org/apache/commons/jxpath/util/ClassLoaderUtil.java +++ b/src/main/java/org/apache/commons/jxpath/util/ClassLoaderUtil.java @@ -60,7 +60,6 @@ public class ClassLoaderUtil { } // Class loading - // ---------------------------------------------------------------------- /** * Returns the class represented by <code>className</code> using the * <code>classLoader</code>. This implementation supports names like diff --git a/src/main/java/org/apache/commons/jxpath/util/ValueUtils.java b/src/main/java/org/apache/commons/jxpath/util/ValueUtils.java index 2e02c31..a963dd6 100644 --- a/src/main/java/org/apache/commons/jxpath/util/ValueUtils.java +++ b/src/main/java/org/apache/commons/jxpath/util/ValueUtils.java @@ -532,7 +532,6 @@ public class ValueUtils { }); } - // -------------------------------------------------------- Private Methods // // The rest of the code in this file was copied FROM // org.apache.commons.beanutils.PropertyUtil. We don't want to introduce diff --git a/src/test/java/org/apache/commons/jxpath/TestBean.java b/src/test/java/org/apache/commons/jxpath/TestBean.java index 25aa0e7..421b8df 100644 --- a/src/test/java/org/apache/commons/jxpath/TestBean.java +++ b/src/test/java/org/apache/commons/jxpath/TestBean.java @@ -30,8 +30,6 @@ import org.apache.commons.jxpath.util.ValueUtils; */ public class TestBean { - // ------------------------------------------------------------- Properties - /** * An array of nested java beans. */ diff --git a/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java b/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java index e4b23d5..de007c3 100644 --- a/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java +++ b/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java @@ -119,8 +119,6 @@ public class MixedModelTest extends JXPathTestCase { "$container/vendor[1]/location[2]/address[1]/city[1]"); } - // ---------------------------------------------------------------------- - public void testBeanPrimitive() { assertXPathValueAndPointer(context, "string", "string", "/string"); } @@ -169,8 +167,6 @@ public class MixedModelTest extends JXPathTestCase { "/container/vendor[1]/location[2]/address[1]/city[1]"); } - // ---------------------------------------------------------------------- - public void testMapPrimitive() { assertXPathValueAndPointer( context, @@ -229,8 +225,6 @@ public class MixedModelTest extends JXPathTestCase { + "/vendor[1]/location[2]/address[1]/city[1]"); } - // ---------------------------------------------------------------------- - public void testListPrimitive() { assertXPathValueAndPointer(context, "list[1]", "string", "/list[1]"); } diff --git a/src/test/java/org/apache/commons/jxpath/ri/model/XMLModelTestCase.java b/src/test/java/org/apache/commons/jxpath/ri/model/XMLModelTestCase.java index 4939cf9..18798f2 100644 --- a/src/test/java/org/apache/commons/jxpath/ri/model/XMLModelTestCase.java +++ b/src/test/java/org/apache/commons/jxpath/ri/model/XMLModelTestCase.java @@ -90,8 +90,6 @@ public abstract class XMLModelTestCase extends JXPathTestCase { assertEquals("XML Signature mismatch: ", signature, sig); } - // ------------------------------------------------ Individual Test Methods - public void testDocumentOrder() { assertDocumentOrder( context,