On Thu, 3 Mar 2022 21:27:36 GMT, Phil Race <p...@openjdk.org> wrote: >> As per the bug eval, we are returning a pointer to a native raw LittleCMS >> profile and it should be a cooked one. >> The simple test fails before the fix with >> ~/jdk17/Contents/Home/bin/java SetTagDataValidation >> >> Exception in thread "main" java.awt.color.CMMException: LCMS error 13: >> Couldn't link the profiles >> at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native >> Method) >> at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104) >> at >> java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143) >> at >> java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633) >> at >> java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224) >> at SetTagDataValidation.main(SetTagDataValidation.java:61) >> >> and with the fix just prints >> java SetTagDataValidation >> 0.4 >> 0.5000076 >> 0.6 > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile
The current bug looks similar to this one https://bugs.openjdk.java.net/browse/JDK-8272865 I tried to fix it the same way but I was not sure if it is safe to ignore the ReadWriteProfileTest.java or not, since it has five related bugids. test/jdk/java/awt/color/ICC_Profile/MTGetData.java line 106: > 104: icc.setData(tag, data2); > 105: } > 106: } catch (CMMException cmmEx) { Don't we need to throw the IllegalArgumentException instead of CMMException? ------------- PR: https://git.openjdk.java.net/jdk/pull/7668
