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

Sergey Bylokhov serb at openjdk.org
Tue Jan 2 05:34:51 UTC 2024


On Tue, 2 Jan 2024 03:53:06 GMT, Renjith Kannath Pariyangad <rkannathpari at openjdk.org> wrote:

>> 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 one additional commit since the last revision:
> 
>   Removed unnecessary try block

test/jdk/java/awt/color/NonICCFilterTest.java line 47:

> 45: 
> 46:         protected TestColorSpace(boolean bSrc) {
> 47:             super(CS_sRGB, 3);

CS_sRGB is not a constant of the ColorSpace type, the ICC_Profile.getInstance(xx).getColorSpaceType() and profile.getNumComponents() should be used.

test/jdk/java/awt/color/NonICCFilterTest.java line 53:

> 51: 
> 52:         public float[] toRGB(float[] colorvalue) {
> 53:             return colorvalue;

Just for completeness, it is probably better to use csRGB.toRGB and csRGB.fromRGB in these two methods

test/jdk/java/awt/color/NonICCFilterTest.java line 109:

> 107:             throw new RuntimeException("Test failed: Source equal to Destination");
> 108:     }
> 109: }

The "}" is missing in the latest version.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16895#discussion_r1439162717
PR Review Comment: https://git.openjdk.org/jdk/pull/16895#discussion_r1439163073
PR Review Comment: https://git.openjdk.org/jdk/pull/16895#discussion_r1439163248


More information about the client-libs-dev mailing list