<div dir="ltr"><div class="gmail_extra">As a follow up, here is some vtkpython code that generates a spurious error:</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_extra">a = vtkVariantArray()</div><div class="gmail_extra">a.SetNumberOfValues(1)</div><div class="gmail_extra">a.SetNumberOfValues(0)</div><div class="gmail_extra">a.Squeeze()</div><div class="gmail_extra">ERROR: In Common/Core/vtkVariantArray.cxx, line 730</div><div class="gmail_extra">vtkVariantArray (0x7fe1bbca8cb0): Memory size must be positive</div><div class="gmail_extra"><br></div><div class="gmail_quote">On Wed, Feb 25, 2015 at 10:42 AM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">It looks like the added vtkErrorMacro() will be called whenever Squeeze()<div>is called on an empty array, which is a common occurrence and should</div><div>definitely not generate an error.</div><div><br></div><div>I think the patch should be reverted (though I was the reviewer... oops!).</div><span class=""><font color="#888888"><div><br></div><div> - David</div></font></span><div><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Wed, Feb 25, 2015 at 9:59 AM, David E DeMarle <span dir="ltr"><<a href="mailto:dave.demarle@kitware.com" target="_blank">dave.demarle@kitware.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><span class=""><div>Keep us informed. If it needs fixing we're still making fixes and taking patches based off anywhere upstream of the release branch point and tested in master.<br></div><div><br></div><div>We'll make a decision on turning the release branch into either rc2 or final on Monday some time.</div></span><div class="gmail_extra"><span><div></div>
<br></span><span><div class="gmail_quote"><span class="">On Wed, Feb 25, 2015 at 11:52 AM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:</span><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>
<br>
</span>No serious issues found.<br>
<br>
Maybe found a mistake I made though:<br>
<br>
<<a href="http://review.source.kitware.com/#/c/18331/1//COMMIT_MSG" target="_blank">http://review.source.kitware.com/#/c/18331/1//COMMIT_MSG</a>><br>
<br>
I think some of the vtkErrorMacros I added should maybe not be there, as I see them running my app where I don't believe I saw them before.  Then again, they are maybe flagging a real issue that was not flagged before.  Haven't had time to dig...</blockquote></span></div></span></div></div></blockquote></div></div></div></div>
</blockquote></div><br></div></div>