Merge lp:~jocke-karlsson/forssim/reset_drillspeed into lp:forssim

Proposed by jocke
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jocke-karlsson/forssim/reset_drillspeed
Merge into: lp:forssim
Diff against target: None lines
To merge this branch: bzr merge lp:~jocke-karlsson/forssim/reset_drillspeed
Reviewer Review Type Date Requested Status
Ulrika Dreifaldt Needs Fixing
Bo Rasmusson (community) Approve
Review via email: mp+12279@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Bo Rasmusson (bo-rasmusson) wrote :

Ok!

review: Approve
Revision history for this message
Ulrika Dreifaldt (ulrika-dreifaldt) wrote :

Looks good

review: Approve
Revision history for this message
Ulrika Dreifaldt (ulrika-dreifaldt) wrote :

Needs to be tested as well

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'FsWisdom/RemoveWisdomToothWindow.cpp'
2--- FsWisdom/RemoveWisdomToothWindow.cpp 2009-09-17 10:00:49 +0000
3+++ FsWisdom/RemoveWisdomToothWindow.cpp 2009-09-23 12:45:13 +0000
4@@ -345,6 +345,11 @@
5 ui.rotationMinusZ_2->setEnabled(false);
6 ui.progressBar_2->setValue(0);
7
8+ //Need to reset drillspeed. For some reason it will still have a value > 0 if starting
9+ //a case after exiting a case with the drill on. And it is not enough to reset the field
10+ //in ADrillableNode.cpp for some reason. Is the python-classes not reseted on exit?
11+ aDrillableNode->drillSpeed.get()->setValue(0);
12+
13 //Update ui according to state
14 toggleShowHideHead();
15 toggleShowHideTeeth();

Subscribers

People subscribed via source and target branches

to all changes: