diff options
author | Nicolas Pena <npm@chromium.org> | 2017-07-12 14:45:08 -0400 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-07-12 18:58:24 +0000 |
commit | d79ac21b7595442bce82ec93f59cda9f8fb3cb24 (patch) | |
tree | 2c479141e360094a21cad7a1566a0c1a04e8a9c9 /core/fxcodec/codec/fx_codec_jpx_unittest.cpp | |
parent | 58bf3de718e510648ee136c887be5b2d577644a7 (diff) | |
download | pdfium-d79ac21b7595442bce82ec93f59cda9f8fb3cb24.tar.xz |
Move CMYKtoRGB methods to fx_codec and clean them up
This CL moves CMYKtoRGB methods to fx_codec. It also cleans them up a
bit, including allowing them to return tuples instead of having
non-const refs in their parameters.
Change-Id: Ib3ec45102ec7eff623cd07a624e852d39bf335e4
Reviewed-on: https://pdfium-review.googlesource.com/7591
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core/fxcodec/codec/fx_codec_jpx_unittest.cpp')
-rw-r--r-- | core/fxcodec/codec/fx_codec_jpx_unittest.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fxcodec/codec/fx_codec_jpx_unittest.cpp b/core/fxcodec/codec/fx_codec_jpx_unittest.cpp index 6d1e31a0ee..7e2510546d 100644 --- a/core/fxcodec/codec/fx_codec_jpx_unittest.cpp +++ b/core/fxcodec/codec/fx_codec_jpx_unittest.cpp @@ -34,10 +34,10 @@ TEST(fxcodec, CMYK_Rounding) { // Iterate through floats by incrementing the representation, as discussed in // https://randomascii.wordpress.com/2012/01/23/stupid-float-tricks-2/ for (Float_t f = startValue; f.f < endValue; f.i++) { - AdobeCMYK_to_sRGB(f.f, f.f, f.f, f.f, R, G, B); + std::tie(R, G, B) = AdobeCMYK_to_sRGB(f.f, f.f, f.f, f.f); } // Check various other 'special' numbers. - AdobeCMYK_to_sRGB(0.0f, 0.25f, 0.5f, 1.0f, R, G, B); + std::tie(R, G, B) = AdobeCMYK_to_sRGB(0.0f, 0.25f, 0.5f, 1.0f); } TEST(fxcodec, DecodeDataNullDecodeData) { |