[gwt-contrib] [google-web-toolkit] r10263 committed - Removed an extraneous copy of the columns variable in CellTable. Cell...

[codesite-noreply]

Revision: 10263
Author:   ms...@google.com
Date:     Wed Jun  1 11:16:17 2011
Log:      Removed an extraneous copy of the columns variable in CellTable.   
CellTable now uses getColumnCount to set the colspan of the empty-table  
widget.

Review by: jlaba...@google.com
http://code.google.com/p/google-web-toolkit/source/detail?r=10263

Modified:
 /trunk/user/src/com/google/gwt/user/cellview/client/CellTable.java

=======================================
--- /trunk/user/src/com/google/gwt/user/cellview/client/CellTable.java	Thu  
May 26 04:44:13 2011
+++ /trunk/user/src/com/google/gwt/user/cellview/client/CellTable.java	Wed  
Jun  1 11:16:17 2011
@@ -37,9 +37,7 @@
 import com.google.gwt.user.client.ui.Widget;
 import com.google.gwt.view.client.ProvidesKey;

-import java.util.ArrayList;
 import java.util.HashSet;
-import java.util.List;
 import java.util.Set;

 /**
@@ -485,7 +483,6 @@
   }

   final TableColElement colgroup;
-  private final List<Column<T, ?>> columns = new ArrayList<Column<T, ?>>();
   private final SimplePanel emptyTableWidgetContainer = new SimplePanel();
   private final SimplePanel loadingIndicatorContainer = new SimplePanel();

@@ -805,7 +802,7 @@
     }

     // Adjust the colspan of the messages panel container.
-    tbodyLoadingCell.setColSpan(Math.max(1, columns.size()));
+    tbodyLoadingCell.setColSpan(Math.max(1, getColumnCount()));

     // Show the correct container.
     showOrHide(getChildContainer(), message == null);

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors