Skip to content

Fix potential NPE in EnumPropertyEditor#1382

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:npe-in-property-editor
Feb 13, 2026
Merged

Fix potential NPE in EnumPropertyEditor#1382
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:npe-in-property-editor

Conversation

@ptziegler
Copy link
Contributor

When an unknown property is selected, a null text is returned. Because the SWT combo doesn't support null strings, an exception might be thrown. Instead, an empty string has to be returned.

Follow-up on 6ee0cd8

@github-actions
Copy link

github-actions bot commented Feb 13, 2026

Test Results

 2 720 files  ±0   2 720 suites  ±0   12h 50m 56s ⏱️ + 6m 30s
 7 018 tests ±0   6 960 ✅ ±0   58 💤 ±0  0 ❌ ±0 
28 072 runs  ±0  27 830 ✅ ±0  242 💤 ±0  0 ❌ ±0 

Results for commit ecfe2c4. ± Comparison against base commit 11c9cfb.

♻️ This comment has been updated with latest results.

When an unknown property is selected, a `null` text is returned. Because
the SWT combo doesn't support `null` strings, an exception might be
thrown. Instead, an empty string has to be returned.

Follow-up on 6ee0cd8
@ptziegler ptziegler force-pushed the npe-in-property-editor branch from c538e87 to ecfe2c4 Compare February 13, 2026 20:09
@ptziegler ptziegler merged commit 8d9b8bb into eclipse-windowbuilder:master Feb 13, 2026
8 checks passed
@ptziegler ptziegler deleted the npe-in-property-editor branch February 13, 2026 21:20
@ptziegler ptziegler added this to the 1.23.0 milestone Feb 13, 2026
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.

1 participant

Comments