Re: RFR: 8270116: Expand ButtonGroupLayoutTraversalTest.java to run in all LaFs, including Aqua on macOS

2021-07-30 Thread rajat mahajan
On Fri, 30 Jul 2021 05:35:58 GMT, Prasanta Sadhukhan wrote: >> Summary: Expanded ButtonGroupLayoutTraversalTest.java to run in all LAFs on >> all OS. Added synchronization for focusCnt. > >

Re: RFR: 8270116: Expand ButtonGroupLayoutTraversalTest.java to run in all LaFs, including Aqua on macOS

2021-07-30 Thread Alexey Ivanov
On Thu, 29 Jul 2021 20:29:31 GMT, rajat mahajan wrote: > Summary: Expanded ButtonGroupLayoutTraversalTest.java to run in all LAFs on > all OS. Added synchronization for focusCnt. Looks good to me. - Marked as reviewed by aivanov (Reviewer). PR:

RFR: 8266079: Lanai: AlphaComposite shows differences on Metal compared to OpenGL

2021-07-30 Thread Alexey Ushakov
Keep MTLLayer opacity in sync with window content view Keep layer translucent for translucent windows - Commit messages: - 8266079: Lanai: AlphaComposite shows differences on Metal compared to OpenGL Changes: https://git.openjdk.java.net/jdk/pull/4946/files Webrev:

Re: RFR: 8270058: Use Objects.check{Index, FromIndexSize} for java.desktop [v3]

2021-07-30 Thread Jayathirth D V
On Fri, 30 Jul 2021 08:40:34 GMT, Pankaj Bansal wrote: >> src/java.desktop/share/classes/javax/swing/JTabbedPane.java line 1793: >> >>> 1791: >>> 1792: private void checkIndex(int index) { >>> 1793: Objects.checkIndex(index, pages.size()); >> >> I think we should not replace IOOBE

Re: RFR: 8270058: Use Objects.check{Index, FromIndexSize} for java.desktop [v3]

2021-07-30 Thread Pankaj Bansal
On Thu, 22 Jul 2021 06:29:13 GMT, Yi Yang wrote: >> After JDK-8265518(#3615), it's possible to replace all variants of >> checkIndex by >> Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in >> the whole JDK codebase. >> >> As Mandy suggested, I create this PR for

Re: RFR: 8270058: Use Objects.check{Index, FromIndexSize} for java.desktop [v3]

2021-07-30 Thread Pankaj Bansal
On Fri, 30 Jul 2021 08:17:29 GMT, Jayathirth D V wrote: >> Yi Yang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> typo > > src/java.desktop/share/classes/javax/swing/JTabbedPane.java line 1793: > >> 1791: >> 1792: private void

Re: RFR: 8270058: Use Objects.check{Index, FromIndexSize} for java.desktop [v3]

2021-07-30 Thread Jayathirth D V
On Thu, 22 Jul 2021 06:29:13 GMT, Yi Yang wrote: >> After JDK-8265518(#3615), it's possible to replace all variants of >> checkIndex by >> Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in >> the whole JDK codebase. >> >> As Mandy suggested, I create this PR for

Re: RFR: 8270058: Use Objects.check{Index, FromIndexSize} for java.desktop [v2]

2021-07-30 Thread Yi Yang
On Fri, 30 Jul 2021 06:23:40 GMT, Prasanta Sadhukhan wrote: >> Hi, sorry for the late response. checkFromToIndex has many constraints, this >> code may not satisfy those constraints, so I don't think we should do that >> in this PR.(Maybe we can file a new PR to do replacement carefully) > >

Re: RFR: 8270058: Use Objects.check{Index, FromToIndex, FromIndexSize} for java.desktop [v3]

2021-07-30 Thread Prasanta Sadhukhan
On Thu, 22 Jul 2021 06:29:13 GMT, Yi Yang wrote: >> After JDK-8265518(#3615), it's possible to replace all variants of >> checkIndex by >> Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in >> the whole JDK codebase. >> >> As Mandy suggested, I create this PR for