On 19/08/2009, Niall Pemberton <niall.pember...@gmail.com> wrote: > On Wed, Aug 19, 2009 at 1:37 PM, Jukka Zitting<jukka.zitt...@gmail.com> wrote: > > Hi, > > > > On Wed, Aug 19, 2009 at 10:47 AM, Jörg Schaible<joerg.schai...@gmx.de> > wrote: > >> The only concern I have are the new Broken(In/Out)putStream > implementations. > >> IMHO those are somewhat out-of-scope for IO, you explicitly mention that > >> these implementations are for testing purposes. I'd rather see > commons-test > >> reactivated than putting such code into a component that is normally used > >> in production code. > > > > If you feel strongly about this I guess we can do that or move the > > broken streams to src/test/java inside Commons IO where they are > > already used for testing the tagged stream classes (any downstream > > users would then need to copy the sources). > > > > But personally I don't understand the "normally used in production" > > argument. Test code benefits from reuse just as much as normal code. > > And there are a lot of test projects (TCKs, etc.) that are in > > production use. > > > > AFAIUI the purpose of Commons is to create reusable code, regardless > > of the scope or field of that reuse. And Commons IO is defined as > > "library of utilities to assist with developing IO functionality". > > IMHO utilities to help test IO code are clearly within this scope. > > > +1
+1, OK by me to keep them in IO. > > Niall > > > > BR, > > > > Jukka Zitting > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org