<div dir="ltr">Damn,<div>you are right,</div><div>even with grossly incorrect edge-linking passes the test.</div><div><br></div><div>I visually checked, result looks good, but with topology it is hard to be sure.</div><div>Cheers,</div><div>Rémi-C</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-02-17 15:34 GMT+01:00 Sandro Santilli <span dir="ltr"><<a href="mailto:strk@keybit.net" target="_blank">strk@keybit.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Feb 17, 2015 at 03:03:58PM +0100, Rémi Cura wrote:<br>
> Ok the edge-linking update function seems to work<br>
> (tested on a 40 edge geom:<br>
> inserting node, then inserting edge without setting next_left and next_right<br>
> then using function to straighten things up<br>
> Afterward, topology passes ValidateTopology)<br>
<br>
</span>I don't think ValidateTopology really checks edge-linking.<br>
Did you check calling it before fixing that ?<br>
Chances are it's indirectly checked by means of face geometry construction<br>
(now that it makes use of that info).<br>
<br>
--strk;<br>
</blockquote></div><br></div>