Reviewers: jgw,
Description:
In incubator, HTMLTable is a copy of the HTMLTable in gwt trunk, and its
very outdated. HTMLTable.iterator() currently returns a non-type case
Iterator.
Fix:
====
Change the return type to Iterator<Widget>. Note that in the long term,
we are replacing PagingScrollTable with various new widgets in trunk, at
which point the old versions of files will be deprecated and eventually
removed.
Testing:
========
All tests pass. This change shouldn't affect anything.
Please review this at http://gwt-code-reviews.appspot.com/137808
Affected files:
src/com/google/gwt/gen2/table/override/client/HTMLTable.java
src/com/google/gwt/widgetideas/table/client/overrides/HTMLTable.java
Index: src/com/google/gwt/widgetideas/table/client/overrides/HTMLTable.java
===================================================================
--- src/com/google/gwt/widgetideas/table/client/overrides/HTMLTable.java
(revision 1746)
+++ src/com/google/gwt/widgetideas/table/client/overrides/HTMLTable.java
(working copy)
@@ -1023,7 +1023,7 @@
*
* @return the iterator
*/
- public Iterator iterator() {
+ public Iterator<Widget> iterator() {
return widgetMap.widgetIterator();
}
Index: src/com/google/gwt/gen2/table/override/client/HTMLTable.java
===================================================================
--- src/com/google/gwt/gen2/table/override/client/HTMLTable.java (revision
1746)
+++ src/com/google/gwt/gen2/table/override/client/HTMLTable.java (working
copy)
@@ -1019,7 +1019,7 @@
*
* @return the iterator
*/
- public Iterator iterator() {
+ public Iterator<Widget> iterator() {
return widgetMap.widgetIterator();
}
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
