<div dir="auto">Agreed; they should be supported as is Optional itself.<div dir="auto"><br></div><div dir="auto">Matt</div></div><div class="gmail_extra"><br><div class="gmail_quote">On May 9, 2017 10:39 AM, "Gunnar Morling" <<a href="mailto:gunnar@hibernate.org">gunnar@hibernate.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I'm curious about your take on supporting the types in ${subject}<br>
(BVAL-579 [1]).<br>
<br>
They are non-generic wrappers for int, long and double. Should we<br>
support them with the numeric constraints such as @Min et al.? The<br>
easiest way to do so would be to just mandate support in the JavaDoc<br>
of the numeric constraint types.<br>
<br>
The only thing I can see speaking against this is that we may migrate<br>
to an extractor-based approach in a future revision. Currently<br>
extractors cannot be used, as those types don't have any type<br>
parameter which could be extracted. But assuming we extend the<br>
extractor API in a future revision to deal with non-generic types,<br>
too, we could then rather mandate built-in extractors for those types.<br>
Which will allow to put *any* constraint applying to int also to<br>
OptionalInt.<br>
<br>
Should we do such change in a future revision, I don't think anything<br>
would change for users. Only providers would have to implement support<br>
for these types via extractors instead of dedicated constraint<br>
validators. I think such change is acceptable.<br>
<br>
What do others think?<br>
<br>
Thanks,<br>
<br>
--Gunnar<br>
<br>
[1] <a href="https://hibernate.atlassian.net/browse/BVAL-579" rel="noreferrer" target="_blank">https://hibernate.atlassian.<wbr>net/browse/BVAL-579</a><br>
______________________________<wbr>_________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.<wbr>jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/<wbr>beanvalidation-dev</a><br>
</blockquote></div></div>