Skip to content

fix CodeMiningLineHeaderAnnotation#getMultilineHeight #2787

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

Merged

Conversation

tobiasmelcher
Copy link
Contributor

it should never return 0. Fix race condition when code minings are not yet available.
#2786

@iloveeclipse
Copy link
Member

This PR seem to reduce redraw count, see also #2788 where you can count them with tracing option

@iloveeclipse
Copy link
Member

Please rebase on master

@tobiasmelcher tobiasmelcher force-pushed the editor_content_jumps branch 2 times, most recently from ac4f037 to edf3294 Compare February 10, 2025 17:12
it should never return 0. Fix race condition when code minings are not
yet available.
eclipse-platform#2786
Copy link
Contributor

Test Results

 1 821 files  ±0   1 821 suites  ±0   1h 34m 37s ⏱️ + 6m 44s
 7 724 tests +1   7 495 ✅ ±0  228 💤 ±0  1 ❌ +1 
24 333 runs  +3  23 583 ✅ +2  749 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit b506f3d. ± Comparison against base commit c9f129d.

@BeckerWdf
Copy link
Contributor

testSingleSelectionAndTwoInitialSelectionsWithInitialPattern (UiTestSuite org.eclipse.ui.tests.dialogs.FilteredResourcesSelectionDialogTestSuite org.eclipse.ui.tests.dialogs.ResourceInitialSelectionTest)

looks anrelated

@iloveeclipse
Copy link
Member

@tobiasmelcher : I'm not familiar with the code, so not right person for a review. @mickaelistria maybe?

@BeckerWdf BeckerWdf requested review from mickaelistria and removed request for iloveeclipse February 11, 2025 09:34
@mickaelistria mickaelistria merged commit 433cdb9 into eclipse-platform:master Feb 11, 2025
15 of 17 checks passed
@BeckerWdf BeckerWdf added this to the 4.35 M3 milestone Feb 11, 2025
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.

5 participants