Commit 8c169aa0 authored by Sankhesh Jhaveri's avatar Sankhesh Jhaveri

Prevent the seed widget from removing handles twice

The DeleteAction calls RemoveHandle and DeleteSeed whereas DeleteSeed,
in turn, calls RemoveHandle. This change ensures that the handles are
removed only once.

Removed the unnecessary duplicate setting of HandleSize in
vtkWidgetRepresentation.
parent 2fb678e1
......@@ -314,7 +314,6 @@ void vtkSeedWidget::DeleteAction(vtkAbstractWidget *w)
// Invoke event for seed handle before actually deleting
self->InvokeEvent(vtkCommand::DeletePointEvent, &(removeId));
rep->RemoveHandle(removeId);
self->DeleteSeed(removeId);
// Got this event, abort processing if it
self->EventCallbackCommand->SetAbortFlag(1);
......
......@@ -36,7 +36,6 @@ vtkWidgetRepresentation::vtkWidgetRepresentation()
this->PlaceFactor = 0.5;
this->Placed = 0;
this->HandleSize = 0.05;
this->ValidPick = 0;
this->HandleSize = 0.01;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment