Re: [Geotools-devel] ReferencedEnvelope3D integration

2012-12-04 Thread Jody Garnett
> Hi Jody, > > Sounds like it might be a lot of work ? I'd imagine there are too > many fiddly cases in the code to be able to push it all through a > shell script. > > Having a look now - searching for "new ReferencedEnvelope" --> 630 matches. I think I could do a search and replace for "new

Re: [Geotools-devel] ReferencedEnvelope3D integration

2012-12-04 Thread Niels Charlier
I think with the different ReferencedEnvelope classes, it is best to use a factory-like strategy to create envelopes rather than to call the constructor directly. Makes sense. On 12/04/2012 08:51 AM, Jody wrote: Correct. The alternative is to check the CRS yourself. It is a pure factory meth

Re: [Geotools-devel] ReferencedEnvelope3D integration

2012-12-03 Thread Jody
Correct. The alternative is to check the CRS yourself. It is a pure factory method in that respect. -- Jody Sent with Sparrow (http://www.sparrowmailapp.com/?sig) On Tuesday, 4 December 2012 at 5:11 PM, Andrea Aime wrote: > On Mon, Dec 3, 2012 at 11:02 PM, Jody Garnett (mailto:jody.garn.

Re: [Geotools-devel] ReferencedEnvelope3D integration

2012-12-03 Thread Andrea Aime
On Mon, Dec 3, 2012 at 11:02 PM, Jody Garnett wrote: > Niels and myself were discussing what more can be done for > ReferenceEnveloped3D integration in GeoTools. > > I have asked Niels to look over the javadocs, and add a code example to > the user guide. > > The other thing we *could* do is go t

Re: [Geotools-devel] ReferencedEnvelope3D integration

2012-12-03 Thread Michael Bedward
Hi Jody, Sounds like it might be a lot of work ? I'd imagine there are too many fiddly cases in the code to be able to push it all through a shell script. But it would be nice for the sources to be consistent with recommended usage which I assume is to use create() rather than constructor. Micha