@Bob: Kudos for fixing this instead of just observing the problem and moving on. You rock for setting this kind of example.
On Tue, Apr 7, 2009 at 1:52 PM, <codesite-nore...@google.com> wrote: > > Author: b...@google.com > Date: Tue Apr 7 10:47:05 2009 > New Revision: 5193 > > Modified: > trunk/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java > > Log: > Fix error message to include the property name when a binding vs. > configuration property mismatch occurs. > > Patch by: bobv > Review by: jgw (TBR) > > > Modified: trunk/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java > > ============================================================================== > --- trunk/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java > (original) > +++ trunk/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java Tue > Apr > 7 10:47:05 2009 > @@ -791,8 +791,8 @@ > if (concreteType.isInstance(prop)) { > return prop; > } > - logger.log(TreeLogger.ERROR, "The specified property '" + attr > - + "' is not of the correct type; found '" > + logger.log(TreeLogger.ERROR, "The specified property '" > + + prop.getName() + "' is not of the correct type; found '" > + prop.getClass().getSimpleName() + "' expecting '" > + concreteType.getSimpleName() + "'"); > } else { > > > > --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---