Andrea Aime wrote:
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?
I agree with Andrea here. It's nice to have discrete units. And it also makes it so if part of that module is not maintained then we're not releasing broken or bad code. We can just drop a module or put it in the community section if no developers are interested, so there's no huge burden on the imageio module maintainer to keep everything up to date.

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...
Sounds good.

   * 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 :-)
+1 here too.

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

!DSPAM:1003,44b92b17261254750375898!


--
Chris Holmes
The Open Planning Project
http://topp.openplans.org
begin:vcard
fn:Chris Holmes
n:Holmes;Chris
org:The Open Planning Project
adr:;;377 Broadway, 11th Floor;New York;NY;10013;USA
email;internet:[EMAIL PROTECTED]
title:VP, Strategic Development
x-mozilla-html:FALSE
url:http://topp.openplans.org
version:2.1
end:vcard

-------------------------------------------------------------------------
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

Reply via email to