<div dir="ltr">+1 Javier</div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, 10 Feb 2025 at 20:40, Howard Butler 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:1px solid rgb(204,204,204);padding-left:1ex">+1 <br>
<br>
Howard<br>
<br>
> On Feb 10, 2025, at 9:53 AM, Even Rouault via gdal-dev <<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a>> wrote:<br>
> <br>
> 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 OGRLayer::IGetExtent() and OGRLayer::ISetSpatialFilter()<br>
>> <br>
>> 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.<br>
>> <br>
>> Summary<br>
>> -------<br>
>> <br>
>> This RFC changes the prototype of the OGRLayer::GetExtent(), GetExtent3D(),<br>
>> SetSpatialFilter() and SetSpatialFilterRect() methods.<br>
>> <br>
>> Motivation<br>
>> ----------<br>
>> <br>
>> Originally GetExtent(), SetSpatialFilter() and SetSpatialFilterRect() were<br>
>> designed for a single geometry field. When support for multiple geometry fields<br>
>> was added per :ref:`rfc-41`, alternate virtual methods were added to accept a<br>
>> ``int iGeomField`` argument, but this causes slightly repeating code 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>
<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>