@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
-~----------~----~----~----~------~----~------~--~---

Reply via email to