Still LGTM I mean one rethrow nit below HERE
http://gwt-code-reviews.appspot.com/68805/diff/1/24 File user/src/com/google/gwt/uibinder/parsers/DockLayoutPanelParser.java (right): http://gwt-code-reviews.appspot.com/68805/diff/1/24#newcode115 Line 115: writer.die("Unexpected: Unable to find %s type", SplitLayoutPanel.class Oh, I see. NFE is checked. Boooo. Still would prefer to see throw new RuntimeException(e); http://gwt-code-reviews.appspot.com/68805 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---