[gdal-dev] ISO WKB
Even Rouault
even.rouault at mines-paris.org
Tue Dec 17 13:50:59 PST 2013
Le mardi 17 décembre 2013 22:38:26, Paul Ramsey a écrit :
> OK, so hide the ISO types from the outside world. No problem.
>
> Is it OK to have getGeometryType and exportToWkb accept wkbVariant
> optional parameters?
For exportToWkb(), it is just a matter of taste whether to add an optional
parameter or to have a dedicated method.
For getGeometryType(), as it returns a OGRwkbGeometryType, you can't add an
optional parameter to return values other than OGRwkbGeometryType.
My latest proposal was to have a - protected - "int
getGeometryType(wkbVariant) { return
(eVariant == wkbVariantOgc) ? getGeometryType() :
getIsoGeometryType(); }" and a public OGRwkbIsoGeometryType
getIsoGeometryType().
>
> P.
>
> On Tue, Dec 17, 2013 at 1:03 AM, Even Rouault
>
> <even.rouault at mines-paris.org> wrote:
> > Selon Paul Ramsey <pramsey at cleverelephant.ca>:
> >> Back to this, is it OK?
> >
> > As said in
> > http://lists.osgeo.org/pipermail/gdal-dev/2013-December/037738.html, I
> > feel a bit unconfortable with the extension of the OGRwkbGeometryType
> > enumeration that has possible impacts on other parts of OGR. There's
> > perhaps a time where we will touch it, but I'd expect it to ideally
> > embrace Z, M, ZM, circular geometries at once. And that would deserve a
> > RFC.
> >
> > What do you think of keeping it an internal enumeration of OGR, since
> > that's probably all you need for now ?
> >
> > "Or have a separate OGRwkbIsoGeometryType enumeration { wkbPointIso, ...
> > wkbGeometryCollectionIso, wkbPointIsoZ, ... wkbGeometryCollectionIsoZ },
> > a getIsoGeometryType() method that returns it, and the exportToWkb()
> > methods that calls int getGeometryType(OGRwkbVariant eVariant) { return
> > (eVariant == wkbVariantOgc) ? getGeometryType() : getIsoGeometryType();
> > }"
> >
> > I'd be happy to hear about other GDAL developers opinion on this.
> >
> >> How are we patching back to SVN? I can convert
> >> it into a patch and attach to a ticket, if that's the path.
> >
> > git-svn can be used to bridge the 2 worlds, but in my recent experience
> > it has been painful to use. So generating a patch and applying it is
> > probably easier.
> >
> > Even
> >
> > --
> > Geospatial professional services
> > http://even.rouault.free.fr/services.html
--
Geospatial professional services
http://even.rouault.free.fr/services.html
More information about the gdal-dev
mailing list