Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-10 Thread Ben Caradoc-Davies
+1. Looks good to me. This change should be fine on master. Kind regards, Ben. On 07/04/13 22:22, Andrea Aime wrote: > Hi all, > here is a proposal to upgrade the shapefile-ng module to supported > status, and make shapefile become (temporarily) an unsupported module > named shapefile-old: > http

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-10 Thread Chris Holmes
+1 here, sounds awesome. On Wed, Apr 10, 2013 at 10:49 AM, Justin Deoliveira wrote: > Sorry for the late vote, +1. > > > On Tue, Apr 9, 2013 at 1:57 AM, Simone Giannecchini < > simone.giannecch...@geo-solutions.it> wrote: > >> Ciao Andrea, >> I tried to put my +1 on the page but it looks like I

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-10 Thread Justin Deoliveira
Sorry for the late vote, +1. On Tue, Apr 9, 2013 at 1:57 AM, Simone Giannecchini < simone.giannecch...@geo-solutions.it> wrote: > Ciao Andrea, > I tried to put my +1 on the page but it looks like I can't log in. > I'll try to reset the pwd, but, yeah, this is a +1. > > Regards, > Simone Giannecch

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-09 Thread Simone Giannecchini
Ciao Andrea, I tried to put my +1 on the page but it looks like I can't log in. I'll try to reset the pwd, but, yeah, this is a +1. Regards, Simone Giannecchini == GeoServer training in Milan, 6th & 7th June 2013! Visit http://geoserver.geo-solutions.it for more information. == Ing. Simone Gianne

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-08 Thread Andrea Aime
On Sun, Apr 7, 2013 at 4:22 PM, Andrea Aime wrote: > Hi all, > here is a proposal to upgrade the shapefile-ng module to supported status, > and make shapefile become (temporarily) an unsupported module named > shapefile-old: > http://docs.codehaus.org/display/GEOTOOLS/Migrate+shapefile+to+shapefil

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-08 Thread Andrea Aime
On Mon, Apr 8, 2013 at 1:34 AM, Jody Garnett wrote: > I tried to provide word from uDig - we already use shapefile-ng in order > to take advantage of sorting. > > I have filled in a few tasks on your proposal (to specific code examples > needing updating). > Ah, sorry, I updated the docs as wel

Re: [Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-07 Thread Jody Garnett
I tried to provide word from uDig - we already use shapefile-ng in order to take advantage of sorting. I have filled in a few tasks on your proposal (to specific code examples needing updating). Thoughts: - tempted to introduce matching constructors, and marking them deprecated to give client

[Geotools-devel] Proposal: upgrading the shapefile module to shapefile NG

2013-04-07 Thread Andrea Aime
Hi all, here is a proposal to upgrade the shapefile-ng module to supported status, and make shapefile become (temporarily) an unsupported module named shapefile-old: http://docs.codehaus.org/display/GEOTOOLS/Migrate+shapefile+to+shapefile-ng and here is the pull request: https://github.com/geotools