Re: RFR: 6510914: JScrollBar.getMinimumSize() breaks the contract of JComponent.setMinimumSize() [v7]

2024-01-02 Thread Prasanta Sadhukhan
On Tue, 21 Nov 2023 03:33:21 GMT, Prasanta Sadhukhan wrote: >> javadoc contract for JComponent.setMinimumSize(Dimension) states: >> >> "Sets the minimum size of this component to a constant value. Subsequent >> calls to getMinimumSize will always return this value..." >> >> However, JScrollBa

Re: RFR: 6510914: JScrollBar.getMinimumSize() breaks the contract of JComponent.setMinimumSize() [v8]

2024-01-02 Thread Prasanta Sadhukhan
> javadoc contract for JComponent.setMinimumSize(Dimension) states: > > "Sets the minimum size of this component to a constant value. Subsequent > calls to getMinimumSize will always return this value..." > > However, JScrollBar overrides getMinimumSize() and breaks this contract - it > always

RFR: 8264102: JTable Keyboards Navigation differs with Test Instructions.

2024-01-02 Thread Tejesh R
Updated the actions for "ctrl shift DOWN" to "`selectLastRowExtendSelection`" and "ctrl shift UP", "`selectFirstRowExtendSelection`" in Basic and other Look and feel. Tested in CI and no regressions found. Test to be used - [javax/swing/JTable/KeyBoardNavigation/KeyBoardNavigation.java](https:

Re: RFR: 8316497 : ColorConvertOp - typo for non-ICC conversions needs one-line fix [v6]

2024-01-02 Thread Renjith Kannath Pariyangad
On Tue, 2 Jan 2024 05:31:17 GMT, Sergey Bylokhov wrote: >> Renjith Kannath Pariyangad has updated the pull request incrementally with >> one additional commit since the last revision: >> >> Removed unnecessary try block > > test/jdk/java/awt/color/NonICCFilterTest.java line 109: > >> 107:

Re: RFR: 8316497 : ColorConvertOp - typo for non-ICC conversions needs one-line fix [v7]

2024-01-02 Thread Renjith Kannath Pariyangad
> Hi Reviewers, > There was a typo for color conversion instead of dstColorSpace function > srcColorSpace was used. Please review and let me know your suggestions if > any. > > Renjith. Renjith Kannath Pariyangad has updated the pull request incrementally with two additional commits since th

Re: RFR: 8316497 : ColorConvertOp - typo for non-ICC conversions needs one-line fix [v6]

2024-01-02 Thread Renjith Kannath Pariyangad
On Tue, 2 Jan 2024 05:29:35 GMT, Sergey Bylokhov wrote: >> Renjith Kannath Pariyangad has updated the pull request incrementally with >> one additional commit since the last revision: >> >> Removed unnecessary try block > > test/jdk/java/awt/color/NonICCFilterTest.java line 47: > >> 45: >>

Re: RFR: JDK-8320405: [Windows Server 2016] java/awt/image/MultiResolutionImage/MultiResolutionImageObserverTest.java shows issues in awt_Win32GraphicsDevice.cpp

2024-01-02 Thread Sergey Bylokhov
On Tue, 2 Jan 2024 20:04:39 GMT, Alexey Ivanov wrote: >> src/java.desktop/windows/native/libawt/windows/awt_Win32GraphicsDevice.cpp >> line 194: >> >>> 192: if (hBM == NULL) { >>> 193: J2dTraceLn(J2D_TRACE_WARNING, >>> "AwtWin32GraphicsDevice::Initialize CreateCompatibleBitmap fail

Re: RFR: 6510914: JScrollBar.getMinimumSize() breaks the contract of JComponent.setMinimumSize() [v7]

2024-01-02 Thread Alexey Ivanov
On Tue, 21 Nov 2023 03:33:21 GMT, Prasanta Sadhukhan wrote: >> javadoc contract for JComponent.setMinimumSize(Dimension) states: >> >> "Sets the minimum size of this component to a constant value. Subsequent >> calls to getMinimumSize will always return this value..." >> >> However, JScrollBa

Re: RFR: JDK-8320405: [Windows Server 2016] java/awt/image/MultiResolutionImage/MultiResolutionImageObserverTest.java shows issues in awt_Win32GraphicsDevice.cpp

2024-01-02 Thread Alexey Ivanov
On Tue, 2 Jan 2024 19:37:53 GMT, Alexey Ivanov wrote: > It seems that we cannot continue if `hBMDC` or `hBM` are null. Yet it seems > unexpected. Should we rather modify the code to back out and not to call APIs with NULL handles? - PR Review Comment: https://git.openjdk.org/jdk/

Re: RFR: JDK-8320405: [Windows Server 2016] java/awt/image/MultiResolutionImage/MultiResolutionImageObserverTest.java shows issues in awt_Win32GraphicsDevice.cpp

2024-01-02 Thread Alexey Ivanov
On Thu, 28 Dec 2023 09:48:52 GMT, Matthias Baesken wrote: > When running with fastdebug binaries we run intermittent into the issue below > in > jtreg test > java/awt/image/MultiResolutionImage/MultiResolutionImageObserverTest.java . > Seems we miss checking of successful HBITMAP creation befor

Re: RFR: 8293862: javax/swing/JFileChooser/8046391/bug8046391.java failed with 'Cannot invoke "java.awt.Image.getWidth(java.awt.image.ImageObserver)" because "retVal" is null'

2024-01-02 Thread Alexey Ivanov
On Mon, 16 Jan 2023 14:19:52 GMT, Alexey Ivanov wrote: > This is somewhat a continuation of #11104 where the issue was discussed and > where I found [the root > cause](https://github.com/openjdk/jdk/pull/11104#issuecomment-1382435784). > > **Root Cause** > > The icon extraction code compares

Integrated: 8320343 : Generate GIF images for AbstractButton/5049549/bug5049549.java

2024-01-02 Thread Renjith Kannath Pariyangad
On Fri, 8 Dec 2023 03:23:22 GMT, Renjith Kannath Pariyangad wrote: > Hi Reviewers, > > Updated the test and it will generate required images on the fly so storing > and loading image from repo could be avoided. Please review and let me know > your suggestions. > > Regards, > Renjith. This p

Re: RFR: 8320343 : Generate GIF images for AbstractButton/5049549/bug5049549.java [v6]

2024-01-02 Thread Alexey Ivanov
On Wed, 27 Dec 2023 03:47:16 GMT, Renjith Kannath Pariyangad wrote: >> Hi Reviewers, >> >> Updated the test and it will generate required images on the fly so storing >> and loading image from repo could be avoided. Please review and let me know >> your suggestions. >> >> Regards, >> Renjith

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

2024-01-02 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 , and shows on Windows server 2019 the following JNI warning , so the test fails on this Windows version. stdout: [WARNING in native method: JNI cal

Re: RFR: 8293862: javax/swing/JFileChooser/8046391/bug8046391.java failed with 'Cannot invoke "java.awt.Image.getWidth(java.awt.image.ImageObserver)" because "retVal" is null'

2024-01-02 Thread Karl T
On Mon, 16 Jan 2023 14:19:52 GMT, Alexey Ivanov wrote: > This is somewhat a continuation of #11104 where the issue was discussed and > where I found [the root > cause](https://github.com/openjdk/jdk/pull/11104#issuecomment-1382435784). > > **Root Cause** > > The icon extraction code compares

Re: RFR: 8319598: SMFParser misinterprets interrupted running status [v2]

2024-01-02 Thread Sergey Bylokhov
On Sat, 11 Nov 2023 12:32:15 GMT, Jan Trukenmüller wrote: >> The MIDI file parser misinterprets events without status byte when they >> appear directly after a Meta of SysEx event. >> >> For my bugfix I had to decide between two possible solutions: >> - Strict solution: Throw an InvalidMidiData