Seems I'm blind; thanks Ian. Mind changing that, Dan? We really need tests for this stuff sometime soon -- of course, other than syntax bugs like this, it's damned near impossible to test this sort of thing, because so many of these properties have no testable effects, and having a typo in the property name just gives you an expando...
On Thu, Jul 16, 2009 at 1:30 PM, Ian Petersen <[email protected]> wrote: > > On Thu, Jul 16, 2009 at 8:12 AM, <[email protected]> wrote: > > Index: user/src/com/google/gwt/dom/client/AnchorElement.java > > =================================================================== > > --- user/src/com/google/gwt/dom/client/AnchorElement.java (revision > 5739) > > +++ user/src/com/google/gwt/dom/client/AnchorElement.java (working > copy) > > @@ -165,7 +165,7 @@ > > * @see <a > > href=" > http://www.w3.org/TR/1999/REC-html401-19991224/struct/links.html#adef-rel > ">W3C > > HTML Specification</a> > > */ > > public final native void setRel(String name) /*-{ > > - this.name = name; > > + this.rel = rel; > > }-*/; > > Is that right? Do you need to change the argument's name from "name" > to "rel" or is there some bug in the diff? > > Ian > > > > --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
