Ok. I made that change and it appears to work. Latest code available here: https://github.com/klassenjs/ gdal/tree/esrijson-curves
On Mon, Jul 2, 2018 at 1:56 PM, Even Rouault <[email protected]> wrote: > > > > In the first two cases (without OLCCurveGeometries set) I see > > curveToLineString is being called when there are curves in the data, but > > not on every result page (again curves are relatively rare in the > dataset). > > ok, then replace the call to OGRMemLayer::SetFeature() with > OGRMemLayer::ISetFeature() in ogrgeojsonlayer.cpp to shortcut the > linearization that is done by the generic OGRLayer::SetFeature() > > And you can then remove OLCCurveGeometries > > > > In the last two cases (with OLCCurveGeometries set) I see > > curveToLineString is being called with a different back trace (related > > to OGRGeometryFactory::organizePolygons): > > That one is fine. It is an implementation detail of organizePolygons() > > -- > Spatialys - Geospatial professional services > http://www.spatialys.com >
_______________________________________________ gdal-dev mailing list [email protected] https://lists.osgeo.org/mailman/listinfo/gdal-dev
