[gdal-dev] Call for review on RFC 107: Add OGRLayer::IGetExtent() and OGRLayer::ISetSpatialFilter()
Even Rouault
even.rouault at spatialys.com
Thu Feb 6 13:19:47 PST 2025
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.
More information about the gdal-dev
mailing list