[bv-dev] Node#isInIterable() for arrays

Emmanuel Bernard emmanuel at hibernate.org
Mon Jun 19 08:58:59 EDT 2017


> On 16 Jun 2017, at 12:48, Gunnar Morling <gunnar at hibernate.org> wrote:
> 
>> 
>> 
>> Back to the core subject, I can’t quite remember why I added Map to the list of isIterable() elements.
>> I suppose that if we follow that path, any container that returns multiple elements should return isIterable == true (i.e. containers that call iterableValue, indexedValue or keyedValue.
>> So I’d add arrays and any multi-valued container to the definition of isIterable()
> 
> Ok, cool. I'll add arrays then.
> 
> For other containers, I'd add this when adding more built-in
> containers to the spec in the future.
> 
> 

My point is that isIterable goes beyond the built-in containers. It’s is true of custom containers (e.g. Guava’s etc). If we don’t change the JavaDoc, this is misleading.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/beanvalidation-dev/attachments/20170619/f198e80e/attachment.html 


More information about the beanvalidation-dev mailing list