Author: ruschein
Date: 2011-08-24 14:43:13 -0700 (Wed, 24 Aug 2011)
New Revision: 26631

Modified:
   
core3/impl/trunk/table-browser-impl/src/main/java/org/cytoscape/browser/internal/BrowserTableModel.java
Log:
Performance optimisation.

Modified: 
core3/impl/trunk/table-browser-impl/src/main/java/org/cytoscape/browser/internal/BrowserTableModel.java
===================================================================
--- 
core3/impl/trunk/table-browser-impl/src/main/java/org/cytoscape/browser/internal/BrowserTableModel.java
     2011-08-24 17:18:52 UTC (rev 26630)
+++ 
core3/impl/trunk/table-browser-impl/src/main/java/org/cytoscape/browser/internal/BrowserTableModel.java
     2011-08-24 21:43:13 UTC (rev 26631)
@@ -129,16 +129,7 @@
                if (!tableHasBooleanSelected)
                        return dataTable.getRowCount();
 
-               final List<CyRow> rows = dataTable.getAllRows();
-
-               int selectedCount = 0;
-               for (final CyRow row : rows) {
-                       final Boolean selected = row.get(CyNetwork.SELECTED, 
Boolean.class);
-                       if (selected != null && selected)
-                               ++selectedCount;
-               }
-
-               return selectedCount;
+               return dataTable.getMatchingRows(CyNetwork.SELECTED, 
Boolean.valueOf(true)).size();
        }
 
        @Override

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to