<div dir="ltr">+1 Dan<br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Feb 10, 2025 at 3:27 PM Even Rouault via gdal-dev <<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
I move to adopt RFC 107<br>
<br>
Starting with my +1<br>
<br>
Even<br>
<br>
Le 06/02/2025 à 22:19, Even Rouault via gdal-dev a écrit :<br>
> Hi,<br>
><br>
> Please review <a href="https://github.com/OSGeo/gdal/pull/11814" rel="noreferrer" target="_blank">https://github.com/OSGeo/gdal/pull/11814</a>: Add <br>
> OGRLayer::IGetExtent() and OGRLayer::ISetSpatialFilter()<br>
><br>
> It could have gone through a pull request business-as-usual, but as <br>
> this impacts a number of drivers, including out-of-tree ones, this is <br>
> worth this small RFC.<br>
><br>
> Summary<br>
> -------<br>
><br>
> This RFC changes the prototype of the OGRLayer::GetExtent(), <br>
> GetExtent3D(),<br>
> SetSpatialFilter() and SetSpatialFilterRect() methods.<br>
><br>
> Motivation<br>
> ----------<br>
><br>
> Originally GetExtent(), SetSpatialFilter() and SetSpatialFilterRect() <br>
> were<br>
> designed for a single geometry field. When support for multiple <br>
> geometry fields<br>
> was added per :ref:`rfc-41`, alternate virtual methods were added to <br>
> accept a<br>
> ``int iGeomField`` argument, but this causes slightly repeating code <br>
> patterns<br>
> in most drivers, and omissions of the boilerplate can cause bugs.<br>
><br>
> Even<br>
><br>
-- <br>
<a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
My software is free, but my time generally not.<br>
<br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
</blockquote></div>