diff options
author | Mike Klein <mtklein@chromium.org> | 2018-07-11 14:49:51 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-11 20:14:38 +0000 |
commit | 4eebd9eed06039d265f06edb759765731f963271 (patch) | |
tree | a60d5bf110250ad75593977b39fdf93a019c1fff /src/core/SkConvertPixels.cpp | |
parent | c0a27081722eaebd005cf9c2eb78d7d1e4db7e2c (diff) |
collapse parametric_{r,g,b} into parametric, remove _a and gamma_dst
parametric_a and gamma_dst were unused outside of unit tests.
In all other cases, we always use parametric_{r,g,b} together
and always pass them the same argument. So we can collapse
them into a single stage like gamma and to/from_srgb.
Change-Id: I08cea896c7744f97b4f4bf9e029f5d643e45e177
Reviewed-on: https://skia-review.googlesource.com/140576
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Brian Osman <brianosman@google.com>
Diffstat (limited to 'src/core/SkConvertPixels.cpp')
-rw-r--r-- | src/core/SkConvertPixels.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/core/SkConvertPixels.cpp b/src/core/SkConvertPixels.cpp index 25645d485c..0434b92c2e 100644 --- a/src/core/SkConvertPixels.cpp +++ b/src/core/SkConvertPixels.cpp @@ -308,9 +308,7 @@ static void convert_with_pipeline(const SkImageInfo& dstInfo, void* dstRow, size if (is_just_gamma(srcFn)) { pipeline.append(SkRasterPipeline::gamma, &srcFn.fG); } else { - pipeline.append(SkRasterPipeline::parametric_r, &srcFn); - pipeline.append(SkRasterPipeline::parametric_g, &srcFn); - pipeline.append(SkRasterPipeline::parametric_b, &srcFn); + pipeline.append(SkRasterPipeline::parametric, &srcFn); } } @@ -340,9 +338,7 @@ static void convert_with_pipeline(const SkImageInfo& dstInfo, void* dstRow, size if (is_just_gamma(dstFn)) { pipeline.append(SkRasterPipeline::gamma, &dstFn.fG); } else { - pipeline.append(SkRasterPipeline::parametric_r, &dstFn); - pipeline.append(SkRasterPipeline::parametric_g, &dstFn); - pipeline.append(SkRasterPipeline::parametric_b, &dstFn); + pipeline.append(SkRasterPipeline::parametric, &dstFn); } } |