Skip to content

Dispose the images of the e4 model editor dialogs - #2548

Open
vogella wants to merge 1 commit into
eclipse-pde:masterfrom
vogella:vogella/e4-leaks-dialog-images
Open

vogella wants to merge 1 commit into
eclipse-pde:masterfrom
vogella:vogella/e4-leaks-dialog-images

Conversation

@vogella

@vogella vogella commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

The URI dialog created its toolbar images on every open, and the project folder page created a new image for every tree item it labelled; neither was ever disposed, and both now go through a resource manager that owns them. BundleImageCache kept its placeholder image in a static field, so disposing one cache broke the placeholder every other open cache was still showing; each cache now has its own. ContributionDataFile gets equals and hashCode, so the icon search dialog's image cache stops reloading icons on every repaint.

@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Test Results

  144 files  + 6    144 suites  +6   37m 11s ⏱️ + 1m 14s
3 605 tests + 4  3 551 ✅ + 4   54 💤 ±0  0 ❌ ±0 
9 564 runs  +12  9 434 ✅ +12  130 💤 ±0  0 ❌ ±0 

Results for commit 6a0eda7. ± Comparison against base commit 7558f74.

♻️ This comment has been updated with latest results.

The URI dialog created its two toolbar images on every open and the
project folder page created a new image for every tree item it labelled,
and none of them was disposed; both now go through a resource manager
owned by the toolbar or the label provider. BundleImageCache kept its
placeholder image in a static field, so disposing one cache disposed the
placeholder still shown by every other one; each cache now owns its own.
ContributionDataFile gets equals and hashCode, so the icon search
dialog's image cache finds the images it already loaded instead of
loading them again on every repaint.

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@vogella
vogella force-pushed the vogella/e4-leaks-dialog-images branch from 13aebaa to 6a0eda7 Compare October 10, 2026 10:13
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