Martin Desruisseaux ha scritto: > Hello all > > I noticed that two new experimental modules, "arcgrid" and "ArcGrid_ImageIO" > have been commited to > trunk. Thanks for commiting them on trunk; it make easier for me to take a > look at them soon (I tend > to look at branches much least frequently than trunk). I would like to share > the following though: > > * Would it be better to put those modules in "plugins" rather than "ext", > since they looks like Image I/O and GridCoverageExchange plugins to me? > They are not there because they are meant to be a replacement for the arcgrid module > * Since I do not expect "ArcGrid_ImageIO" to be a huge module, and since I > do not expect > most of the other pure-Java Image I/O modules to be huge neither, would > it make sense > to create a plain "plugin/imageio" module instead and put ArcGrid as > well as any other > "reasonably sized" image I/O plugins there? I would prefer a ~500 kb > module instead than > dozen of ~50 kb modules. I disagree. I do prefer smaller modules, because I don't want to force people to add weight to their apps distribution if it does not provide any value to them. So, if I don't need to read the arcgrid format, why should I have the correspondent classes in my classpath? Yet, I'd prefer to see both ext arcgrid modules merged into one, since if I need one I also need the other as well... > * The current package names are: > > - org.geotools.gce.imageio.asciigrid > - org.geotools.gce.arcgrid > > Can I suggest the following package names instead? > > - org.geotools.image.io.<something> > - org.geotools.coverage.io.<something> > > I like the convention you propose better too, +1 :-) Cheers Andrea
------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
