[vtk-developers] announce: vtk 6.2.0 release candidate 1 is ready

David Gobbi david.gobbi at gmail.com
Wed Feb 25 13:15:15 EST 2015


As a follow up, here is some vtkpython code that generates a spurious error:

a = vtkVariantArray()
a.SetNumberOfValues(1)
a.SetNumberOfValues(0)
a.Squeeze()
ERROR: In Common/Core/vtkVariantArray.cxx, line 730
vtkVariantArray (0x7fe1bbca8cb0): Memory size must be positive

On Wed, Feb 25, 2015 at 10:42 AM, David Gobbi <david.gobbi at gmail.com> wrote:

> It looks like the added vtkErrorMacro() will be called whenever Squeeze()
> is called on an empty array, which is a common occurrence and should
> definitely not generate an error.
>
> I think the patch should be reverted (though I was the reviewer... oops!).
>
>  - David
>
>
> On Wed, Feb 25, 2015 at 9:59 AM, David E DeMarle <dave.demarle at kitware.com
> > wrote:
>
>> 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.
>>
>> We'll make a decision on turning the release branch into either rc2 or
>> final on Monday some time.
>>
>> On Wed, Feb 25, 2015 at 11:52 AM, Sean McBride <sean at rogue-research.com>
>> wrote:
>>>
>>>
>>> No serious issues found.
>>>
>>> Maybe found a mistake I made though:
>>>
>>> <http://review.source.kitware.com/#/c/18331/1//COMMIT_MSG>
>>>
>>> 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...
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20150225/6b23616c/attachment.html>


More information about the vtk-developers mailing list