Scarab commit: svn commit: r10717 - trunk/src/java/org/tigris/scarab/om/RModuleOption.java

Hussayn Dabbous <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: dabbous
Date: 2009-05-08 08:03:44-0700
New Revision: 10717

Modified:
   trunk/src/java/org/tigris/scarab/om/RModuleOption.java

Log:
SCB2620: small optimisation: removed a condition which was allways true.

Modified: trunk/src/java/org/tigris/scarab/om/RModuleOption.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/RModuleOption.java?view=diff&pathrev=10717&r1=10716&r2=10717
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/RModuleOption.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/RModuleOption.java	2009-05-08 08:03:44-0700
@@ -278,7 +278,7 @@
                 }
                
                 // Need to perform the correction only if the deleted option had follow up options
-                if(optIds != null && optIds.size() > 0)
+                if(optIds.size() > 0)
                 {
                     // update the list
                     final Criteria c2 = new Criteria()

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2115431
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.