You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pa...@apache.org on 2008/10/31 20:58:42 UTC

svn commit: r709548 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java

Author: pauls
Date: Fri Oct 31 12:58:42 2008
New Revision: 709548

URL: http://svn.apache.org/viewvc?rev=709548&view=rev
Log:
Make some members in the ldap operators final to hopefully fix toString() visibility issues. (FELIX-765)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java?rev=709548&r1=709547&r2=709548&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/util/ldap/Parser.java Fri Oct 31 12:58:42 2008
@@ -541,7 +541,7 @@
     // Exclusive inner classes
     private static final class AndOperator extends Operator
     {
-        private int operandCount;
+        private final int operandCount;
 
         public AndOperator(int opcnt)
         {
@@ -610,7 +610,7 @@
 
     private static final class OrOperator extends Operator
     {
-        private int operandCount;
+        private final int operandCount;
 
         public OrOperator(int opcnt)
         {
@@ -1016,7 +1016,7 @@
 
     private static final class PresentOperator extends Operator
     {
-        String attribute;
+        final String attribute;
 
         public PresentOperator(String attribute)
         {
@@ -1050,7 +1050,7 @@
 
     private static final class PushOperator extends Operator
     {
-        String attribute;
+        final String attribute;
 
         public PushOperator(String attribute)
         {
@@ -1093,7 +1093,7 @@
 
     private static final class ConstOperator extends Operator
     {
-        Object val;
+        final Object val;
 
         public ConstOperator(Object val)
         {
@@ -1130,7 +1130,7 @@
     private static final class SubStringOperator extends Operator
         implements OperatorConstants
     {
-        String[] pieces;
+        final String[] pieces;
 
         public SubStringOperator(String[] pieces)
         {