Looks fine.

On 06/06/2018 13:12, Phil Race wrote:
As suggested :

--- a/test/jdk/java/awt/font/MonospacedGlyphWidth/MonospacedGlyphWidthTest.java +++ b/test/jdk/java/awt/font/MonospacedGlyphWidth/MonospacedGlyphWidthTest.java
@@ -22,10 +22,11 @@
   */

  /* @test

- * @bug 8073400 8198412

+ * @bug 8073400 8198412 8204126

   * @summary Some Monospaced logical fonts have a different width

   * @author Dmitry Markov

   * @run main MonospacedGlyphWidthTest

+ * @requires (os.family == "windows" | os.family == "mac")

   */

  import java.awt.Font;

  import java.awt.GraphicsEnvironment;


-phil

On 06/04/2018 03:50 PM, Sergey Bylokhov wrote:
Hi, Phil.
Maybe we can make it at least windows(and/or mac specific) specific?
(Related bug was win specific JDK-8073400)

On 04/06/2018 15:13, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8204126

I think it is time to stop running this test by default.
So one step short of deleting it is to ignore it

diff inline below :

--- a/test/jdk/java/awt/font/MonospacedGlyphWidth/MonospacedGlyphWidthTest.java +++ b/test/jdk/java/awt/font/MonospacedGlyphWidth/MonospacedGlyphWidthTest.java
@@ -22,10 +22,11 @@
   */

  /* @test
- * @bug 8073400 8198412
+ * @bug 8073400 8198412 8204126
   * @summary Some Monospaced logical fonts have a different width
   * @author Dmitry Markov
   * @run main MonospacedGlyphWidthTest
+ # @ignore too dependent on the system fonts to be a reliable test
   */
  import java.awt.Font;
  import java.awt.GraphicsEnvironment;

-phil.





--
Best regards, Sergey.

Reply via email to