[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] RFR: 8204126: MonospacedGlyphWidthTest.java failed again
From: Sergey Bylokhov <Sergey.Bylokhov () oracle ! com>
Date: 2018-06-06 21:00:29
Message-ID: 2eb5d1b3-58dc-1268-99ad-3ded3ebb356b () oracle ! com
[Download RAW message or body]
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.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic