Hi Rene-Luc<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 7, 2012 at 9:42 AM, rldhont <span dir="ltr"><<a href="mailto:rldhont@gmail.com" target="_blank">rldhont@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
As Martin Dobias asked me, I updated my code and implemented Spatial Operator in QgsExpression as NodeFunction.<br>
You can review the code here :<br>
<a href="https://github.com/rldhont/Quantum-GIS/tree/6535" target="_blank">https://github.com/rldhont/<u></u>Quantum-GIS/tree/6535</a><br>
<br></blockquote><div><br></div><div>Thanks for the update. I went through the code and things look good. I have only one concern: QgsGeometry instances stored in QVariant do not seem to be released... When "$geometry" is used, then it's returning the instance from the feature (which should not be deleted), but temporary geometries constructed from wkt/gml must be deleted.</div>
<div><br></div><div>Maybe we should store directly QgsGeometry object instead of just a pointer inside?</div><div><br></div><div>Regards</div><div>Martin</div><div><br></div></div></div>