Use GDB as default debugger backend
ClosedPublic
Actions

Authored by qi437103 on Sep 5 2016, 5:18 PM.

Details

Reviewers

kfunk

Commits

Summary

As discussed in IRC, make sure GDB is the default debugger backend when creating new launch profile.

Before this patch, it defaults to whatever comes first in the registered list.

This may not be the cleanest way to go. But adding new interfaces for setting default values from plugin side seems overkill.

Repository

R33 KDevPlatform

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

qi437103 retitled this revision from to Use GDB as default debugger backend.

qi437103 updated this object.

qi437103 edited the test plan for this revision. (Show Details)

qi437103 added a reviewer: kfunk.

Use a different way to implement this.
The first version is flawed in that it'll overwrite user preference,
and it doesn't handle missing GDB plugin.

Well, that's the best we can get at the moment. +1.

Thanks!

This revision is now accepted and ready to land.Sep 5 2016, 10:35 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			shell/launchconfiguration.cpp (13 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	6454	dce32ca		Sep 5 2016, 5:18 PM	★	★
Diff 2	6456	dce32ca	Use a different way to implement this.	Sep 5 2016, 7:56 PM	★	★
Diff 3	6457	dfa608a	rKDEVPLATFORM2fc1720781e8744c9df4a44cd943322889a55300	Sep 6 2016, 12:35 AM	★	★