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()) {
On 2011/02/28 19:42:45, rice wrote:
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).

Hmm, surprised this works on JDK 1.6.

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.

Yes, how about just doing the loop for
beanDescriptor.getConstraintDescriptors().size() times instead?

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

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

Reply via email to