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

Renjith Kannath Pariyangad rkannathpari at openjdk.org
Fri Jan 12 09:53:47 UTC 2024


On Fri, 12 Jan 2024 02:55:59 GMT, Sergey Bylokhov <serb at openjdk.org> 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

@mrserb thank you for clarifying, repeated the check for covering all cases please review.

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

PR Comment: https://git.openjdk.org/jdk/pull/16895#issuecomment-1888770449


More information about the client-libs-dev mailing list