Re: RFR: 7001973: java/awt/Graphics2D/CopyAreaOOB.java fails [v2]

2021-11-11 Thread Masanori Yano
On Tue, 26 Oct 2021 04:52:37 GMT, Sergey Bylokhov wrote: >> Masanori Yano has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> com

Re: RFR: 8275715: D3D pipeline processes multiple PaintEvent at initial drawing [v2]

2021-11-11 Thread Masanori Yano
On Thu, 28 Oct 2021 09:03:55 GMT, Sergey Bylokhov wrote: >> Yes, this run() is called on "D3D Screen Updater" thread. It is reasonable >> that a new PaintEvent is posted when SurfaceData is replaced on this thread. >> I would limit posting new PaintEvent via createGraphics() only. > > Probably

Re: RFR: JDK-8276800 Fix table headers in NumericShaper.html [v4]

2021-11-11 Thread Ludvig Janiuk
On Wed, 10 Nov 2021 17:41:08 GMT, Ludvig Janiuk wrote: >> This change introduces no visual difference, but improves a11y. >> EDIT: There is a slight visual change regarding centering in a few cells. > > Ludvig Janiuk has updated the pull request incrementally with one additional > commit since t

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v6]

2021-11-11 Thread Jeremy
> This removes code that relied on consulting the Bezier control points to > calculate the Rectangle2D bounding box. Instead it's pretty straight-forward > to convert the Bezier control points into the x & y parametric equations. At > their most complex these equations are cubic polynomials, so

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v2]

2021-11-11 Thread Jeremy
On Wed, 10 Nov 2021 10:25:55 GMT, Laurent Bourgès wrote: >> @prrace @mrserb what do you think on this bug in terms of precision >> requirements? >> - should bbox always include control points ? >> - should ideal shape always fit in bbox ? So I propose to add a margin > >> numerical inaccuracies

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v2]

2021-11-11 Thread Laurent Bourgès
On Thu, 11 Nov 2021 09:40:07 GMT, Jeremy wrote: >> Ideally jdk could provide new math methods: >> - evalPoly(eqn[]) >> - find roots... >> Based on compensated horner scheme and Math.fma() ~ 0.5 ulp > > I attempted to explore a numerical approach to pad the bounding box here > 4b9d87d6d03e923bb26

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v2]

2021-11-11 Thread Laurent Bourgès
On Thu, 11 Nov 2021 10:18:29 GMT, Laurent Bourgès wrote: >> I attempted to explore a numerical approach to pad the bounding box here >> 4b9d87d6d03e923bb2663fc8f4f77b7549df6e70 ... but I don't like the results so >> far. Any suggestions? (Or if you have a better approach in mind we can >> disc

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS

2021-11-11 Thread Magnus Ihse Bursie
On Thu, 11 Nov 2021 05:52:18 GMT, Jayathirth D V wrote: > When JDK is built on BigSur with Xcode12.5 and used to run jtreg tests in > macOS10.15 with Xcode <=12.4 we are getting compilation error. We dont set > any deployment target when when we use xcrun to create .air file and this > issue l

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

2021-11-11 Thread Jayathirth D V
On Thu, 11 Nov 2021 13:52:13 GMT, Magnus Ihse Bursie wrote: > We should not hard-code version numbers like that. > > Fortunately for you, there already exists a variable $(MACOSX_VERSION_MIN) > that you can use. Thanks for the review i have updated code to use the Macro. - PR: ht

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

2021-11-11 Thread Jayathirth D V
> When JDK is built on BigSur with Xcode12.5 and used to run jtreg tests in > macOS10.15 with Xcode <=12.4 we are getting compilation error. We dont set > any deployment target when when we use xcrun to create .air file and this > issue looks similar to https://developer.apple.com/forums/thread/

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS

2021-11-11 Thread Jayathirth D V
On Thu, 11 Nov 2021 13:53:05 GMT, Magnus Ihse Bursie wrote: > Also, if you did not create this patch yourself, please make sure to use > `/contributor` to give proper credits. Or maybe you mean that Vitaly > submitted the bug report, not the patch? By Submitter i meant submitter of bug in JBS.

