Skip to content

Conversation

@akoch-yatta
Copy link
Contributor

@akoch-yatta akoch-yatta commented May 5, 2025

This PR adapts the extraction of the correct image handle in the windows implementation when using an Image with a GC. As the image will always pass the exact zoom it needs to the GC using the autoscaled zoom together with an Image can lead to unexpected results when fixed values for swt.autoScale are passed, e.g. scaling ImageData can result in endless loops.

This effect can be seen e.g. by executing Snippet119 with

-Dswt.autoScale=200
-Dswt.autoScale.updateOnRuntime=true

This commit adapts the extraction of the correct image handle in the
windows implementation when using an Image with a GC. As the image will
always pass the exact zoom it needs to the GC using the autoscaled zoom
together with an Image can lead to unexpected results when fixed values
for swt.autoScale are passed, e.g. scaling ImageData can result in
endless loops.
@github-actions
Copy link
Contributor

github-actions bot commented May 5, 2025

Test Results

   545 files  ±0     545 suites  ±0   32m 45s ⏱️ + 7m 12s
 4 374 tests ±0   4 356 ✅ ±0   18 💤 ±0  0 ❌ ±0 
16 638 runs  ±0  16 498 ✅ ±0  140 💤 ±0  0 ❌ ±0 

Results for commit 62c08fd. ± Comparison against base commit dd30aff.

♻️ This comment has been updated with latest results.

@fedejeanne fedejeanne merged commit d675f85 into eclipse-platform:master May 5, 2025
17 checks passed
@fedejeanne fedejeanne deleted the win32-image-gc-endless-loop branch May 5, 2025 11: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.

2 participants