+1 on everything Hardy said :) There's not really much I could add.<div><br></div><div>--Gunnar</div><div class="gmail_extra"><br><br><div class="gmail_quote">2012/12/12 Hardy Ferentschik <span dir="ltr"><<a href="mailto:hardy@hibernate.org" target="_blank">hardy@hibernate.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 for 2b as well and I am fine with package annotation. Given that many people group their packages<br>
around things like domain vs service classes, I think it can make configuration easier.<br>
<br>
I have to admit that 1b is growing on me, if getters would just not be excluded per default.<br>
In contrast to Sebastian I actually think that if a user configures its app to use method validation via some sort of interceptor or similar he<br>
wants it to occur for all methods. If a getter is annotated with @NotNull and I call this method I don't care whether<br>
is is returning a state variable or whether this is a calculated value. The returned value is supposed to be non<br>
null.<br>
<br>
Note, I am still against enabling method validation per default in a CDI environment. I still think it should an active choice to enable the<br>
appropriate technology. This also mitigates the problem of backwards compatibility imo.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Hardy<br>
</font></span><div class="im HOEnZb"><br>
<br>
On 11 Jan 2012, at 7:32 PM, Gerhard Petracek <<a href="mailto:gerhard.petracek@gmail.com">gerhard.petracek@gmail.com</a>> wrote:<br>
<br>
> +1 for 2.b but -1 for a package annotation and/or config.<br>
<br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
</div></div></blockquote></div><br></div>