[bv-dev] Bean Validation 1.1 -> 2.0 review
gunnar at hibernate.org
Thu Jun 8 04:18:42 EDT 2017
Thanks! PR addressing your remarks is coming up.
2017-06-02 9:35 GMT+02:00 Emmanuel Bernard <emmanuel at hibernate.org>:
> Hey all
> I could not do much but I promised Gunnar to review one last time the 1.1 ->
> 2.0 diff
> I did not find much and dod not checked it all but here is at least a few
> Example 3.10: Multi-valued constraint declaration using explicit @List
> I’d put in the example title that it is a discouraged pattern
> "composing constraints are directly given on the composed constraint” ->
> "composing constraints are directly given on the composed constraint (i.e.
> via the repeatable annotation feature)"
> "refers to the left-to-right order” -> I got puzzled by that one especially
> as we can put them on new lines. I would say “refers to the order in which
> they are declared”
"left-to-right-order" is a term used by the JLS, I think it's fine to
use it here as well.
> Example 3.21 / 3.24
> I find it odd to ahve the private method return null
> Can we make it a full example?
Ok, this example was pre-existing from 1.x times :) Replacing it with
a complete example.
> 4.1 add an exemple of non generic container usage maybe
There is an example for this (OptionalInt) in 4.4 (which contains all
the examples for this chapter).
> A value extractor for java.util.Optional:
> Maybe a comment reminding that the null is passed to have no path node entry
> Maybe more to come later
> beanvalidation-dev mailing list
> beanvalidation-dev at lists.jboss.org
More information about the beanvalidation-dev