Update of /var/cvs/src/org/mmbase/storage/search/implementation
In directory james.mmbase.org:/tmp/cvs-serv17704

Modified Files:
        BasicStepField.java 
Log Message:
equals method is called very very often. Tried to make it as efficient as 
possible


See also: 
http://cvs.mmbase.org/viewcvs/src/org/mmbase/storage/search/implementation


Index: BasicStepField.java
===================================================================
RCS file: 
/var/cvs/src/org/mmbase/storage/search/implementation/BasicStepField.java,v
retrieving revision 1.27
retrieving revision 1.28
diff -u -b -r1.27 -r1.28
--- BasicStepField.java 8 Aug 2007 09:38:28 -0000       1.27
+++ BasicStepField.java 19 Aug 2008 17:47:47 -0000      1.28
@@ -18,7 +18,7 @@
  * The field alias is not set on default.
  *
  * @author Rob van Maris
- * @version $Id: BasicStepField.java,v 1.27 2007/08/08 09:38:28 michiel Exp $
+ * @version $Id: BasicStepField.java,v 1.28 2008/08/19 17:47:47 michiel Exp $
  * @since MMBase-1.7
  */
 public class BasicStepField implements StepField {
@@ -168,7 +168,7 @@
     }
 
     // javadoc is inherited
-    public String getFieldName() {
+    public final String getFieldName() {
         return field.getName();
     }
 
@@ -190,10 +190,10 @@
     // javadoc is inherited
     public boolean equals(Object obj) {
         if (obj instanceof StepField) {
-            StepField field = (StepField) obj;
-            return BasicStepField.compareSteps(getStep(), field.getStep())
-                && getFieldName().equals(field.getFieldName())
-                && (alias == null? field.getAlias() == null : 
alias.equals(field.getAlias()));
+            StepField f = (StepField) obj;
+            return field.getName().equals(f.getFieldName())
+                && compareSteps(step, f.getStep())
+                && (alias == null? f.getAlias() == null : 
alias.equals(f.getAlias()));
         } else {
             return false;
         }
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to