Skip to content

Commit 7e3ee0c

Browse files
committed
Deprecate DefaultSWTFontRegistry and switch to ScalingSWTFontRegistry
1)Always use ScalingSWTFontRegistry 2)Rename DefaultSWTFontRegistry -> LegacySWTFontRegistry as this is not default anymore 3)Add a system property to use LegacySWTFontRegistry without monitor specific scaling active 4)Deprecate the LegacySWTFontRegistry(DefaultSWTFontRegistry)
1 parent 8c7b933 commit 7e3ee0c

File tree

5 files changed

+50
-13
lines changed

5 files changed

+50
-13
lines changed
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@
2323
import org.junit.jupiter.api.extension.*;
2424

2525
@ExtendWith(PlatformSpecificExecutionExtension.class)
26-
class DefaultSWTFontRegistryTests {
26+
class LegacySWTFontRegistryTests {
2727
private static String TEST_FONT = "Helvetica";
2828
private Display display;
2929
private SWTFontRegistry fontRegistry;
3030

3131
@BeforeEach
3232
public void setUp() {
3333
this.display = Display.getDefault();
34-
this.fontRegistry = new DefaultSWTFontRegistry(display);
34+
this.fontRegistry = new LegacySWTFontRegistry(display);
3535
}
3636

3737
@AfterEach

bundles/org.eclipse.swt/Eclipse SWT Tests/win32/org/eclipse/swt/widgets/ControlWin32Tests.java

Lines changed: 27 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,15 +55,38 @@ public void testSetFontWithMonitorSpecificScalingEnabled() {
5555
}
5656

5757
@Test
58-
public void testDoNotScaleFontCorrectlyInNoAutoScaleSzenario() {
58+
public void testScaleFontCorrectlyInNoAutoScaleSzenario() {
5959
DPIUtil.setMonitorSpecificScaling(false);
6060
Display display = Display.getDefault();
6161

62-
assertFalse("Autoscale property is not set to false", display.isRescalingAtRuntime());
62+
assertFalse("Autoscale property is not set to true", display.isRescalingAtRuntime());
6363
int scalingFactor = 2;
6464
FontComparison fontComparison = updateFont(scalingFactor);
65-
assertEquals("Font height in pixels is different when setting the same font again",
66-
fontComparison.originalFontHeight, fontComparison.currentFontHeight);
65+
assertEquals("Font height in pixels is not adjusted according to the scale factor",
66+
fontComparison.originalFontHeight * scalingFactor, fontComparison.currentFontHeight);
67+
}
68+
69+
@Test
70+
public void testDoNotScaleFontCorrectlyInNoAutoScaleSzenarioWithLegacyFontRegistry() {
71+
DPIUtil.setMonitorSpecificScaling(false);
72+
String originalValue = System.getProperty("swt.fontRegistry");
73+
System.setProperty("swt.fontRegistry", "legacy");
74+
try {
75+
Display display = Display.getDefault();
76+
77+
assertFalse("Autoscale property is not set to false", display.isRescalingAtRuntime());
78+
int scalingFactor = 2;
79+
FontComparison fontComparison = updateFont(scalingFactor);
80+
assertEquals("Font height in pixels is different when setting the same font again",
81+
fontComparison.originalFontHeight, fontComparison.currentFontHeight);
82+
}
83+
finally {
84+
if (originalValue != null) {
85+
System.setProperty("swt.fontRegistry", originalValue);
86+
} else {
87+
System.clearProperty("swt.fontRegistry");
88+
}
89+
}
6790
}
6891

6992
@Test
Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,25 @@
1919
import org.eclipse.swt.internal.win32.*;
2020

2121
/**
22-
* This class is used in the win32 implementation only to support
23-
* unscaled fonts in multiple DPI zoom levels.
22+
* <p>
23+
* Formerly {@code DefaultSWTFontRegistry}, this class is deprecated. Use {@code ScalingSWTFontRegistry} instead.
24+
* To temporarily fall back to legacy font behavior ({@code LegacySWTFontRegistry})
25+
* (e.g., if issues arise in existing RCP products), set the system property: {@code
26+
* -Dswt.fontRegistry=legacy
27+
* }
28+
* </p>
2429
*
2530
* As this class is only intended to be used internally via {@code SWTFontProvider},
2631
* it should neither be instantiated nor referenced in a client application.
2732
* The behavior can change any time in a future release.
2833
*/
29-
final class DefaultSWTFontRegistry implements SWTFontRegistry {
34+
@Deprecated(forRemoval= true, since= "2025-09")
35+
final class LegacySWTFontRegistry implements SWTFontRegistry {
3036
private static FontData KEY_SYSTEM_FONTS = new FontData();
3137
private Map<FontData, Font> fontsMap = new HashMap<>();
3238
private Device device;
3339

34-
DefaultSWTFontRegistry(Device device) {
40+
LegacySWTFontRegistry(Device device) {
3541
this.device = device;
3642
}
3743

bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/internal/SWTFontProvider.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,14 @@ public static void disposeFontRegistry(Device device) {
105105
}
106106
}
107107

108+
private static final String SWT_FONT_REGISTRY = "swt.fontRegistry";
109+
108110
private static SWTFontRegistry newFontRegistry(Device device) {
109-
if (device instanceof Display display && display.isRescalingAtRuntime()) {
110-
return new ScalingSWTFontRegistry(device);
111+
if ("legacy".equals(System.getProperty(SWT_FONT_REGISTRY)) && device instanceof Display display
112+
&& !display.isRescalingAtRuntime()) {
113+
return new LegacySWTFontRegistry(device);
111114
}
112-
return new DefaultSWTFontRegistry(device);
115+
return new ScalingSWTFontRegistry(device);
116+
113117
}
114118
}

bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/internal/ScalingSWTFontRegistry.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,11 @@ private Font getScaledFont(int zoom) {
4444

4545
private Font createAndCacheFont(int zoom) {
4646
Font newFont = createFont(zoom);
47+
<<<<<<< HEAD
4748
FontData clonedFontData = new FontData(newFont.getFontData()[0]);
49+
=======
50+
FontData clonedFontData = new FontData(newFont.getFontData()[0].toString());
51+
>>>>>>> af6c974e4c (Deprecate DefaultSWTFontRegistry and switch to ScalingSWTFontRegistry)
4852
fontsKeyMap.put(clonedFontData, this);
4953
fontHandlesKeyMap.put(Font.win32_getHandle(newFont), this);
5054
scaledFonts.put(zoom, newFont);

0 commit comments

Comments
 (0)