View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015533VTK(No Category)public2015-06-11 12:452015-08-07 08:20
ReporterCory Quammen 
Assigned ToT.J. Corona 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version6.3.0 
Target VersionFixed in Version6.3.0 
Summary0015533: vtkAbstractArray::Reset() does not really reset arrays
DescriptionThe Reset() method does not reset all array state. For instance, in vtkVariantArray, the lookup structure is not reset. This leads to unexpected behavior. This example shows the problem.

vtkVariantArray* array = vtkVariantArray::New()
array->SetNumberOfTuples(5);
array->InsertNextTuple(value);
...

vtkIdType idx = array->LookupValue(value); // idx is > -1

array->Reset();
idx = array->LookupValue(value); // idx should be -1, but is larger than -1

Reset() should be made virtual to give subclasses a chance to override and reset any additional state they define. A pass through VTK data arrays should be made to verify that all state is reset when Reset() is called.
TagsNo tags attached.
ProjectTBD
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0034915)
T.J. Corona (developer)
2015-08-06 18:28

This issue has been fixed.
(0034919)
Cory Quammen (developer)
2015-08-06 22:22

T.J.

Could you copy and paste the git log entry for the commit that fixes the issue here?

Thanks,
Cory
(0034922)
T.J. Corona (developer)
2015-08-07 08:20

Hi Cory,

The commit message can be found here:

https://gitlab.kitware.com/vtk/vtk/commit/ac031ed42a939f8f79f827def05a96995068cb62 [^]

-T.J.

 Issue History
Date Modified Username Field Change
2015-06-11 12:45 Cory Quammen New Issue
2015-08-05 10:21 Utkarsh Ayachit Assigned To => T.J. Corona
2015-08-06 18:28 T.J. Corona Note Added: 0034915
2015-08-06 18:28 T.J. Corona Status backlog => closed
2015-08-06 18:28 T.J. Corona Resolution open => fixed
2015-08-06 18:28 T.J. Corona Fixed in Version => 6.3.0
2015-08-06 22:22 Cory Quammen Note Added: 0034919
2015-08-07 08:20 T.J. Corona Note Added: 0034922


Copyright © 2000 - 2018 MantisBT Team