<div dir="ltr">Hi Even, great work!<div><br></div><div>Quick testing was fine and revealed no errors. </div><div><br></div><div>I was able to compile it in QGIS master very easily. The only "issue" I encountered was related to GDALRegisterAll() and OGRAllRegister() registering all drivers (raster and vector), requiring to ignore the vector-only drivers in the raster-only related code in QGIS.</div>
<div><br></div><div>May I suggest a change that would improve backwards-compatibility and avoid unwanted issues? Perhaps these 2 functions should only register raster or vector drivers (as in GDAL 1.x), instead of all drivers, and a new function would register both raster and vector drivers. This would ensure old code behaves as previously, and ensure that no extra drivers are loaded and probed when only raster/vector drivers are needed.</div>
<div><br></div><div>Just to be clearer here is how I would see it:</div><div>GDALRegisterAll() -> registers raster drivers (those with <span style="color:rgb(0,0,0);font-family:Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif;font-size:13px">GDAL_DCAP_RASTER="YES")</span></div>
<div>OGRAllRegister() -> registers vector drivers (those with <span style="font-size:13px;color:rgb(0,0,0);font-family:Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif">GDAL_DCAP_VECTOR="YES")</span></div>
<div>GDALRegisterDrivers() -> registers all drivers</div><div><br></div><div>On the other habd, I imagine this may not be easy to implement and could cause some confusion in the future. Also, probing optimization can instead be achieved using GDALOpenEx() specifying raster vs.vector drivers.</div>
<div><br></div><div>cheers,</div><div>Etienne</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, May 24, 2014 at 11:35 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@mines-paris.org" target="_blank">even.rouault@mines-paris.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Motion : I move to adopt RFC 46: GDAL/OGR unification<br>
><br>
>     <a href="http://trac.osgeo.org/gdal/wiki/rfc46_gdal_ogr_unification" target="_blank">http://trac.osgeo.org/gdal/wiki/rfc46_gdal_ogr_unification</a><br>
><br>
<br>
Hi,<br>
<br>
The motion has been adopted with support from PSC members JukkaR, FrankW,<br>
DanielM, TamasS and myself.<br>
<br>
The code has also been merged in trunk now. Version number upgraded to 2.0dev.<br>
Full rebuild needed ! Let me know if issues arise.<br>
<br>
I've also done some unification of the online documentation on <a href="http://gdal.org" target="_blank">gdal.org</a> too, by<br>
merging the previous <a href="http://gdal.org/ogr/index.html" target="_blank">gdal.org/ogr/index.html</a> into the main<br>
<a href="http://gdal.org/index.html" target="_blank">gdal.org/index.html</a>. Basically the "make docs" target in GDAL root directory<br>
generate the whole doc. Those changes will be reflected on the server in a few<br>
hours. Still work to do to update the tutorials.<br>
I've installed in <a href="http://gdal.org/1.11/" target="_blank">http://gdal.org/1.11/</a> the snapshot of the 1.11<br>
documentation, that will be mentionned in <a href="http://gdal.org" target="_blank">gdal.org</a> main page, so that the<br>
trunk documentation can evolve more easily.<br>
<br>
Question : I see that in <a href="http://gdal.org/credits.html" target="_blank">http://gdal.org/credits.html</a> we still have mentions<br>
to past sponsors, but the sponsorship program is more or less abandonned, so I<br>
think we could remove the Sponsorship section, and make sure that those past<br>
sponsors are just mentionned in the "Corporate" section.<br>
<br>
Best regards,<br>
<br>
Even<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Geospatial professional services<br>
<a href="http://even.rouault.free.fr/services.html" target="_blank">http://even.rouault.free.fr/services.html</a><br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
</font></span></blockquote></div><br></div>