Skip to content

Prevent NPE in LaunchConfigurationTabGroupViewer.handleApplyPressed#2572

Open
SougandhS wants to merge 1 commit intoeclipse-platform:masterfrom
SougandhS:Null_check_LaunchCOnf
Open

Prevent NPE in LaunchConfigurationTabGroupViewer.handleApplyPressed#2572
SougandhS wants to merge 1 commit intoeclipse-platform:masterfrom
SougandhS:Null_check_LaunchCOnf

Conversation

@SougandhS
Copy link
Contributor

Add a null check for fOriginal in
LaunchConfigurationTabGroupViewer.handleApplyPressed to prevent a NullPointerException when switching between launch configurations.

Fixes : #2571

@github-actions
Copy link
Contributor

github-actions bot commented Mar 15, 2026

Test Results

    54 files  ±0      54 suites  ±0   38m 12s ⏱️ + 4m 10s
 4 546 tests ±0   4 523 ✅ ±0   23 💤 ±0  0 ❌ ±0 
12 234 runs  ±0  12 075 ✅ ±0  159 💤 ±0  0 ❌ ±0 

Results for commit ffa3b80. ± Comparison against base commit 4d46f12.

♻️ This comment has been updated with latest results.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Prevents a sporadic NullPointerException in the Debug UI launch configuration dialog when rapidly switching/toggling configurations, by guarding against a transient null original configuration during Apply handling (Fixes #2571).

Changes:

  • Add a null check for fOriginal in LaunchConfigurationTabGroupViewer.handleApplyPressed() before calling getWorkingCopy().
  • Update the file copyright year.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add a null check for fOriginal in
LaunchConfigurationTabGroupViewer.handleApplyPressed to prevent a
NullPointerException when switching between launch configurations.

Fixes : eclipse-platform#2571
@SougandhS SougandhS force-pushed the Null_check_LaunchCOnf branch from 9e60d4c to ffa3b80 Compare March 16, 2026 01:38
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.

NPE in LaunchConfigurationTabGroupViewer.handleApplyPressed

2 participants