On Fri, 12 Jan 2024 02:55:59 GMT, Sergey Bylokhov <[email protected]> wrote:
>> @mrserb , >> >> Did you mean to use the wrapper for the middle like `ColorSpace mid = >> createCS(ColorSpaceSelector.WRAPPED_PYCC);` , So we can achieve : >> >> **From** >> wrapper->icc_color_space->wrapper >> icc_color_space->icc_color_space->icc_color_space >> >> **To** : >> wrapper->wrapper->wrapper >> icc_color_space->wrapper->icc_color_space >> >> Correct me if I am wrong > >>Did you mean to use the wrapper for the middle like ColorSpace mid = >>createCS(ColorSpaceSelector.WRAPPED_PYCC); , So we can achieve : > > Just repeat existed checks using wrapper. So you will have all combinations: > >> wrapper->icc_color_space->wrapper >> icc_color_space->icc_color_space->icc_color_space >> wrapper->wrapper->wrapper >> icc_color_space->wrapper->icc_color_space Thank you, @mrserb, for the valuable review, the fix and the test have become more comprehensive as the result. Greatly appreciated! ------------- PR Comment: https://git.openjdk.org/jdk/pull/16895#issuecomment-1898170895
