From eeb3cb1286c5ca8104da62fb36b07bb0fe04aee5 Mon Sep 17 00:00:00 2001 From: Matt Sarett Date: Wed, 29 Mar 2017 20:00:18 -0400 Subject: [PATCH] Reverse clamp logic in SkColorSpaceXform Follow-up from https://skia-review.googlesource.com/c/10581/ Bug: skia: Change-Id: I499f0908e1375854202c1cfd476827500fc89237 Reviewed-on: https://skia-review.googlesource.com/10645 Reviewed-by: Mike Klein Commit-Queue: Mike Klein --- src/core/SkColorSpaceXform.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/core/SkColorSpaceXform.cpp b/src/core/SkColorSpaceXform.cpp index 82b442b..52a6851 100644 --- a/src/core/SkColorSpaceXform.cpp +++ b/src/core/SkColorSpaceXform.cpp @@ -1227,9 +1227,8 @@ bool SkColorSpaceXform_XYZ if (kNone_ColorSpaceMatch == kCSM) { pipeline.append(SkRasterPipeline::matrix_3x4, fSrcToDst); - if (kRGBA_8888_ColorFormat == dstColorFormat || - kBGRA_8888_ColorFormat == dstColorFormat || - kBGR_565_ColorFormat == dstColorFormat) + if (kRGBA_F16_ColorFormat != dstColorFormat && + kRGBA_F32_ColorFormat != dstColorFormat) { bool need_clamp_0, need_clamp_1; analyze_3x4_matrix(fSrcToDst, &need_clamp_0, &need_clamp_1); -- 2.7.4