Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-12 Thread Ben Caradoc-Davies
On 13/08/12 14:08, Andrea Aime wrote: > Not really, as I said, CRS:84 is expected to be in lon/lat regardless of > the hints you provide, > so the test above is not actually testing the effects of the hints (as > there should be none). OK. This test only does what the URN equivalent does. I think

Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-12 Thread Andrea Aime
On Mon, Aug 13, 2012 at 4:19 AM, Ben Caradoc-Davies < [email protected]> wrote: > > This is exactly what happens in the GeoServer test, where the fixture > forces lon/lat for "http" (http://www.opengis.net/gml). The rest of the > test confirms that > http://www.opengis.net/def/**crs/EPSG

Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-12 Thread Ben Caradoc-Davies
Andrea, thanks very much for your prompt review. On 12/08/12 23:15, Andrea Aime wrote: > However, I'm wondering what happens if you set the following: > Hints.putSystemDefault(Hints.FORCE_AXIS_ORDER_HONORING, > "http"); > Hints.putSystemDefault(Hints.FORCE_LONGITUDE_FIRST_AXIS_ORDER,

Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-12 Thread Andrea Aime
On Thu, Aug 9, 2012 at 5:53 AM, Ben Caradoc-Davies < [email protected]> wrote: > Thanks, if you are the module maintainer I am more than happy for you > review the patch. > Reviewed the patch, looks good but I still have some doubts about the bits below. > > > What I'm wondering afte

Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-08 Thread Ben Caradoc-Davies
On 08/08/12 20:08, Andrea Aime wrote: > I only managed to have a quick look at the patch so far, if you don't > mind as the current referencing > maintainer I would like to give it a better look during the weekend > before it get committed. Thanks, if you are the module maintainer I am more than h

Re: [Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-08 Thread Andrea Aime
On Wed, Aug 8, 2012 at 9:36 AM, Ben Caradoc-Davies < [email protected]> wrote: > For the record, this has turned into a substantial refactoring of > URN_AuthorityFactory and its helpers to ensure the new > HTTP_URI_AuthorityFactory has the same functionality and that code > duplication i

[Geotools-devel] Referencing changes to support OGC HTTP URI srsName parsing

2012-08-08 Thread Ben Caradoc-Davies
For the record, this has turned into a substantial refactoring of URN_AuthorityFactory and its helpers to ensure the new HTTP_URI_AuthorityFactory has the same functionality and that code duplication is avoided: https://jira.codehaus.org/browse/GEOT-4160 This change targets master only, and use