[jboss-svn-commits] JBL Code SVN: r7157 - labs/jbossrules/trunk/drools-core/src/main/java/org/drools/util

[jboss-svn-commits at lists.jboss.org]

Author: mark.proctor at jboss.com
Date: 2006-10-26 21:03:45 -0400 (Thu, 26 Oct 2006)
New Revision: 7157

Modified:
   labs/jbossrules/trunk/drools-core/src/main/java/org/drools/util/LinkedList.java
Log:
-made the remove() method more efficient.

Modified: labs/jbossrules/trunk/drools-core/src/main/java/org/drools/util/LinkedList.java
===================================================================
--- labs/jbossrules/trunk/drools-core/src/main/java/org/drools/util/LinkedList.java	2006-10-27 00:19:26 UTC (rev 7156)
+++ labs/jbossrules/trunk/drools-core/src/main/java/org/drools/util/LinkedList.java	2006-10-27 01:03:45 UTC (rev 7157)
@@ -89,19 +89,16 @@
      *      The <code>LinkedListNode</code> to be removed.
      */
     public void remove(final LinkedListNode node) {
-        if ( (this.firstNode != node) && (this.lastNode != node) ) {
-            node.getPrevious().setNext( node.getNext() );
-            ((LinkedListNode)node.getNext()).setPrevious( node.getPrevious() );
-            this.size--;
-            node.setPrevious( null );
-            node.setNext( null );
-
+        if ( this.firstNode == node ) {
+            removeFirst();
+        } else if ( this.lastNode == node ) {
+            removeLast();
         } else {
-            if ( this.firstNode == node ) {
-                removeFirst();
-            } else if ( this.lastNode == node ) {
-                removeLast();
-            }
+           node.getPrevious().setNext( node.getNext() );
+           ((LinkedListNode)node.getNext()).setPrevious( node.getPrevious() );
+           this.size--;
+           node.setPrevious( null );
+           node.setNext( null );
         }
     }