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

Add static method to the interface of ISharedImages to get the instance #2736

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Jan 19, 2025

Currently one needs to first obtain the PaltformUI Workbench and then call getSharedImages on this instance. As this actually maps down to WorkBenchPlugin that do not require a running workbench this limits the usage of such code only wanting to share images.

This adds a new ISharedImages.get() that directly calls the code the IWorkbench implementation would call and replace static calls to the workbench with this one.

See for example:

Currently one needs to first obtain the PaltformUI Workbench and then
call getSharedImages on this instance. As this actually maps down to
WorkBenchPlugin that do not require a running workbench this limits the
usage of such code only wanting to share images.

This adds a new ISharedImages.get() that directly calls the code the
IWorkbench implementation would call and replace static calls to the
workbench with this one.
@laeubi laeubi requested a review from akurtakov January 19, 2025 19:20
Copy link
Contributor

Test Results

 1 818 files  ±0   1 818 suites  ±0   1h 28m 19s ⏱️ + 1m 33s
 7 715 tests ±0   7 487 ✅ ±0  228 💤 ±0  0 ❌ ±0 
24 306 runs  ±0  23 557 ✅ ±0  749 💤 ±0  0 ❌ ±0 

Results for commit c87d92c. ± Comparison against base commit aff1ccc.

Copy link
Member

@akurtakov akurtakov left a comment

Choose a reason for hiding this comment

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

Looks good to me to have more direct access.

@laeubi laeubi merged commit 8029b47 into eclipse-platform:master Jan 20, 2025
17 checks passed
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