<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 17, 2014 at 3:01 PM, 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"><div><div>> I suppose, but the problem is that without it the whole thing becomes<br>
> useless for large datasets (> 100k features) due to GME restrictions. I<br>
> just thought that it would be good to have a GME driver that works on those<br>
> large tables too. :)<br>
<br>
</div></div>The problem is that at some point we must take a snapshot and say "hey, this is<br>
GDAL 1.11, the latest and greatest, use it please". I think it is OK if new<br>
drivers are still a bit experimental.<br>
<br></blockquote><div><br></div><div>Ok, no need for it then. Do you want me to roll my changes back from the 1.11 branch or apply my patch to it?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




Reviewing the commit, I think that it has at least one issue because<br>
SetSpatialFilter() will segfault in switch( poGeomIn->getGeometryType() ) if<br>
passed a NULL geometry, which is perfectly legal, in order to uninstall a<br>
spatial filter.<br>
Did you test the driver with the test_ogrsf utility ? (cd apps; make test_ogrsf)<br>
<br></blockquote><div><br></div><div>Noo :( I was too tired and forgot. I only ran my own python test which doesn't try to unset the spatial filter. Will need to add that test later. Anyways just committed a fix for the crash and now test_ogrsf works mostly. A few errors, but nothing that causes a crash. They are mostly GME -related things, like the it doesn't support infinity and big numbers. So will add a check for that later.</div>

<div><br></div><div>--Wolf</div><div><br></div>

</div></div></div>