Do not write a CICP tag for gray color profiles

The ICC version 4.4 specification states that the CICP tag is only supported for RGB, YCbCr or XYZ color profiles. Commit 9c9f559 modified the wrong SetCICPTag call.
0xC0000054 · Nov 19, 2022 · 07d982a · 07d982a
1 parent db1e5d5
commit 07d982a
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/src/common/ColorProfileGeneration.cpp b/src/common/ColorProfileGeneration.cpp
@@ -298,7 +298,6 @@ void SetIccProfileFromNclx(FormatRecord* formatRecord, const heif_color_profile_
 
                 if (profile)
                 {
-                    SetCICPTag(profile.get(), primaries, transferCharacteristics, matrixCoefficients, fullRange);
                     SaveColorProfileToHandle(profile.get(), formatRecord);
                 }
             }
@@ -412,12 +411,7 @@ void SetIccProfileFromNclx(FormatRecord* formatRecord, const heif_color_profile_
 
             if (profile)
             {
-                // The ICC version 4.4 specification states that the
-                // CICP tag is not supported for gray color profiles.
-                if (!IsMonochromeImage(formatRecord))
-                {
-                    SetCICPTag(profile.get(), primaries, transferCharacteristics, matrixCoefficients, fullRange);
-                }
+                SetCICPTag(profile.get(), primaries, transferCharacteristics, matrixCoefficients, fullRange);
                 SaveColorProfileToHandle(profile.get(), formatRecord);
             }
         }