[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