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

Remove unused leftovers of WPF launcher in JNIBridge #624

Merged

Conversation

HannesWell
Copy link
Member

@HannesWell HannesWell commented May 9, 2024

For the equinox launcher for Windows WPF launcher the native methods OleInitialize() and OleUninitialize() where introduced that where only implemented in the native library available for Windows WPF, see Bug 172367 / 3cf8518. But during the removal of the WPF launcher in Bug 479180 / 121b7c5 these methods have not been removed.

Since the native implementation is not available anymore these native methods cannot be used.

Copy link

github-actions bot commented May 9, 2024

Test Results

  660 files  ±0    660 suites  ±0   1h 12m 44s ⏱️ -7s
2 195 tests ±0  2 148 ✅ ±0   47 💤 ±0  0 ❌ ±0 
6 729 runs  ±0  6 586 ✅ ±0  143 💤 ±0  0 ❌ ±0 

Results for commit 6001d53. ± Comparison against base commit fadf6d1.

♻️ This comment has been updated with latest results.

For the equinox launcher for Windows WPF the native methods
OleInitialize() and OleUninitialize() where introduced and only
implemented in the native library available for Windows WPF [1].
But during the removal of the WPF launcher [2] these methods have not
been removed.

Since the native implementation is not available anymore these native
methods cannot be used.

[1] - https://bugs.eclipse.org/bugs/show_bug.cgi?id=172367
[2] - https://bugs.eclipse.org/bugs/show_bug.cgi?id=479180
@HannesWell HannesWell force-pushed the remove-OLEInitialization branch from e21d483 to 6001d53 Compare May 13, 2024 16:45
@HannesWell HannesWell merged commit 5bed96a into eclipse-equinox:master May 13, 2024
26 of 27 checks passed
@HannesWell HannesWell deleted the remove-OLEInitialization branch May 13, 2024 17:50
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