> -0, to pull in another dependency just for something that is trivial to > code by hand.
We could also add the small validation class I already wrote for JGit [1]. We could easily put them into plexus-utils which we already use excessively. Licensed under ASL for sure. LieGrue, strub [1] http://github.com/sonatype/JGit/blob/ab251fae2b9f22806eecf58656db357275e1a8b2/org.spearce.jgit/src/org/spearce/jgit/util/Validate.java --- Benjamin Bentmann <[email protected]> schrieb: > Mark Struberg wrote: > > > Shouldn't we use commons.lang.Validate > > > > Validate.notNull(blablubParameter, "blablubParameter must not be null!"); > > > > or at least throw InvalidArgumentExceptions instead of NPE? > > +1, to throw IAE instead of NPEs. The former one gives a clearer > indication which party (client vs provider) misbehaved. > > -0, to pull in another dependency just for something that is trivial to > code by hand. > > > Benjamin --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
