+1

Vadim

On 11/02/2017 16:15, dmitry markov wrote:
Hi Dmitry,

The fix looks good to me too, but I am not a reviewer. So you need one more +1 from someone else with reviewer status.
I will sponsor integration of the fix once it is finally approved.

Thanks,
Dmitry
On 11/02/2017 02:48, Phil Race wrote:
Looks good to me.
It used to be a pain keeping several places in sync, hence the single util function. I guess that pre-dated the Apple code and we didn't realise it had its own copy ..
So much code ... so little time ...

-phil.

On 02/10/2017 01:07 AM, Dmitry Batrak wrote:
Hello,

I'd like to propose a fix for JDK-8170552.
  bug: https://bugs.openjdk.java.net/browse/JDK-8170552
webrev: http://cr.openjdk.java.net/~dbatrak/8170552/webrev.00/ <http://cr.openjdk.java.net/%7Edbatrak/8170552/webrev.00/>

I don't have a Committer status, so I'll require a sponsor.

I've proposed this patch earlier via support ticket (it's also attached to JIRA issue). It brings the code used to detect complex text when working with system fonts on macOS on par with similar code used for other fonts and platforms.

As compared to the previously provided patch, I've also added a test case. It depends on specifics of the font being used (Menlo), but I couldn't think of
another way to create an automated test for this issue.

Best regards,
Dmitry Batrak



Reply via email to