Re: RFR: 8297296: java/awt/Mouse/EnterExitEvents/DragWindowTest.java fails with "No MouseReleased event on label!" [v2]

2022-12-02 Thread Damon Nguyen
On Fri, 2 Dec 2022 19:53:14 GMT, Alisen Chung wrote: >> Test was run on mac, windows, linux 50 times and passed after change >> Before fix, test fails about once every 50 runs on linux and windows >> platforms > > Alisen Chung has updated the pull request incrementally with one additional > com

Re: RFR: 8273357: SecurityManager deprecation warning from java/awt/regtesthelpers/Util.java

2022-12-02 Thread Alexander Zuev
On Fri, 2 Dec 2022 23:07:45 GMT, Phil Race wrote: > The compilation of a number of tests that use regtesthelpers/Util.java are > getting a warning about > the deprecation-for-removal of the SecurityManager. > This suppresses that warning. Marked as reviewed by kizune (Reviewer). -

Re: RFR: JDK-8266245: AWT Test FullScreenInsets.java fails due to incorrect pixel color and wrong window bounds

2022-12-02 Thread Harshitha Onkar
On Fri, 2 Dec 2022 03:43:57 GMT, Sergey Bylokhov wrote: >> The above test was failing due to incorrect pixel color and wrong window >> bounds on Mac-ARM, Windows and Linux platforms. >> >> The following changes have been made: >> - Correct window bounds are obtained using the current ScreenDevi

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Phil Race
On Thu, 1 Dec 2022 19:38:44 GMT, Nikita Gubarkov wrote: >> It was implemented in JetBrains Runtime a year ago and was ported & >> refactored for this PR >> It includes: >> - Bitmap glyph loading via Freetype >> - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or >> bilin

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Phil Race
On Fri, 2 Dec 2022 19:27:30 GMT, Phil Race wrote: >> Nikita Gubarkov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix pointer to jlong conversion on x86 > > I imported the changes and submitted to our CI test system. There seem to be

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Nikita Gubarkov
On Fri, 2 Dec 2022 19:27:30 GMT, Phil Race wrote: >> Nikita Gubarkov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix pointer to jlong conversion on x86 > > I imported the changes and submitted to our CI test system. There seem to be

Re: RFR: 4512626: Non-editable JTextArea provides no visual indication of keyboard focus [v2]

2022-12-02 Thread Alexander Zuev
On Fri, 2 Dec 2022 23:52:55 GMT, Alexander Zuev wrote: >> Set the text caret to be visible but not blinking on the non-editable text >> area. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since the last revision: > > Handling the case when blink r

Re: RFR: 4512626: Non-editable JTextArea provides no visual indication of keyboard focus

2022-12-02 Thread Alexander Zuev
On Fri, 2 Dec 2022 04:16:21 GMT, Sergey Bylokhov wrote: > I wonder how other native applications handle this. Can the non-blinking > cursor be considered a hang of the application? Different application handle it differently but most os them do not claim to pass the VPAT certification. I do no

Re: RFR: 4512626: Non-editable JTextArea provides no visual indication of keyboard focus [v2]

2022-12-02 Thread Alexander Zuev
> Set the text caret to be visible but not blinking on the non-editable text > area. Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision: Handling the case when blink rate is set while component is unedited Test HidingSelectionTest.ja

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Nikita Gubarkov
On Thu, 1 Dec 2022 19:38:44 GMT, Nikita Gubarkov wrote: >> It was implemented in JetBrains Runtime a year ago and was ported & >> refactored for this PR >> It includes: >> - Bitmap glyph loading via Freetype >> - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or >> bilin

RFR: 8273357: SecurityManager deprecation warning from java/awt/regtesthelpers/Util.java

2022-12-02 Thread Phil Race
The compilation of a number of tests that use regtesthelpers/Util.java are getting a warning about the deprecation-for-removal of the SecurityManager. This suppresses that warning. - Commit messages: - 8273357 Changes: https://git.openjdk.org/jdk/pull/11492/files Webrev: https://w

Re: RFR: 8295729: Add jcheck whitespace checking for properties files [v3]

