8203485: [freetype] text rotated on 180 degrees is too narrow
authorbae
Fri, 25 May 2018 11:35:31 +0300
changeset 50346 9e530b150333
parent 50345 276cb4b17f79
child 50347 b2f046ae8eb6
8203485: [freetype] text rotated on 180 degrees is too narrow Reviewed-by: prr, serb
src/java.desktop/share/native/libfontmanager/freetypeScaler.c
test/jdk/java/awt/font/Rotate/RotatedTextTest.java
--- a/src/java.desktop/share/native/libfontmanager/freetypeScaler.c	Wed May 23 23:02:02 2018 -0700
+++ b/src/java.desktop/share/native/libfontmanager/freetypeScaler.c	Fri May 25 11:35:31 2018 +0300
@@ -45,7 +45,7 @@
 #define  FloatToFTFixed(f) (FT_Fixed)((f) * (float)(ftFixed1))
 #define  FTFixedToFloat(x) ((x) / (float)(ftFixed1))
 #define  FT26Dot6ToFloat(x)  ((x) / ((float) (1<<6)))
-#define  ROUND(x) ((int) (x+0.5))
+#define  FT26Dot6ToInt(x) (((int)(x)) >> 6)
 
 typedef struct {
     /* Important note:
@@ -765,12 +765,12 @@
     } else {
         if (!ftglyph->advance.y) {
             glyphInfo->advanceX =
-                (float) ROUND(FT26Dot6ToFloat(ftglyph->advance.x));
+                (float) FT26Dot6ToInt(ftglyph->advance.x);
             glyphInfo->advanceY = 0;
         } else if (!ftglyph->advance.x) {
             glyphInfo->advanceX = 0;
             glyphInfo->advanceY =
-                (float) ROUND(FT26Dot6ToFloat(-ftglyph->advance.y));
+                (float) FT26Dot6ToInt(-ftglyph->advance.y);
         } else {
             glyphInfo->advanceX = FT26Dot6ToFloat(ftglyph->advance.x);
             glyphInfo->advanceY = FT26Dot6ToFloat(-ftglyph->advance.y);
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/jdk/java/awt/font/Rotate/RotatedTextTest.java	Fri May 25 11:35:31 2018 +0300
@@ -0,0 +1,140 @@
+/*
+ * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test RotatedTextTest
+ * @bug 8203485
+ * @summary This test verifies that rotated text preserves the width.
+ * @run main RotatedTextTest
+ */
+
+import java.awt.Color;
+import java.awt.Dimension;
+import java.awt.Font;
+import java.awt.FontMetrics;
+import java.awt.Graphics;
+import java.awt.Graphics2D;
+import java.awt.geom.Rectangle2D;
+import java.awt.image.BufferedImage;
+import java.io.BufferedOutputStream;
+import java.io.File;
+import java.io.FileOutputStream;
+import javax.imageio.ImageIO;
+
+public class RotatedTextTest
+{
+    static final int size = 720;
+    static final Font fnt = new Font(Font.SERIF, Font.PLAIN, 12);
+    static final String text = "The quick brown fox jumps over the lazy dog";
+
+    static void drawRotatedText(Graphics g) {
+        Graphics2D g2d = (Graphics2D)g;
+
+        g2d.setFont(fnt);
+
+        FontMetrics metrics = g2d.getFontMetrics();
+        Rectangle2D bounds = metrics.getStringBounds(text, g2d);
+
+
+        g2d.setColor(Color.black);
+        g2d.fillRect(0, 0, size, size);
+
+        g2d.setColor(Color.white);
+
+        float[] angles = new float[] { 0, 15, 45, 60, 90, -45, -90, -135, -180, 135, };
+
+        for (float a : angles) {
+            Graphics2D rotated = (Graphics2D)g2d.create();
+
+            rotated.translate(size/2, size/2);
+            rotated.rotate(Math.toRadians(a));
+            rotated.translate(30, 0);
+
+            int width_original = metrics.stringWidth(text);
+            int width_rotated = rotated.getFontMetrics().stringWidth(text);
+
+            rotated.drawString(text, 0, 0);
+            rotated.drawString(String.format("  %.0f", a), width_original, 0);
+
+            rotated.setColor(Color.green);
+            rotated.draw(bounds);
+
+            System.out.printf("Angle: %.0f, width diff: %d\n", a, (width_rotated - width_original));
+
+            rotated.dispose();
+
+            if (width_rotated != width_original) {
+                throw new RuntimeException("Test failed for angle " + a);
+            }
+        }
+    }
+
+    public static void main(String args[])
+    {
+        final BufferedImage dst = new BufferedImage(size, size, BufferedImage.TYPE_INT_RGB);
+        Graphics g = dst.createGraphics();
+
+        try {
+            drawRotatedText(g);
+        } finally {
+            g.dispose();
+            writeToFile(dst);
+        }
+    }
+
+    static final String title;
+    static final String file;
+
+    static {
+        String vendorName = System.getProperty("java.vendor");
+        String version = System.getProperty("java.version");
+        String runtimeName = System.getProperty("java.runtime.name");
+
+
+        int index = runtimeName.indexOf(" Runtime Environment");
+        runtimeName = runtimeName.substring(0, index).trim();
+
+        title = vendorName + ", " + runtimeName + ", " + version;
+        file = "rotated-text-" + title.replace(", ", "-")
+                .replace(" ", "-") + ".png";
+    }
+
+    private static void writeToFile(final BufferedImage bi) {
+        File imageFile = new File(file);
+        FileOutputStream imageOutStream;
+        BufferedOutputStream imageBOS = null;
+        try {
+            imageOutStream = new FileOutputStream(imageFile);
+            imageBOS = new BufferedOutputStream(imageOutStream);
+
+            ImageIO.setUseCache(false);
+            ImageIO.write(bi, "png", imageBOS);
+
+            imageBOS.close();
+            imageOutStream.close();
+        }
+        catch (Exception e) {
+            System.out.println(e.getMessage());
+        }
+    }
+}