(0001612)
Brad King (manager)
2004-10-13 08:38

The scalars array is shared amongst all outputs, but in a certain case the reader was calling scalars->Delete() once for each output even though it did only one ::New() for it. This means for case files with more than one output the scalars array is deleted too many times when the reader is destroyed.

I've committed a fix:

/cvsroot/ParaView/ParaView/VTK/IO/vtkEnSight6BinaryReader.cxx,v <-- vtkEnSight6BinaryReader.cxx
new revision: 1.45; previous revision: 1.44
/cvsroot/ParaView/ParaView/VTK/IO/vtkEnSight6Reader.cxx,v <-- vtkEnSight6Reader.cxx
new revision: 1.51; previous revision: 1.50

This may also address bug 0000394.

Mathieu, please confirm that this fixes the problem for all the case files that produced it and then close the bug.

(0001632)
Mathieu Malaterre (reporter)
2004-10-22 15:14

All the EnSight files are now being properly read. Lee if you can, could you double-check ? You need ParaView from CVS.

(0001812)
Ken Martin (reporter)
2004-12-15 08:53

Closed. As far as we know the issues raised by this bug have been addressed. Also bug id 1400 1401 and 1403 have helped with this. If more issues arice they will have seperate bug ids.

Issue History
Date Modified	Username	Field	Change

Notes
(0001612) Brad King (manager) 2004-10-13 08:38	The scalars array is shared amongst all outputs, but in a certain case the reader was calling scalars->Delete() once for each output even though it did only one ::New() for it. This means for case files with more than one output the scalars array is deleted too many times when the reader is destroyed. I've committed a fix: /cvsroot/ParaView/ParaView/VTK/IO/vtkEnSight6BinaryReader.cxx,v <-- vtkEnSight6BinaryReader.cxx new revision: 1.45; previous revision: 1.44 /cvsroot/ParaView/ParaView/VTK/IO/vtkEnSight6Reader.cxx,v <-- vtkEnSight6Reader.cxx new revision: 1.51; previous revision: 1.50 This may also address bug 0000394. Mathieu, please confirm that this fixes the problem for all the case files that produced it and then close the bug.

(0001632) Mathieu Malaterre (reporter) 2004-10-22 15:14	All the EnSight files are now being properly read. Lee if you can, could you double-check ? You need ParaView from CVS.

(0001812) Ken Martin (reporter) 2004-12-15 08:53	Closed. As far as we know the issues raised by this bug have been addressed. Also bug id 1400 1401 and 1403 have helped with this. If more issues arice they will have seperate bug ids.

Relationships