Btw, regading this code in ArchiveStreamFactory :
if (clazz == null) {
throw new ArchiveException("ArchiverFactory could not create
instance");
}
Shouldn't this throw a NullPointerException instead ?
I would like to be carefully with throwing NullPointerException. In
that
case we simply
wouldn't find a matching implementation - i think a better message
but still
archiveexception
will do fine.
+1 that would not be the place to throw a NPE IMO
cheers
--
Torsten
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
