Skip to content

Commit 806ddb5

Browse files
arunjose696arunjose696
committed
Using eclipse Automatic refactoring to move Win32DPIUtils.pointToPixel(size,zoom) to DPIUtils
1 parent 0cd839a commit 806ddb5

File tree

31 files changed

+111
-110
lines changed

31 files changed

+111
-110
lines changed

‎bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/Edge.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1254,8 +1254,8 @@ int handleContextMenuRequested(long pView, long pArgs) {
12541254
// to PIXEL coordinates with the real native zoom value
12551255
// independent from the swt.autoScale property:
12561256
Point pt = new Point( //
1257-
Win32DPIUtils.pointToPixel(win32Point.x, DPIUtil.getNativeDeviceZoom()), //
1258-
Win32DPIUtils.pointToPixel(win32Point.y, DPIUtil.getNativeDeviceZoom()));
1257+
DPIUtil.pointToPixel(win32Point.x, DPIUtil.getNativeDeviceZoom()), //
1258+
DPIUtil.pointToPixel(win32Point.y, DPIUtil.getNativeDeviceZoom()));
12591259
// - then, scale back down from PIXEL to DISPLAY coordinates, taking
12601260
// swt.autoScale property into account
12611261
// which is also later considered in Menu#setLocation()

‎bundles/org.eclipse.swt/Eclipse SWT Drag and Drop/win32/org/eclipse/swt/dnd/DragSource.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -538,8 +538,8 @@ private void drag(Event dragEvent) {
538538
int flags = OS.RDW_UPDATENOW | OS.RDW_ALLCHILDREN;
539539
OS.RedrawWindow (topControl.handle, null, 0, flags);
540540
POINT pt = new POINT ();
541-
pt.x = Win32DPIUtils.pointToPixel(dragEvent.x, zoom);// To Pixels
542-
pt.y = Win32DPIUtils.pointToPixel(dragEvent.y, zoom);// To Pixels
541+
pt.x = DPIUtil.pointToPixel(dragEvent.x, zoom);// To Pixels
542+
pt.y = DPIUtil.pointToPixel(dragEvent.y, zoom);// To Pixels
543543
OS.MapWindowPoints (control.handle, 0, pt, 1);
544544
RECT rect = new RECT ();
545545
OS.GetWindowRect (hwndDrag, rect);

‎bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/graphics/ImageData.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2617,8 +2617,8 @@ static void fillGradientRectangle(GC gc, Device device,
26172617
int redBits, int greenBits, int blueBits, int zoom) {
26182618
/* Create the bitmap and tile it */
26192619
ImageDataProvider imageDataProvider = imageZoom -> {
2620-
int scaledWidth = Win32DPIUtils.pointToPixel(width, imageZoom);
2621-
int scaledHeight = Win32DPIUtils.pointToPixel(height, imageZoom);
2620+
int scaledWidth = DPIUtil.pointToPixel(width, imageZoom);
2621+
int scaledHeight = DPIUtil.pointToPixel(height, imageZoom);
26222622
return createGradientBand(scaledWidth, scaledHeight, vertical, fromRGB, toRGB, redBits, greenBits,
26232623
blueBits);
26242624
};

‎bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,15 @@ public static int mapZoomToDPI (int zoom) {
229229
return roundedDpi;
230230
}
231231

232+
/**
233+
* Auto-scale up int dimensions to match the given zoom level
234+
*/
235+
public static int pointToPixel(int size, int zoom) {
236+
if (zoom == 100 || size == SWT.DEFAULT) return size;
237+
float scaleFactor = getScalingFactor(zoom);
238+
return Math.round (size * scaleFactor);
239+
}
240+
232241
/**
233242
* Represents an element, such as some image data, at a specific zoom level.
234243
*

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/graphics/Image.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2194,8 +2194,8 @@ private ImageHandle createHandle(int zoom) {
21942194

21952195
private long initHandle(int zoom) {
21962196
if (isDisposed()) SWT.error(SWT.ERROR_GRAPHIC_DISPOSED);
2197-
int scaledWidth = Win32DPIUtils.pointToPixel (width, zoom);
2198-
int scaledHeight = Win32DPIUtils.pointToPixel (height, zoom);
2197+
int scaledWidth = DPIUtil.pointToPixel (width, zoom);
2198+
int scaledHeight = DPIUtil.pointToPixel (height, zoom);
21992199
long hDC = device.internal_new_GC(null);
22002200
long newHandle = OS.CreateCompatibleBitmap(hDC, scaledWidth, scaledHeight);
22012201
/*
@@ -2593,8 +2593,8 @@ protected ImageHandle newImageHandle(int zoom) {
25932593
int gcStyle = drawer.getGcStyle();
25942594
Image image;
25952595
if ((gcStyle & SWT.TRANSPARENT) != 0) {
2596-
int scaledHeight = Win32DPIUtils.pointToPixel(height, zoom);
2597-
int scaledWidth = Win32DPIUtils.pointToPixel(width, zoom);
2596+
int scaledHeight = DPIUtil.pointToPixel(height, zoom);
2597+
int scaledWidth = DPIUtil.pointToPixel(width, zoom);
25982598
/* Create a 24 bit image data with alpha channel */
25992599
final ImageData resultData = new ImageData (scaledWidth, scaledHeight, 24, new PaletteData (0xFF, 0xFF00, 0xFF0000));
26002600
resultData.alphaData = new byte [scaledWidth * scaledHeight];

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/graphics/Region.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,8 @@ public boolean contains (int x, int y) {
202202
if (isDisposed()) SWT.error(SWT.ERROR_GRAPHIC_DISPOSED);
203203
return applyUsingAnyHandle(regionHandle -> {
204204
int zoom = regionHandle.zoom();
205-
int xInPixels = Win32DPIUtils.pointToPixel(x, zoom);
206-
int yInPixels = Win32DPIUtils.pointToPixel(y, zoom);
205+
int xInPixels = DPIUtil.pointToPixel(x, zoom);
206+
int yInPixels = DPIUtil.pointToPixel(y, zoom);
207207
return containsInPixels(regionHandle.handle(), xInPixels, yInPixels);
208208
});
209209
}

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/graphics/TextLayout.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -379,9 +379,9 @@ void computeRuns (GC gc) {
379379
}
380380
SCRIPT_LOGATTR logAttr = new SCRIPT_LOGATTR();
381381
SCRIPT_PROPERTIES properties = new SCRIPT_PROPERTIES();
382-
int wrapIndentInPixels = Win32DPIUtils.pointToPixel(wrapIndent, getZoom(gc));
383-
int indentInPixels = Win32DPIUtils.pointToPixel(indent, getZoom(gc));
384-
int wrapWidthInPixels = Win32DPIUtils.pointToPixel(wrapWidth, getZoom(gc));
382+
int wrapIndentInPixels = DPIUtil.pointToPixel(wrapIndent, getZoom(gc));
383+
int indentInPixels = DPIUtil.pointToPixel(indent, getZoom(gc));
384+
int wrapWidthInPixels = DPIUtil.pointToPixel(wrapWidth, getZoom(gc));
385385
int[] tabsInPixels = Win32DPIUtils.pointToPixel(tabs, getZoom(gc));
386386
int lineWidth = indentInPixels, lineStart = 0, lineCount = 1;
387387
for (int i=0; i<allRuns.length - 1; i++) {
@@ -1896,8 +1896,8 @@ Rectangle getBoundsInPixels (int start, int end) {
18961896
}
18971897
left = Math.min(left, runLead);
18981898
right = Math.max(right, runTrail);
1899-
top = Math.min(top, Win32DPIUtils.pointToPixel(lineY[lineIndex], getZoom(null)));
1900-
bottom = Math.max(bottom, Win32DPIUtils.pointToPixel(lineY[lineIndex + 1] - lineSpacingInPoints, getZoom(null)));
1899+
top = Math.min(top, DPIUtil.pointToPixel(lineY[lineIndex], getZoom(null)));
1900+
bottom = Math.max(bottom, DPIUtil.pointToPixel(lineY[lineIndex + 1] - lineSpacingInPoints, getZoom(null)));
19011901
}
19021902
return new Rectangle(left, top, right - left, bottom - top + getScaledVerticalIndent());
19031903
}
@@ -2057,14 +2057,14 @@ int getLineIndent(int lineIndex) {
20572057
}
20582058

20592059
int getLineIndentInPixel (int lineIndex) {
2060-
int lineIndent = Win32DPIUtils.pointToPixel(wrapIndent, getZoom());
2060+
int lineIndent = DPIUtil.pointToPixel(wrapIndent, getZoom());
20612061
if (lineIndex == 0) {
2062-
lineIndent = Win32DPIUtils.pointToPixel(indent, getZoom());
2062+
lineIndent = DPIUtil.pointToPixel(indent, getZoom());
20632063
} else {
20642064
StyleItem[] previousLine = runs[lineIndex - 1];
20652065
StyleItem previousRun = previousLine[previousLine.length - 1];
20662066
if (previousRun.lineBreak && !previousRun.softBreak) {
2067-
lineIndent = Win32DPIUtils.pointToPixel(indent, getZoom());
2067+
lineIndent = DPIUtil.pointToPixel(indent, getZoom());
20682068
}
20692069
}
20702070
if (wrapWidth != -1) {
@@ -2078,8 +2078,8 @@ int getLineIndentInPixel (int lineIndex) {
20782078
if (partialLine) {
20792079
int lineWidth = this.lineWidthInPixels[lineIndex] + lineIndent;
20802080
switch (alignment) {
2081-
case SWT.CENTER: lineIndent += (Win32DPIUtils.pointToPixel(wrapWidth, getZoom()) - lineWidth) / 2; break;
2082-
case SWT.RIGHT: lineIndent += Win32DPIUtils.pointToPixel(wrapWidth, getZoom()) - lineWidth; break;
2081+
case SWT.CENTER: lineIndent += (DPIUtil.pointToPixel(wrapWidth, getZoom()) - lineWidth) / 2; break;
2082+
case SWT.RIGHT: lineIndent += DPIUtil.pointToPixel(wrapWidth, getZoom()) - lineWidth; break;
20832083
}
20842084
}
20852085
}

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/internal/ImageList.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -332,8 +332,8 @@ public int getStyle () {
332332

333333
public long getHandle(int targetZoom) {
334334
if (!zoomToHandle.containsKey(targetZoom)) {
335-
int scaledWidth = Win32DPIUtils.pointToPixel(DPIUtil.pixelToPoint(width, this.zoom), targetZoom);
336-
int scaledHeight = Win32DPIUtils.pointToPixel(DPIUtil.pixelToPoint(height, this.zoom), targetZoom);
335+
int scaledWidth = DPIUtil.pointToPixel(DPIUtil.pixelToPoint(width, this.zoom), targetZoom);
336+
int scaledHeight = DPIUtil.pointToPixel(DPIUtil.pixelToPoint(height, this.zoom), targetZoom);
337337
long newImageListHandle = OS.ImageList_Create(scaledWidth, scaledHeight, flags, 16, 16);
338338
int count = OS.ImageList_GetImageCount (handle);
339339
for (int i = 0; i < count; i++) {

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/internal/Win32DPIUtils.java

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -174,18 +174,9 @@ public static int[] pointToPixel(Drawable drawable, int[] pointArray, int zoom)
174174
return pointToPixel (pointArray, zoom);
175175
}
176176

177-
/**
178-
* Auto-scale up int dimensions to match the given zoom level
179-
*/
180-
public static int pointToPixel(int size, int zoom) {
181-
if (zoom == 100 || size == SWT.DEFAULT) return size;
182-
float scaleFactor = DPIUtil.getScalingFactor(zoom);
183-
return Math.round (size * scaleFactor);
184-
}
185-
186177
public static int pointToPixel(Drawable drawable, int size, int zoom) {
187178
if (drawable != null && !drawable.isAutoScalable()) return size;
188-
return pointToPixel (size, zoom);
179+
return DPIUtil.pointToPixel (size, zoom);
189180
}
190181

191182
public static float pointToPixel(float size, int zoom) {

‎bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Button.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -362,10 +362,10 @@ else if ((style & SWT.RIGHT) != 0) {
362362
Rectangle rect = Win32DPIUtils.scaleBounds(image.getBounds(), this.getZoom(), 100);
363363
width = rect.width;
364364
if (hasText && text.length () != 0) {
365-
width += Win32DPIUtils.pointToPixel(MARGIN * 2, getZoom());;
365+
width += DPIUtil.pointToPixel(MARGIN * 2, getZoom());;
366366
}
367367
height = rect.height;
368-
extra = Win32DPIUtils.pointToPixel(MARGIN * 2, getZoom());;
368+
extra = DPIUtil.pointToPixel(MARGIN * 2, getZoom());;
369369
}
370370
}
371371
if (hasText) {
@@ -379,7 +379,7 @@ else if ((style & SWT.RIGHT) != 0) {
379379
if (length == 0) {
380380
height = Math.max (height, lptm.tmHeight);
381381
} else {
382-
extra = Math.max (Win32DPIUtils.pointToPixel(MARGIN * 2, getZoom()), lptm.tmAveCharWidth);
382+
extra = Math.max (DPIUtil.pointToPixel(MARGIN * 2, getZoom()), lptm.tmAveCharWidth);
383383
char [] buffer = text.toCharArray ();
384384
RECT rect = new RECT ();
385385
int flags = OS.DT_CALCRECT | OS.DT_SINGLELINE;
@@ -1315,7 +1315,7 @@ private int getCheckboxTextOffset(long hdc) {
13151315
OS.GetThemePartSize(display.hButtonTheme(nativeZoom), hdc, OS.BP_CHECKBOX, OS.CBS_UNCHECKEDNORMAL, null, OS.TS_TRUE, size);
13161316
result += size.cx;
13171317
} else {
1318-
result += Win32DPIUtils.pointToPixel(13, nativeZoom);
1318+
result += DPIUtil.pointToPixel(13, nativeZoom);
13191319
}
13201320

13211321
// Windows uses half width of '0' as checkbox-to-text distance.

0 commit comments

Comments
 (0)