Wrapping 5 ArrayUtils issues up in one email to see if we can make
decisions on them.

LANG-470        Add containsAll(Object[], Object[]) methods to ArrayUtils
LANG-531        Add defaultIfEmpty or nullIfEmpty to ArrayUtils
LANG-536        Add isSorted() to ArrayUtils
LANG-537        Add ArrayUtils.toArray to create generic arrays
LANG-534        ArrayUtils should have method to convert null arrays to
empty ones to help with Defensive coding

Going through each:

LANG-470.
  Vaguely interesting, but I don't know what containsAll( array, null
) and containsAll( null, null ) would do. This is effectively a set
method on arrays, so I'm thinking more in the direction of adding an
Set<T> ArrayUtils.toSet(T[]) method. It would mean a, potentially
non-public, ArraySet class.

LANG-531.
  I don't get the usefulness of the nullIfEmpty notion. Seems like an
odd paradigm to prefer null over empty (i.e. null safe) values. I'm
inclined to WONTFIX.

LANG-536.
  Needs tests, but I'm +1 to add this. Feels valuable despite the
annoyance of more primitive[] overloads bloating code.

LANG-537.
  I don't get this one. Presumably I'm missing something.

LANG-534.
  More irritating code bloat, but feels like a fair enough feature.
Opposite of LANG-531 and the more common pattern imo.


Thoughts?

Hen

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to