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 2022/05/24 12:26:16 UTC

[commons-configuration] branch master updated: Makse some private classes static.

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-configuration.git


The following commit(s) were added to refs/heads/master by this push:
     new a62c05b8 Makse some private classes static.
a62c05b8 is described below

commit a62c05b8094a7aff8c9ba38f4a26870926e0facb
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Tue May 24 08:26:09 2022 -0400

    Makse some private classes static.
---
 .../commons/configuration2/tree/ModelTransaction.java       | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java b/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
index e2e7f9f2..d62bd9c0 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
@@ -30,7 +30,7 @@ import java.util.TreeMap;
 
 /**
  * <p>
- * An internally used helper class for an atomic update of an {@link InMemoryNodeModel}.
+ * An internal helper class for a atomic updates of an {@link InMemoryNodeModel}.
  * </p>
  * <p>
  * This class performs updates on the node structure of a node model consisting of {@link ImmutableNode} objects.
@@ -56,6 +56,7 @@ import java.util.TreeMap;
  *
  */
 class ModelTransaction {
+
     /**
      * Constant for the maximum number of entries in the replacement mapping. If this number is exceeded, the parent mapping
      * is reconstructed. The number is a bit arbitrary. If it is too low, updates - especially on large node structures -
@@ -688,7 +689,7 @@ class ModelTransaction {
     /**
      * A specialized operation class for adding an attribute to a target node.
      */
-    private class AddAttributeOperation extends Operation {
+    private static class AddAttributeOperation extends Operation {
         /** The attribute name. */
         private final String attributeName;
 
@@ -715,7 +716,7 @@ class ModelTransaction {
     /**
      * A specialized operation class for adding multiple attributes to a target node.
      */
-    private class AddAttributesOperation extends Operation {
+    private static class AddAttributesOperation extends Operation {
         /** The map with attributes. */
         private final Map<String, Object> attributes;
 
@@ -737,7 +738,7 @@ class ModelTransaction {
     /**
      * A specialized operation class for removing an attribute from a target node.
      */
-    private class RemoveAttributeOperation extends Operation {
+    private static class RemoveAttributeOperation extends Operation {
         /** The attribute name. */
         private final String attributeName;
 
@@ -759,7 +760,7 @@ class ModelTransaction {
     /**
      * A specialized operation class which changes the value of a node.
      */
-    private class ChangeNodeValueOperation extends Operation {
+    private static class ChangeNodeValueOperation extends Operation {
         /** The new value for the affected node. */
         private final Object newValue;
 
@@ -781,7 +782,7 @@ class ModelTransaction {
     /**
      * A specialized operation class which changes the name of a node.
      */
-    private class ChangeNodeNameOperation extends Operation {
+    private static class ChangeNodeNameOperation extends Operation {
         /** The new node name. */
         private final String newName;