Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some operations on Core Build Settings pages apply immediately instead of waiting until OK is pressed #1057

Open
jonahgraham opened this issue Jan 24, 2025 · 0 comments

Comments

@jonahgraham
Copy link
Member

  1. Open the CMake Build Settings tab
  2. Check (or uncheck) "Use these settings"
  3. Press Cancel
  4. Re-open CMake Build Settings tab

Expectation is that the change in 2. is not persisted.

The change is persisted because each modification in a launch tab calls updateLaunchConfigurationDialog (fairly standard thing to do, not core build specific).updateLaunchConfigurationDialog eventually calls ILaunchConfigurationDialog.updateButtons() which for the launch bar does this:

@Override
public void updateButtons() {
// Lots of tabs want to be applied when this is called
if (!initing) {
ILaunchConfigurationTab[] tabs = getTabs();
if (tabFolder != null && tabs != null) {
int pageIndex = tabFolder.getSelectionIndex();
if (pageIndex >= 0) {
tabs[pageIndex].performApply(workingCopy);
}
}
}
}

which causes apply to be pressed on every page.

This means that basically any edit in the launch configuration causes every page to apply its changes immediately instead of waiting for the user to press OK.

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

No branches or pull requests

1 participant