<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> as they are a bit shorter to use and, more importantly, better to<br> read.</blockquote><div><br></div><div>I agree.</div><div><br></div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> It's late in the game, but I feel inclined to extract @PastOrPresent<br> and @FutureOrPresent. What do you think?</blockquote><br></div><div><br></div><div>Ok for me.<br><div><font color="#500050"><br></font></div><div><br></div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 21, 2017 at 7:33 PM, Gunnar Morling <span dir="ltr"><<a href="mailto:gunnar@hibernate.org" target="_blank">gunnar@hibernate.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Just from - very quickly - thinking, no options are coming to my mind.<br>
<br>
Note that we're about to push the Proposed Final Draft (CR1) with the<br>
separate annotations in a bit. We still an re-consider until the Final<br>
if something strongly speaks against it, but it seemed like the more<br>
likely option to me and the PFD should be out today so it's up online<br>
by the end of the week.<br>
<br>
I wish the question would have come up earlier so it wouldn't be such<br>
a rush, but well...<br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
<br>
2017-06-21 19:00 GMT+02:00 Emmanuel Bernard <<a href="mailto:emmanuel@hibernate.org">emmanuel@hibernate.org</a>>:<br>
> My argument against is that @Past / @Future could need additional options: like maybe past per year or month instead of instant time. I understand we do use the type structure to extract that information.<br>
> Is there potential options people can think of around time constraints?<br>
><br>
> For @Positive/@Negative, we decided there were none.<br>
><br>
>> On 21 Jun 2017, at 11:50, Gunnar Morling <<a href="mailto:gunnar@hibernate.org">gunnar@hibernate.org</a>> wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> A colleague of mine reviewing the BV 2.0 changes pointed out an<br>
>> inconsistency in the added/modified constraints.<br>
>><br>
>> As per the community poll we have now @NegativeOrZero and<br>
>> @PositiveOrZero, whereas we still have the orPresent() option on @Past<br>
>> and @Future. Should it be @PastOrPresent and @FutureOrPresent instead<br>
>> for the sake of consistency?<br>
>><br>
>> Now one could argue that having only attributes (orPresent(),<br>
>> orZero()) is most consistent with other constraints such as<br>
>> @DigitalMin. But indeed the explicit annotations grew on me over time<br>
>> as they are a bit shorter to use and, more importantly, better to<br>
>> read.<br>
>><br>
>> It's late in the game, but I feel inclined to extract @PastOrPresent<br>
>> and @FutureOrPresent. What do you think?<br>
>><br>
>> Thanks,<br>
>><br>
>> --Gunnar<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>
><br>
><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>
______________________________<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>
</div></div></blockquote></div><br></div></div></div></div>