2022-12-02 Thread Magnus Ihse Bursie
On Fri, 2 Dec 2022 17:12:55 GMT, Andy Goryachev wrote: >> This turned out to be much more complicated than anticipated. I am going to >> close this PR (and bug), and instead I have split up this work in five >> different bugs: >> >> [JDK-8298047](https://bugs.openjdk.org/browse/JDK-8298047) is

Re: RFR: 8298042: Fix hidden but significant trailing whitespace in properties files for client code

2022-12-02 Thread SWinxy
On Fri, 2 Dec 2022 16:38:54 GMT, Magnus Ihse Bursie wrote: > According to [the > specification](https://docs.oracle.com/en/java/javase/19/docs/api/java.base/java/util/Properties.html#load(java.io.Reader)) > trailing whitespaces in the values of properties files are (somewhat > surprisingly) ac

Re: RFR: JDK-8266245: AWT Test FullScreenInsets.java fails due to incorrect pixel color and wrong window bounds

2022-12-02 Thread Harshitha Onkar
On Fri, 2 Dec 2022 03:43:57 GMT, Sergey Bylokhov wrote: >> The above test was failing due to incorrect pixel color and wrong window >> bounds on Mac-ARM, Windows and Linux platforms. >> >> The following changes have been made: >> - Correct window bounds are obtained using the screen device bein

Re: RFR: 8297296: java/awt/Mouse/EnterExitEvents/DragWindowTest.java fails with "No MouseReleased event on label!" [v2]

2022-12-02 Thread Alisen Chung
> Test was run on mac, windows, linux 50 times and passed after change > Before fix, test fails about once every 50 runs on linux and windows platforms Alisen Chung has updated the pull request incrementally with one additional commit since the last revision: fixed imports, changed robot autod

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Phil Race
On Thu, 1 Dec 2022 19:38:44 GMT, Nikita Gubarkov wrote: >> It was implemented in JetBrains Runtime a year ago and was ported & >> refactored for this PR >> It includes: >> - Bitmap glyph loading via Freetype >> - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or >> bilin

Re: RFR: JDK-8266245: AWT Test FullScreenInsets.java fails due to incorrect pixel color and wrong window bounds

2022-12-02 Thread Harshitha Onkar
On Fri, 2 Dec 2022 03:47:16 GMT, Sergey Bylokhov wrote: >> The above test was failing due to incorrect pixel color and wrong window >> bounds on Mac-ARM, Windows and Linux platforms. >> >> The following changes have been made: >> - Correct window bounds are obtained using the screen device bein

Re: RFR: JDK-8266245: AWT Test FullScreenInsets.java fails due to incorrect pixel color and wrong window bounds

2022-12-02 Thread Harshitha Onkar
On Fri, 2 Dec 2022 03:37:47 GMT, Sergey Bylokhov wrote: >> The above test was failing due to incorrect pixel color and wrong window >> bounds on Mac-ARM, Windows and Linux platforms. >> >> The following changes have been made: >> - Correct window bounds are obtained using the screen device bein

Re: RFR: 8295729: Add jcheck whitespace checking for properties files [v3]

2022-12-02 Thread Andy Goryachev
On Fri, 2 Dec 2022 17:10:17 GMT, Magnus Ihse Bursie wrote: > and instead I have split up this work in five different bugs would you consider also adding a unit test to check whether the localized resources also contain leading/trailing whitespace, and possibly special characters (like {, }, ,

RFR: 8298047: Remove all non-significant trailing whitespace from properties files

2022-12-02 Thread Magnus Ihse Bursie
[JDK-8295729](https://bugs.openjdk.org/browse/JDK-8295729) was created in an attempt to remove all trailing whitespace from properties files, and enable a jcheck verification that they did not come back, similar to other source code. It turned out that this was not so simple, however, since trai

Withdrawn: 8295729: Add jcheck whitespace checking for properties files

2022-12-02 Thread Magnus Ihse Bursie
On Thu, 20 Oct 2022 11:58:58 GMT, Magnus Ihse Bursie wrote: > Properties files is essentially source code. It should have the same > whitespace checks as all other source code, so we don't get spurious trailing > whitespace changes. > > With the new Skara jcheck, it is possible to increase the

Re: RFR: 8295729: Add jcheck whitespace checking for properties files [v3]

2022-12-02 Thread Magnus Ihse Bursie
On Mon, 24 Oct 2022 19:21:07 GMT, Magnus Ihse Bursie wrote: >> Properties files is essentially source code. It should have the same >> whitespace checks as all other source code, so we don't get spurious >> trailing whitespace changes. >> >> With the new Skara jcheck, it is possible to increas

RFR: 8298042: Fix hidden but significant trailing whitespace in properties files for client code

2022-12-02 Thread Magnus Ihse Bursie
According to [the specification](https://docs.oracle.com/en/java/javase/19/docs/api/java.base/java/util/Properties.html#load(java.io.Reader)) trailing whitespaces in the values of properties files are (somewhat surprisingly) actually significant. We have multiple files in the JDK with trailing

Re: RFR: 8298042: Fix hidden but significant trailing whitespace in properties files for client code

2022-12-02 Thread Magnus Ihse Bursie
On Fri, 2 Dec 2022 16:38:54 GMT, Magnus Ihse Bursie wrote: > According to [the > specification](https://docs.oracle.com/en/java/javase/19/docs/api/java.base/java/util/Properties.html#load(java.io.Reader)) > trailing whitespaces in the values of properties files are (somewhat > surprisingly) ac

Re: RFR: 8298006: Build failure by maybe-uninitialized error on Linux s390x GCC8

2022-12-02 Thread Magnus Ihse Bursie
On Fri, 2 Dec 2022 08:41:18 GMT, Ichiroh Takiguchi wrote: > I changed GCC toolchain from GCC6 to GCC8 on SLES12SP5 Linux s390x. > I could see following errors: > > src/java.desktop/share/native/libawt/awt/medialib/awt_ImagingLib.c: In > function 'allocateRasterArray': > src/java.desktop/share/

Re: RFR: 8269806: Emoji rendering on Linux [v16]

2022-12-02 Thread Laurent Bourgès
On Thu, 1 Dec 2022 19:38:44 GMT, Nikita Gubarkov wrote: >> It was implemented in JetBrains Runtime a year ago and was ported & >> refactored for this PR >> It includes: >> - Bitmap glyph loading via Freetype >> - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or >> bilin

Re: RFR: 8298006: Build failure by maybe-uninitialized error on Linux s390x GCC8

2022-12-02 Thread Ichiroh Takiguchi
On Fri, 2 Dec 2022 08:41:18 GMT, Ichiroh Takiguchi wrote: > I changed GCC toolchain from GCC6 to GCC8 on SLES12SP5 Linux s390x. > I could see following errors: > > src/java.desktop/share/native/libawt/awt/medialib/awt_ImagingLib.c: In > function 'allocateRasterArray': > src/java.desktop/share/

Re: RFR: 8296546: Add @spec tags to API [v4]

2022-12-02 Thread Daniel Fuchs
On Thu, 1 Dec 2022 19:36:16 GMT, Jonathan Gibbons wrote: >> Please review a "somewhat automated" change to insert `@spec` tags into doc >> comments, as appropriate, to leverage the recent new javadoc feature to >> generate a new page listing the references to all external specifications >> lis

Integrated: 8296084: javax/swing/JSpinner/4788637/bug4788637.java fails intermittently on a VM

2022-12-02 Thread Prasanta Sadhukhan
On Thu, 1 Dec 2022 15:29:48 GMT, Prasanta Sadhukhan wrote: > Test seems to fail intermittently in ubuntu OCI VM although it passes locally > and in other physical system. > Although I am not able to reproduce it on VM, tried some robot safeguards > used in other tests. > Modified tests passed

RFR: 8298006: Build failure by maybe-uninitialized error on Linux s390x GCC8

2022-12-02 Thread Ichiroh Takiguchi
I changed GCC toolchain from GCC6 to GCC8 on SLES12SP5 Linux s390x. I could see following errors: src/java.desktop/share/native/libawt/awt/medialib/awt_ImagingLib.c: In function 'allocateRasterArray': src/java.desktop/share/native/libawt/awt/medialib/awt_ImagingLib.c:2944:73: error: 'roff[3]' ma