MantisBT - VTK | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0014034 | VTK | (No Category) | public | 2013-04-25 04:30 | 2013-10-24 11:58 |
| Reporter | santosh | ||||
| Assigned To | Dan Lipsa | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | unable to reproduce | ||
| Platform | OS | OS Version | |||
| Product Version | 5.10.0 | ||||
| Target Version | Fixed in Version | ||||
| Project | TBD | ||||
| Type | incorrect functionality | ||||
| Summary | 0014034: Bug in vtkCellArray::WritePointer function | ||||
| Description | In vtkCellArray::WritePointer function the InsertLocation member variable is becoming zero. After using WritePointer function if user use vtkCellArray::GetInsertLocation, it is giving incorrect location as InsertLocation is not getting updated. I got this issue when I was using output of vtkAppendPolyData. I am inserting new linked polys using vtkPolyData::InsertNextLinkedCell. While using this API the cells are not getting updated correctly and because of which I was getting issues in my algorithm which is based on topological information. I have modified function vtkCellArray::WritePointer and everything is working fine. Following is the modified code of the function inline vtkIdType *vtkCellArray::WritePointer(const vtkIdType ncells, const vtkIdType size) { this->NumberOfCells = ncells; this->TraversalLocation = 0; vtkIdType* pIdArray = this->Ia->WritePointer(0,size); this->InsertLocation = this->Ia->GetMaxId() + 1; return pIdArray; } | ||||
| Steps To Reproduce | |||||
| Additional Information | Indeed vtkCellArray::WritePointer resets the InsertLocation member variable. While this decision can be questioned, it is not necessary a bug. Changing this behavior, might cause problems for other code that depends on it. Please provide us with more information if you have a test case where this causes problems in VTK. Note that you can use vtkPolyData::InsertNextCell (without calling WritePointer) to insert cells beyond the original number of allocated cells as this takes care of the reallocation and also maintains the InsertLocation variable correctly. | ||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2013-04-25 04:30 | santosh | New Issue | |||
| 2013-06-25 12:08 | Dave DeMarle | Note Added: 0031049 | |||
| 2013-10-21 13:39 | Dan Lipsa | Assigned To | => Dan Lipsa | ||
| 2013-10-21 13:39 | Dan Lipsa | Status | backlog => tabled | ||
| 2013-10-21 14:18 | Dave DeMarle | Status | tabled => backlog | ||
| 2013-10-24 11:58 | Dan Lipsa | Status | backlog => closed | ||
| 2013-10-24 11:58 | Dan Lipsa | Resolution | open => unable to reproduce | ||
| 2013-10-24 11:58 | Dan Lipsa | Additional Information Updated | bug_revision_view_page.php?rev_id=662#r662 | ||
| Notes | |||||
|
|
|||||
|
|
||||