<div dir="ltr"><div><div>Hi Matt,<br><br></div>Yes, you're right, looks like our implementation has always been too permissive in this regard.<br><br></div><div>I think I'll make our implementation less permissive and fix the related TCK failures. I've implemented the check in HV and AFAICS you have found all the occurrences of this issue.<br><br></div><div>Will create a TCK issue and fix.<br></div><div><br>-- <br></div><div>Guillaume<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 16, 2018 at 8:18 PM, Matt Benson <span dir="ltr"><<a href="mailto:mbenson@apache.org" target="_blank">mbenson@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">Similarly, ZeroConstraintValidator is declared as targeting @Negative.</div><div class="m_-1257211368113888860HOEnZb"><div class="m_-1257211368113888860h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mar 16, 2018 1:44 PM, "Matt Benson" <<a href="mailto:mbenson@apache.org" target="_blank">mbenson@apache.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Mar 16, 2018 1:39 PM, "Matt Benson" <<a href="mailto:mbenson@apache.org" target="_blank">mbenson@apache.org</a>> wrote:<br type="attribution"><blockquote class="m_-1257211368113888860m_-870945243688009773m_-2050025175811098442quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">FrenchAddressListContainer defines @FrenchZipcodeListContainer as a constraint that is validated by FrenchZipcodeConstraintValidat<wbr>or, but that class declares its constraint type parameter as @FrenchZipCode. </div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Same for FrenchZipcodeMixDirectAnnotati<wbr>onAndListContainer.</div><div dir="auto"><br></div><div dir="auto">Because</div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="m_-1257211368113888860m_-870945243688009773m_-2050025175811098442quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"> that CV type does not override #init() this presumably works in the RI, and the spec is a bit vague on this point, but the interface definition describes type parameter A as "the annotation type handled by an implementation." The spec section ConstraintValidator resolution algorithm also correlates constraint A to the type parameters of the ConstraintValidators described in the resolution rules. Finally, I don't see anything under "What's new in 2.0" to indicate that this should be a legal constraint definition.<div dir="auto"><br></div><div dir="auto">Can I get a ruling? :)</div><font color="#888888"><div dir="auto"><br></div><div dir="auto">Matt</div></font></div>
</blockquote></div><br></div></div></div>
</blockquote></div></div>
</div></div><br>______________________________<wbr>_________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org" target="_blank">beanvalidation-dev@lists.jboss<wbr>.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailma<wbr>n/listinfo/beanvalidation-dev</a><br></blockquote></div><br></div></div></div></div>