<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><div><br></div><div> - David</div><div><br><div class="gmail_extra"><br><div class="gmail_quote">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><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><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><div class="gmail_extra"><span class=""><div></div>
<br></span><span class=""><div class="gmail_quote">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:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc 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></div></span></div></div></blockquote></div></div></div></div>