Skip to content

Added API allows ISV to specify preferred launch mode#1

Closed
DangMinhTam382 wants to merge 1 commit intomainfrom
Add_API_to_allow_set_default_launch_mode
Closed

Added API allows ISV to specify preferred launch mode#1
DangMinhTam382 wants to merge 1 commit intomainfrom
Add_API_to_allow_set_default_launch_mode

Conversation

@DangMinhTam382
Copy link
Copy Markdown
Owner

Added API to ILaunchConfigurationProvider that provide preferred launch mode ID for specified launch descriptor and launch target. Preferred mode is considered when LaunchBarManger call syncActiveMode(). Preferred mode comes after storedMode, lastActiveMode and comes before hard-coded run, debug, first supported mode.

@DangMinhTam382 DangMinhTam382 marked this pull request as draft April 3, 2026 07:42
@DangMinhTam382 DangMinhTam382 self-assigned this Apr 3, 2026
@DangMinhTam382 DangMinhTam382 added enhancement New feature or request and removed enhancement New feature or request labels Apr 3, 2026
Added API to ILaunchConfigurationProvider that provide preferred launch
mode ID for specified launch descriptor and launch target.
Preferred mode is considered when LaunchBarManger call syncActiveMode().
Preferred mode comes after storedMode, lastActiveMode and comes before
hard-coded run, debug, first supported mode.
@DangMinhTam382 DangMinhTam382 force-pushed the Add_API_to_allow_set_default_launch_mode branch from beb4c52 to 21f1d9f Compare April 3, 2026 12:02
@DangMinhTam382 DangMinhTam382 marked this pull request as ready for review April 6, 2026 03:38
Copy link
Copy Markdown
Collaborator

@betamaxbandit betamaxbandit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @DangMinhTam382 ,
thanks for the PR.
LGTM.
I think this PR can be published now.
👍

@DangMinhTam382
Copy link
Copy Markdown
Owner Author

Thanks for review this change, Mr. @betamaxbandit .
PR made official: eclipse-cdt#1443

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants