Hi,
Please review https://github.com/OSGeo/gdal/pull/11814: Add
OGRLayer::IGetExtent() and OGRLayer::ISetSpatialFilter()
It could have gone through a pull request business-as-usual, but as this
impacts a number of drivers, including out-of-tree ones, this is worth
this small RFC.
Summary
-------
This RFC changes the prototype of the OGRLayer::GetExtent(), GetExtent3D(),
SetSpatialFilter() and SetSpatialFilterRect() methods.
Motivation
----------
Originally GetExtent(), SetSpatialFilter() and SetSpatialFilterRect() were
designed for a single geometry field. When support for multiple geometry
fields
was added per :ref:`rfc-41`, alternate virtual methods were added to
accept a
``int iGeomField`` argument, but this causes slightly repeating code
patterns
in most drivers, and omissions of the boilerplate can cause bugs.
Even
--
http://www.spatialys.com
My software is free, but my time generally not.
_______________________________________________
gdal-dev mailing list
gdal-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/gdal-dev
