Integrated: 8300725: Improve performance of ColorConvertOp for default destinations with alpha

2023-01-20 Thread Sergey Bylokhov
On Fri, 20 Jan 2023 06:02:53 GMT, Sergey Bylokhov wrote: > The "default destination" for the ColorConverOp is used when the user passes > "null" as the DST parameter. In this case, the "ColorConverOp.filter" creates > the default image and uses ComponentColorModel: 8 bits per color component

Re: RFR: 8299333: Unify exceptions used by all variants of ICC_Profile.getInstance(null) [v2]

2023-01-20 Thread Sergey Bylokhov
On Sat, 21 Jan 2023 02:03:57 GMT, Phil Race wrote: > So there are 3 TCK tests for the behaviour of getInstance(InputStream) which > expect the specified IOException or an IAE in the event the arg is null. It > will fail if we throw NPE (ie these 3 tests fail on all platforms with your >

RFR: 8299497: Usage of constructors of primitive wrapper classes should be avoided in java.desktop API docs

2023-01-20 Thread Tingjun Yuan
API docs of `java.awt.image.renderable.ParameterBlock` and `java.awt.font.LineBreakMeasurer` used deprecated constructors of `java.lang.Short` and `java.lang.Float`. Replacing by `valueOf` factory methods. - Commit messages: - Update LineBreakMeasurer.java - Update

Re: RFR: 8299333: Unify exceptions used by all variants of ICC_Profile.getInstance(null) [v2]

2023-01-20 Thread Phil Race
On Fri, 20 Jan 2023 20:20:32 GMT, Sergey Bylokhov wrote: >> I tried to clean up the documentation in the `java.awt.color` package and >> specify how the `null` parameters are handled here and there. But it looks >> like the `ICC_profile` class is too specific so I would like to discuss it >>

Re: RFR: 8300692: GCC 12 reports some compiler warnings in bundled freetype [v3]

2023-01-20 Thread Dan Lutker
On Fri, 20 Jan 2023 23:47:37 GMT, Dan Lutker wrote: >> Disabling warnings for bundled Freetype > > Dan Lutker has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains one commit: > > 8300692: GCC 12 reports some compiler warnings in

Re: RFR: 8300692: GCC 12 reports some compiler warnings in bundled freetype [v3]

2023-01-20 Thread Dan Lutker
> Disabling warnings for bundled Freetype Dan Lutker 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. - Changes: - all:

Re: RFR: 8300692: GCC 12 reports some compiler warnings in bundled freetype [v2]

2023-01-20 Thread Dan Lutker
> Disabling warnings for bundled Freetype Dan Lutker 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 two additional commits since the last revision:

Re: RFR: 8300269: The selected item in an editable JComboBox with titled border is not visible in Aqua LAF [v3]

2023-01-20 Thread Damon Nguyen
> The fixed height causes the titled border insets to reserve a majority of the > height available for the component. This fixed height is meant for the > default AquaComboBox usage, so the old default height is more applicable when > a border, such as TitledBorder, is set on an Aqua Editable

Re: RFR: 8300269: The selected item in an editable JComboBox with titled border is not visible in Aqua LAF [v2]

2023-01-20 Thread Damon Nguyen
On Fri, 20 Jan 2023 06:41:46 GMT, Prasanta Sadhukhan wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add test > > test/jdk/javax/swing/JComboBox/JComboBoxWithTitledBorderTest.java line 168: > >> 166: try

Re: RFR: 8300731: Avoid unnecessary array fill after creation in PaletteBuilder

2023-01-20 Thread Sergey Bylokhov
On Thu, 19 Jan 2023 19:32:05 GMT, Andrey Turbanov wrote: > No need to fill elements of array with `null`: it was just created. Java > guarantees that all elements of array are `null`s anyway. Marked as reviewed by serb (Reviewer). - PR: https://git.openjdk.org/jdk/pull/12101

Re: RFR: 8300269: The selected item in an editable JComboBox with titled border is not visible in Aqua LAF [v2]

2023-01-20 Thread Damon Nguyen
On Fri, 20 Jan 2023 06:41:36 GMT, Prasanta Sadhukhan wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add test > > src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxUI.java line 460: > >> 458:

Re: RFR: 8299333: Unify exceptions used by all variants of ICC_Profile.getInstance(null)

2023-01-20 Thread Phil Race
On Fri, 20 Jan 2023 20:16:09 GMT, Sergey Bylokhov wrote: > Something like this? Yes. If you file the CSR I will review it. But I also want to run a test job to see if there are any tests that need adjusting or excluding. - PR: https://git.openjdk.org/jdk/pull/11784

Re: RFR: 8299333: Unify exceptions used by all variants of ICC_Profile.getInstance(null)

2023-01-20 Thread Sergey Bylokhov
On Fri, 23 Dec 2022 23:32:45 GMT, Sergey Bylokhov wrote: > I tried to clean up the documentation in the `java.awt.color` package and > specify how the `null` parameters are handled here and there. But it looks > like the `ICC_profile` class is too specific so I would like to discuss it >

Re: RFR: 8299333: Unify exceptions used by all variants of ICC_Profile.getInstance(null) [v2]

2023-01-20 Thread Sergey Bylokhov
> I tried to clean up the documentation in the `java.awt.color` package and > specify how the `null` parameters are handled here and there. But it looks > like the `ICC_profile` class is too specific so I would like to discuss it > separately. > > The ICC_Profile has these methods: > *

Re: RFR: 8300725: Improve performance of ColorConvertOp for default destinations with alpha

2023-01-20 Thread Phil Race
On Fri, 20 Jan 2023 06:02:53 GMT, Sergey Bylokhov wrote: > The "default destination" for the ColorConverOp is used when the user passes > "null" as the DST parameter. In this case, the "ColorConverOp.filter" creates > the default image and uses ComponentColorModel: 8 bits per color component

Withdrawn: 8298266: "java.home property not set" error in Graal when sun.awt.fontconfig java property is set on Windows

2023-01-20 Thread Alexander Scherbatiy
On Wed, 7 Dec 2022 13:20:10 GMT, Alexander Scherbatiy wrote: > **Environment**: > GraalVM 22.3.0 with jdk 19, Windows OS. > > **Description**: > Create a native image of Swing application using GraalVM and run it with ` > -Dsun.awt.fontconfig=fontconfig.properties.src` java property. >

Re: RFR: 8300731: Avoid unnecessary array fill after creation in PaletteBuilder

2023-01-20 Thread Phil Race
On Thu, 19 Jan 2023 19:32:05 GMT, Andrey Turbanov wrote: > No need to fill elements of array with `null`: it was just created. Java > guarantees that all elements of array are `null`s anyway. Marked as reviewed by prr (Reviewer). - PR: https://git.openjdk.org/jdk/pull/12101

Re: RFR: 8300725: Improve performance of ColorConverOp for default destinations with alpha

2023-01-20 Thread Phil Race
On Fri, 20 Jan 2023 06:02:53 GMT, Sergey Bylokhov wrote: > The "default destination" for the ColorConverOp is used when the user passes > "null" as the DST parameter. In this case, the "ColorConverOp.filter" creates > the default image and uses ComponentColorModel: 8 bits per color component

RFR: 8300725: Improve performance of ColorConverOp for default destinations with alpha

2023-01-20 Thread Sergey Bylokhov
The "default destination" for the ColorConverOp is used when the user passes "null" as the DST parameter. In this case, the "ColorConverOp.filter" creates the default image and uses ComponentColorModel: 8 bits per color component and optionally 8 bits for the alpha channel. For example for the

Withdrawn: 8280359: Try to remove duplicate code for coordinate calculation

2023-01-20 Thread Sergey Bylokhov
On Mon, 12 Dec 2022 01:49:23 GMT, Sergey Bylokhov wrote: > The code from `AccessibilityGraphicsEnvironment` is moved/merged back to the > `SunGraphicsEnvironment` from where it was initially > [copied](https://github.com/openjdk/jdk18/pull/72#issuecomment-1001050200). > The required API is

Re: RFR: 8300405: Screen capture for test JFileChooserSetLocationTest.java, failure case

2023-01-20 Thread Sergey Bylokhov
On Fri, 20 Jan 2023 09:41:37 GMT, Tejesh R wrote: >> test/jdk/javax/swing/JFileChooser/JFileChooserSetLocationTest.java line 189: >> >>> 187: System.out.println("Test passed"); >>> 188: } else { >>> 189: Dimension screenSize = >>>

Re: RFR: 8299592: Fix and reenable warnings in java.desktop native code compilation [v2]

2023-01-20 Thread Daniel Jeliński
> Please review this patch that fixes and re-enables a few warnings in libawt > compilation. > > Verified that debug and release builds finish successfully on Win, Mac and > Linux. Also verified that client libs tests still pass. Daniel Jeliński has updated the pull request with a new target

Re: RFR: 8299592: Fix and reenable warnings in java.desktop native code compilation

2023-01-20 Thread Daniel Jeliński
On Wed, 4 Jan 2023 12:44:39 GMT, Daniel Jeliński wrote: > Please review this patch that fixes and re-enables a few warnings in libawt > compilation. > > Verified that debug and release builds finish successfully on Win, Mac and > Linux. Also verified that client libs tests still pass. In

Re: RFR: 8300405: Screen capture for test JFileChooserSetLocationTest.java, failure case

2023-01-20 Thread Tejesh R
On Fri, 20 Jan 2023 08:09:38 GMT, Sergey Bylokhov wrote: >> This is a diagnostic update related to >> [JDK-8295804](https://bugs.openjdk.org/browse/JDK-8295804). The issue was >> not reproducible and was showing up in particular machines (Where the >> machines are down now). So adding a

Re: RFR: 8300731: Avoid unnecessary array fill after creation in PaletteBuilder

2023-01-20 Thread Sergey Bylokhov
On Thu, 19 Jan 2023 19:32:05 GMT, Andrey Turbanov wrote: > No need to fill elements of array with `null`: it was just created. Java > guarantees that all elements of array are `null`s anyway. looks fine - PR: https://git.openjdk.org/jdk/pull/12101

Re: RFR: 8300731: Avoid unnecessary array fill after creation in PaletteBuilder

2023-01-20 Thread SWinxy
On Thu, 19 Jan 2023 19:32:05 GMT, Andrey Turbanov wrote: > No need to fill elements of array with `null`: it was just created. Java > guarantees that all elements of array are `null`s anyway. Marked as reviewed by swi...@github.com (no known OpenJDK username). - PR:

RFR: 8300731: Avoid unnecessary array fill after creation in PaletteBuilder

2023-01-20 Thread Andrey Turbanov
No need to fill elements of array with `null`: it was just created. Java guarantees that all elements of array are `null`s anyway. - Commit messages: - [PATCH] Avoid unnecessary array fill after creation Changes: https://git.openjdk.org/jdk/pull/12101/files Webrev:

Re: RFR: 8300405: Screen capture for test JFileChooserSetLocationTest.java, failure case

2023-01-20 Thread Sergey Bylokhov
On Wed, 18 Jan 2023 06:37:30 GMT, Tejesh R wrote: > This is a diagnostic update related to > [JDK-8295804](https://bugs.openjdk.org/browse/JDK-8295804). The issue was not > reproducible and was showing up in particular machines (Where the machines > are down now). So adding a screen capture