Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F432
bug3248.diff
Public
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Authored By
xplanes
Jan 21 2010, 9:58 AM
2010-01-21 09:58:49 (UTC+1)
Size
5 KB
Referenced Files
None
Subscribers
None
bug3248.diff
View Options
Index: Code/Interactions/mitkPointSetInteractor.cpp
===================================================================
--- Code/Interactions/mitkPointSetInteractor.cpp (revision 21081)
+++ Code/Interactions/mitkPointSetInteractor.cpp (working copy)
@@ -172,6 +172,7 @@
}
pointSet->ExecuteOperation( doOp );
+ delete doOp;
}
}
}
@@ -204,6 +205,7 @@
}
pointSet->ExecuteOperation( doOp );
+ delete doOp;
}
@@ -260,12 +262,14 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDYES, stateEvent->GetEvent());
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
}
else
{
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDNO, stateEvent->GetEvent());
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
}
ok = true;
break;
@@ -333,6 +337,7 @@
//execute the Operation
pointSet->ExecuteOperation(doOp);
+ delete doOp;
//the point is added and directly selected in PintSet. So no need to call OpSELECTPOINT
@@ -402,6 +407,7 @@
//here no undo is stored, because the movement-steps aren't interesting.
// only the start and the end is interisting to store for undo.
pointSet->ExecuteOperation(doOp);
+ delete doOp;
}
++it;
}
@@ -447,6 +453,7 @@
}
//execute the Operation
pointSet->ExecuteOperation(doOp);
+ delete doOp;
/*now select the point "position-1",
and if it is the first in list,
@@ -509,6 +516,7 @@
m_UndoController->SetOperationEvent(operationEvent);
}
pointSet->ExecuteOperation(doOp);
+ delete doOp;
//after delete the iterator is undefined, so start again
//count to the last existing entry
@@ -599,6 +607,7 @@
new mitk::PointOperation( OpREMOVE, timeInMS, pt, position );
++it;
pointSet->ExecuteOperation( doOp );
+ delete doOp;
}
else it++;
}
@@ -635,6 +644,7 @@
new mitk::StateEvent(EIDYES, newPosEvent);
//call HandleEvent to leave the guard-state
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
}
else
@@ -642,6 +652,7 @@
//new Event with information NO
mitk::StateEvent* newStateEvent = new mitk::StateEvent(EIDNO, posEvent);
this->HandleEvent(newStateEvent );
+ delete newStateEvent;
ok = true;
}
}
@@ -656,6 +667,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDNO, disPosEvent);
this->HandleEvent(newStateEvent );
+ delete newStateEvent;
ok = true;
}
}
@@ -696,6 +708,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent( EIDYES, newPosEvent );
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
//saving the spot for calculating the direction vector in moving
@@ -707,6 +720,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent( EIDNO, newPosEvent );
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
}
}
@@ -736,6 +750,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDSTSMALERNMINUS1, stateEvent->GetEvent());
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
}
else
@@ -746,6 +761,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDSTSMALERNMINUS1, stateEvent->GetEvent());
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
}
else
@@ -754,6 +770,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDSTLARGERNMINUS1, stateEvent->GetEvent());
this->HandleEvent(newStateEvent );
+ delete newStateEvent;
ok = true;
}//else
}//else
@@ -768,6 +785,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDYES, stateEvent->GetEvent());
this->HandleEvent( newStateEvent );
+ delete newStateEvent;
ok = true;
}
else //more than 1 points in list, so stay in the state!
@@ -775,6 +793,7 @@
mitk::StateEvent* newStateEvent =
new mitk::StateEvent(EIDNO, stateEvent->GetEvent());
this->HandleEvent(newStateEvent );
+ delete newStateEvent;
ok = true;
}
}
@@ -814,6 +833,7 @@
//execute the Operation
pointSet->ExecuteOperation(doOp);
+ delete doOp;
ok = true;
}
@@ -851,6 +871,7 @@
}
//execute the Operation
pointSet->ExecuteOperation(doOp);
+ delete doOp;
ok = true;
}
@@ -915,6 +936,7 @@
}
//execute the Operation
pointSet->ExecuteOperation(doOp);
+ delete doOp;
}
++it;
}
@@ -995,6 +1017,7 @@
++it;
pointSet->ExecuteOperation( doOp );
+ delete doOp;
}
//reset the statemachine
File Metadata
Details
Attached
Mime Type
text/plain
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
415
Default Alt Text
bug3248.diff (5 KB)
Attached To
Mode
T3248: Memory leaks when calling new mitk::PointOperation
Attached
Detach File
Event Timeline
xplanes
added a comment.
Jan 21 2010, 9:58 AM
2010-01-21 09:58:49 (UTC+1)
Comment Actions
Unified patch
Log In to Comment