<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Hi Martin,<br>
<br>
I made the update. QgsGeometry object is directly stored.<br>
<br>
I also updated the list of NodeFunction :<br>
* buffer<br>
* intersection<br>
* distance<br>
* exportToWKT<br>
* and other geometruc method<br>
I also add test.<br>
<br>
Regards,<br>
René-Luc<br>
<br>
Le 12/11/2012 10:39, Martin Dobias a écrit :<br>
</div>
<blockquote
cite="mid:CAC2XbFeVnW+W1Ore_qMGU7t+5nbfg-Hzmct3YHm1ZiZXveDwow@mail.gmail.com"
type="cite">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 moz-do-not-send="true"
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 moz-do-not-send="true"
href="https://github.com/rldhont/Quantum-GIS/tree/6535"
target="_blank">https://github.com/rldhont/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>
</blockquote>
<br>
</body>
</html>