+1 to move it to the util package. Rodrigo
On Tue, May 20, 2014 at 10:11 AM, Jörn Kottmann <[email protected]> wrote: > On 05/19/2014 11:24 PM, Mark G wrote: >> >> OK, thanks Carlos, I think I will commit the change, seems like it >> wouldn't >> hurt. Anybody else? > > > We can't do it like that, the cmdline package is marked as internal usage > only. Therefore the classes in there should not be > used outside of it. Inside cmdline pacakge the code is calling this factory > method: > CmdLineUtil.createInputStreamFactory > > Anyway, since user will need to create InputStreamFactory instances here and > there it might be a good > idea to move this class to the util package, or introduce an > InputStreamFactoryUtil class which offers a couple > of factory methods for various inputs (e.g. File, URL, Classpath, etc.). > > Any thoughts? > > Jörn >
