<div dir="ltr"><div>Hi Paul,<br><br>I was part of that fix and I think it's correct. There may have been more to that fix though since it took a couple of shots before getting it correct so the git history may not be as pristine as desired. If you can give me more specifics on the bug though I can try to see what's going on with it. If I remember correctly though, that fix had to do with the fact that some places thought that the order of iterating through cells in a vtkPolyData was first through all vertex cells, then all line cells, then all polys and finally through all triangle strips, irregardless of the order that cells got inserted in (i.e. using InsertNextCell()). The reality is that for polydata that cells are indeed iterated through in their insertion order. The test that was added for those changes hopefully shows the proper functionality along with associated cell data manipulations as well.<br><br></div>Regards,<br>Andy<br><div><br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 13, 2014 at 1:55 PM, Paul Edwards <span dir="ltr"><<a href="mailto:paul.m.edwards@gmail.com" target="_blank">paul.m.edwards@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,<br>
<br>
I've just noticed an issue with vtkPolyData.  I had been calling<br>
vtkPolyData::DeleteCell to remove the unwanted cells, followed by a<br>
call to vtkPolyData::RemoveDeletedCells.  The result is not as it<br>
should be in the latest version.  To fix I just started reverting<br>
previous commits.  I tracked it down to the following:<br>
<br>
bugfix 14459: Iterating through polydata cells incorrectly.<br>
commit: 44131c6d05375692af8b6b38cf4291983bdd8f45<br>
<br>
Sorry but I do not have time to provide a test case or track where in<br>
this commit lies the problem.  I just thought I would give details<br>
here in case anyone else notices the same problem and is searching for<br>
a solution.<br>
<br>
Best regards,<br>
Paul<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the VTK FAQ at: <a href="http://www.vtk.org/Wiki/VTK_FAQ" target="_blank">http://www.vtk.org/Wiki/VTK_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtkusers" target="_blank">http://public.kitware.com/mailman/listinfo/vtkusers</a><br>
</blockquote></div><br></div>