Re: [Geotools-devel] ArcSDEPyramid

Fri, 23 Jan 2009 15:20:57 -0800

Gabriel,

That sounds 100% fine to me.

--saul

Gabriel Roldan wrote: 
btw, I'm moving the ArcSDEPyramid and ArcSDEPyramidLevel classes from the 
org.geotools.arcsde.gce package to the org.geotools.arcsde.gce.imageio package 
as I think they better fit there. Otherwise we end up with cyclic dependencies 
on those packages and the org.geotools.arcsde.gce package interface seems 
overloaded with unneeded clutter to me, where it should only contain the 
Format and Coverage2DReader implementations, and the last one being package 
visible, so the org.geotools.arcsde.gce package interface is just the Format 
class. Does it seem good?

Cheers,
gabriel

On Friday 23 January 2009 15:55:46 Gabriel Roldan wrote:
  
Hey Saul,

I'm having a bit of trouble understanding the usage of ArcSDEPyramid, may
be you can bring some light :)

I'm pretty sure I've come to an exception in ArcSDEPyramid constructor last
week with a raster dataset for which skipLevelOne was set to false.

My question is regarding the following snippet in the constructor:

for (int i = 0; i < numLevels; i++) {
                if (i == 1 && rasterAttributes.skipLevelOne()) {
                    continue;
                }
..........

shouldn't it be like this:
if (i == 0 && rasterAttributes.skipLevelOne())

Cheers
    

  
------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to