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,

Reply via email to