Re: RFR: JDK-8276809: java/awt/font/JNICheck/FreeTypeScalerJNICheck.java shows JNI warning on Windows

2021-11-11 Thread Matthias Baesken
On Wed, 10 Nov 2021 08:47:10 GMT, Matthias Baesken wrote: >> I think Xcheck:jni raises a warning when two JNI calls are made in a row w/o >> calling exception check in between. So it is not necessary to have an actual >> exception to produce a warning. >> >> Probably it is reproduced there, be

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

2021-11-11 Thread Magnus Ihse Bursie
On Thu, 11 Nov 2021 15:32:01 GMT, Jayathirth D V wrote: >> When JDK is built on BigSur with Xcode12.5 and used to run jtreg tests in >> macOS10.15 with Xcode <=12.4 we are getting compilation error. We dont set >> any deployment target when when we use xcrun to create .air file and this >> iss

Integrated: JDK-8276800 Fix table headers in NumericShaper.html

2021-11-11 Thread Ludvig Janiuk
On Mon, 8 Nov 2021 09:13:35 GMT, Ludvig Janiuk wrote: > This change introduces no visual difference, but improves a11y. > EDIT: There is a slight visual change regarding centering in a few cells. This pull request has now been integrated. Changeset: 5e98f993 Author:Ludvig Janiuk Committer:

Integrated: 8079267: [TEST_BUG] Test java/awt/Frame/MiscUndecorated/RepaintTest.java fails

2021-11-11 Thread Alexander Zvegintsev
On Thu, 28 Oct 2021 13:24:41 GMT, Alexander Zvegintsev wrote: > This test does not fail on Windows and does fail on Linux. > > After de-iconification we expect to see the button focused: > ![image](https://user-images.githubusercontent.com/77687766/139171852-54413dba-d58a-46d4-ab83-59d93868e28

Re: RFR: JDK-8276809: java/awt/font/JNICheck/FreeTypeScalerJNICheck.java shows JNI warning on Windows

2021-11-11 Thread Sergey Bylokhov
On Thu, 11 Nov 2021 15:58:50 GMT, Matthias Baesken wrote: >> Hello, in our central tests it was indeed a fastdbg OpenJDK. On my local >> Windows 10 machine I could not reproduce it. >> Our central tests run with -Djava.awt.headless=true , but I think the >> Win2019 server machine itself is n

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

2021-11-11 Thread Phil Race
On Thu, 11 Nov 2021 15:32:01 GMT, Jayathirth D V wrote: >> When JDK is built on BigSur with Xcode12.5 and used to run jtreg tests in >> macOS10.15 with Xcode <=12.4 we are getting compilation error. We dont set >> any deployment target when when we use xcrun to create .air file and this >> iss

Re: RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

2021-11-11 Thread Phil Race
On Fri, 12 Nov 2021 03:56:57 GMT, Phil Race wrote: >> Jayathirth D V has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Use Macro for version > > make/modules/java.desktop/lib/Awt2dLibraries.gmk line 850: > >> 848: COMMAND := $(METAL

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v7]

2021-11-11 Thread Jeremy
> This removes code that relied on consulting the Bezier control points to > calculate the Rectangle2D bounding box. Instead it's pretty straight-forward > to convert the Bezier control points into the x & y parametric equations. At > their most complex these equations are cubic polynomials, so

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v2]

2021-11-11 Thread Jeremy
On Thu, 11 Nov 2021 10:26:30 GMT, Laurent Bourgès wrote: >> You made an amazing job ! >> Your bigdecimal impl looks good. >> I will play with your test experiment... asap. >> I think numerical accuracies are related to the dynamic /magnitude of >> values: points. >> The test should evaluate(max

Re: RFR: JDK-8276809: java/awt/font/JNICheck/FreeTypeScalerJNICheck.java shows JNI warning on Windows [v2]

2021-11-11 Thread Matthias Baesken
> The new test java/awt/font/JNICheck/FreeTypeScalerJNICheck.java introduced > with https://bugs.openjdk.java.net/browse/JDK-8269223 adds -Xcheck:jni to an > awt related test, and shows on Windows server 2019 the following JNI warning > , so the test JNICheck/FreeTypeScalerJNICheck.java fails o