Author: [EMAIL PROTECTED]
Date: Mon Dec 1 11:09:39 2008
New Revision: 4214
Modified:
branches/1_6_datepicker/user/src/com/google/gwt/user/datepicker/client/ElementMapper.java
Log:
beef up TODO per jlabanca's review of r4165. TBR jlabanca
Modified:
branches/1_6_datepicker/user/src/com/google/gwt/user/datepicker/client/ElementMapper.java
==============================================================================
---
branches/1_6_datepicker/user/src/com/google/gwt/user/datepicker/client/ElementMapper.java
(original)
+++
branches/1_6_datepicker/user/src/com/google/gwt/user/datepicker/client/ElementMapper.java
Mon Dec 1 11:09:39 2008
@@ -133,7 +133,9 @@
if (lastIndex < 0) {
throw new IllegalStateException();
}
- // TODO(ecc) ???
+ // TODO(ecc) Why are we casting to Widget and working only in
tables?
+ // Is this code even executed? If not, let's yank it
+ // and throwing an UnsupportedOperationException
Widget w = (Widget) uiObjectList.get(lastIndex);
assert (w.getParent() instanceof HTMLTable);
w.removeFromParent();
--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---
