This is necessary because without that currentTargetAction will have actions which points to already deleted launch configuration which leads to crash.
Details
Details
Tests passed. Everything works as expected.
Diff Detail
Diff Detail
- Repository
- R33 KDevPlatform
- Branch
- fix_crash
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
There's RunController::removeLaunchConfiguration -- can't we reuse that?
I don't know a lot about that code either, to be fair...
Comment Actions
Ah, I think I see. RunController::removeLaunchConfiguration also removes things from the config; we probably don't want that when RunController::removeConfigurationType was called.
Please introduce a RunController::removeLaunchConfigurationInternal (which does *not* touch the config) which is called from RunController::removeLaunchConfiguration