Don't think it matters but it seems source/blender/ is missing from the paths in the patch I sent.<div>I must have been in blender/source/blender when I made it or something?</div><div>Either way all my changes were in that folder so if you successfully patched already then this</div>
<div>is the very same.</div><div><br></div><div><br></div><div><br><br><div class="gmail_quote">On Tue, Mar 15, 2011 at 9:20 AM, Morten Mikkelsen <span dir="ltr"><<a href="mailto:mikkelsen7@gmail.com">mikkelsen7@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Having trouble figuring out where, in the tool, I can add the alternative patch.<div>So I am e-mailing it instead.</div>
<div><div></div><div class="h5"><br><div class="gmail_quote">On Tue, Mar 15, 2011 at 8:22 AM, Morten Mikkelsen <span dir="ltr"><<a href="mailto:mikkelsen7@gmail.com" target="_blank">mikkelsen7@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'll make a version of the patch without any code sharing instead.<div><div></div><div><div><br></div><div>
<br><br><div class="gmail_quote">On Tue, Mar 15, 2011 at 8:02 AM, <span dir="ltr"><<a href="mailto:brechtvanlommel@gmail.com" target="_blank">brechtvanlommel@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 2011/03/15 14:51:05, mikkelsen7 wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
But it has to iterate over the faces twice.<br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Removing the iteration from the abstraction would conflict with that.<br>
</blockquote>
<br></div>
Is the second loop really necessary, those normals should never be used?<br>
Maybe for export.. If it is necessary, just add two loops then.<div><div></div><div><br>
<br>
<a href="http://codereview.appspot.com/4280049/" target="_blank">http://codereview.appspot.com/4280049/</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br>
</div></div></blockquote></div><br></div>