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

Unify internal strict mode flag for win32 and gtk #1753

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

akoch-yatta
Copy link
Contributor

@akoch-yatta akoch-yatta commented Jan 22, 2025

This contribution renames the internal strict mode flag in GTK from org.eclipse.swt.internal.gtk.enableStrictChecks to org.eclipse.swt.internal.enableStrictChecks and adds it to win32 to be used to check for correctly configured GCs for monitor-specific-scaling. The helps tracing invalid custom Drawables or GCs that are created with an invalid configuration.

Copy link
Contributor

github-actions bot commented Jan 22, 2025

Test Results

   492 files   -   2     492 suites   - 2   7m 21s ⏱️ -56s
 4 103 tests  - 227   4 096 ✅  - 221   7 💤  -  6  0 ❌ ±0 
16 151 runs   - 417  16 059 ✅  - 401  92 💤  - 16  0 ❌ ±0 

Results for commit 6c3b12c. ± Comparison against base commit 3cd1f36.

This pull request removes 227 tests.
org.eclipse.swt.graphics.ImageWin32Tests ‑ testImageDataForDifferentFractionalZoomsShouldBeDifferent
org.eclipse.swt.graphics.ImageWin32Tests ‑ testImageShouldHaveDimesionAsPerZoomLevel
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_afterPageReload[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_String[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_boolean[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_integer[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_javaReturningInt[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_javaReturningString[browser flags: 524,288]
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_BrowserFunction_callback_with_multipleValues[browser flags: 524,288]
…
This pull request skips 1 test.
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_Constructor_multipleInstantiationsInDifferentThreads[browser flags: 0]

♻️ This comment has been updated with latest results.

@akoch-yatta akoch-yatta changed the title [win32] Add internal strict mode flag for GC Unify internal strict mode flag for win32 and gtk Jan 22, 2025
Copy link
Member

@iloveeclipse iloveeclipse left a comment

Choose a reason for hiding this comment

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

Thanks

This contribution renames the internal strict mode flag in GTK from
org.eclipse.swt.internal.gtk.enableStrictChecks to
org.eclipse.swt.internal.enableStrictChecks and adds it to win32 to be
used to check for correctly configured GCs for monitor-specific-scaling
@HeikoKlare HeikoKlare merged commit 413426e into eclipse-platform:master Jan 23, 2025
14 checks passed
@HeikoKlare HeikoKlare deleted the gc-strict-mode branch January 23, 2025 17:29
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.

Optional Strict Mode GC Zoom Extraction
3 participants