I noticed that JBeanMethod (and BeanMethod) treat java.lang.Boolean the same as boolean, contrary to my first patch, hence the changes to the unit test between the two patch sets.
http://gwt-code-reviews.appspot.com/1359804/diff/1/3 File user/src/com/google/gwt/editor/rebind/model/EditorModel.java (right): http://gwt-code-reviews.appspot.com/1359804/diff/1/3#newcode504 user/src/com/google/gwt/editor/rebind/model/EditorModel.java:504: String getterName = camelCase("get", parts[i]); On 2011/02/18 14:43:47, bobv wrote:
Use JBeanMethod from the autobean rebind package instead. Build up
something
like a memoized Map<Class, Map<String, JMethod>> for any method
matched by
JBeanMethod.GET (which understands is/has), using the inferName()
method to
extract the property name.
There's also an associated BeanMethod in autobean.server.impl that has
the same
setup, if you ever need to do something similar with reflection-based
code. Done. In the end, I almost completely rewrote the method. I find it more readable now (for instance, it no longer deals with removing the last added thing to StringBuilders, it instead doesn't append in the first place when on the last path), hope you'll agree. http://gwt-code-reviews.appspot.com/1359804/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
