http://gwt-code-reviews.appspot.com/1369802/diff/1/user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java
File
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java
(right):

http://gwt-code-reviews.appspot.com/1369802/diff/1/user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java#newcode468
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:468:
for (ConstraintDescriptor<?> constraint :
beanDescriptor.getConstraintDescriptors()) {
beanDescriptor.getConstraintDescriptors() returns a
ConstraintDescriptor<?>, which is not necessarily a
ConstraintDescriptor<? extends Annotation> (at least, javac thinks so
when I do 'ant dist-dev' -- Eclipse doesn't complain).

But, you'll notice the loop never even dereferences the loop variable
(we could equally well use a simple counted loop).

So, I just removed the bogus generic constraint.  I'd be happy to
rewrite the loop to make a bit more sense.

On 2011/02/28 19:16:56, jat wrote:
So what is this change doing?


http://gwt-code-reviews.appspot.com/1369802/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to