diff options
author | Mike Klein <mtklein@chromium.org> | 2017-06-01 12:37:08 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-06-01 17:05:13 +0000 |
commit | 761d27c4d76bbd553c10cfe835d572b6fa33cf26 (patch) | |
tree | 96a4dab9eb24960554544fe5e359dd903fce9a55 /tests | |
parent | d16084ffdc688bad6d0e9c04f3f98049b3412633 (diff) |
update SkRasterPipeline::run() to also take y
y isn't used yet. This is just a warmup that updates the callers.
Change-Id: I78f4f44e2b82f72b3a39fa8a8bdadef1d1b8a99e
Reviewed-on: https://skia-review.googlesource.com/18381
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/F16StagesTest.cpp | 4 | ||||
-rw-r--r-- | tests/ParametricStageTest.cpp | 2 | ||||
-rw-r--r-- | tests/SRGBTest.cpp | 4 | ||||
-rw-r--r-- | tests/SkRasterPipelineTest.cpp | 14 |
4 files changed, 12 insertions, 12 deletions
diff --git a/tests/F16StagesTest.cpp b/tests/F16StagesTest.cpp index cdf2daf09a..51cb861c89 100644 --- a/tests/F16StagesTest.cpp +++ b/tests/F16StagesTest.cpp @@ -25,7 +25,7 @@ DEF_TEST(F16Stages, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline:: load_f32, &f32); p.append(SkRasterPipeline::store_f16, &f16); - p.run(0,16/4); + p.run(0,0,16/4); } REPORTER_ASSERT(r, f16[0] == 0x0000); REPORTER_ASSERT(r, f16[1] == 0x3400); @@ -40,7 +40,7 @@ DEF_TEST(F16Stages, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline:: load_f16, &f16); p.append(SkRasterPipeline::store_f32, &f32); - p.run(0,16/4); + p.run(0,0,16/4); } REPORTER_ASSERT(r, f32[0] == 0.00f); REPORTER_ASSERT(r, f32[1] == 0.25f); diff --git a/tests/ParametricStageTest.cpp b/tests/ParametricStageTest.cpp index 6edb9d9473..5b34e166c9 100644 --- a/tests/ParametricStageTest.cpp +++ b/tests/ParametricStageTest.cpp @@ -27,7 +27,7 @@ static void check_error(skiatest::Reporter* r, float limit, SkColorSpaceTransfer p.append(SkRasterPipeline::parametric_a, &fn); p.append(SkRasterPipeline::store_f32, &op); - p.run(0, 256/4); + p.run(0,0, 256/4); for (int i = 0; i < 256; i++) { diff --git a/tests/SRGBTest.cpp b/tests/SRGBTest.cpp index b1e4570f50..4a519ca55b 100644 --- a/tests/SRGBTest.cpp +++ b/tests/SRGBTest.cpp @@ -54,7 +54,7 @@ DEF_TEST(sk_pipeline_srgb_roundtrip, r) { p.append(SkRasterPipeline::to_srgb); p.append(SkRasterPipeline::store_8888, &ptr); - p.run(0,256); + p.run(0,0,256); for (int i = 0; i < 256; i++) { if (reds[i] != (uint32_t)i) { @@ -73,7 +73,7 @@ DEF_TEST(sk_pipeline_srgb_edge_cases, r) { p.append(SkRasterPipeline::constant_color, &color); p.append(SkRasterPipeline::to_srgb); p.append(SkRasterPipeline::store_f32, &dst); - p.run(0,4); + p.run(0,0,4); if (color.r() != 0.0f) { ERRORF(r, "expected to_srgb() to map 0.0f to 0.0f, got %f", color.r()); diff --git a/tests/SkRasterPipelineTest.cpp b/tests/SkRasterPipelineTest.cpp index 1af40fd5f6..f65a2c9868 100644 --- a/tests/SkRasterPipelineTest.cpp +++ b/tests/SkRasterPipelineTest.cpp @@ -27,7 +27,7 @@ DEF_TEST(SkRasterPipeline, r) { p.append(SkRasterPipeline::swap); p.append(SkRasterPipeline::srcover); p.append(SkRasterPipeline::store_f16, &store_ctx); - p.run(0,1); + p.run(0,0,1); // We should see half-intensity magenta. REPORTER_ASSERT(r, ((result >> 0) & 0xffff) == 0x3800); @@ -39,7 +39,7 @@ DEF_TEST(SkRasterPipeline, r) { DEF_TEST(SkRasterPipeline_empty, r) { // No asserts... just a test that this is safe to run. SkRasterPipeline_<256> p; - p.run(0,20); + p.run(0,0,20); } DEF_TEST(SkRasterPipeline_nonsense, r) { @@ -47,7 +47,7 @@ DEF_TEST(SkRasterPipeline_nonsense, r) { // srcover() calls st->next(); this makes sure we've always got something there to call. SkRasterPipeline_<256> p; p.append(SkRasterPipeline::srcover); - p.run(0,20); + p.run(0,0,20); } DEF_TEST(SkRasterPipeline_JIT, r) { @@ -69,7 +69,7 @@ DEF_TEST(SkRasterPipeline_JIT, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline:: load_8888, &src); p.append(SkRasterPipeline::store_8888, &dst); - p.run(15, 20); + p.run(15,0, 20); for (int i = 0; i < 36; i++) { if (i < 15 || i == 35) { @@ -120,7 +120,7 @@ DEF_TEST(SkRasterPipeline_tail, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline::load_f32, &src); p.append(SkRasterPipeline::store_f32, &dst); - p.run(0, i); + p.run(0,0, i); for (unsigned j = 0; j < i; j++) { for (unsigned k = 0; k < 4; k++) { if (buffer[j][k] != data[j][k]) { @@ -152,7 +152,7 @@ DEF_TEST(SkRasterPipeline_tail, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline::load_f16, &src); p.append(SkRasterPipeline::store_f16, &dst); - p.run(0, i); + p.run(0,0, i); for (unsigned j = 0; j < i; j++) { REPORTER_ASSERT(r, !memcmp(&data[j][0], &buffer[j][0], sizeof(buffer[j]))); @@ -189,7 +189,7 @@ DEF_TEST(SkRasterPipeline_tail, r) { SkRasterPipeline_<256> p; p.append(SkRasterPipeline::load_rgb_u16_be, &src); p.append(SkRasterPipeline::store_f32, &dst); - p.run(0, i); + p.run(0,0, i); for (unsigned j = 0; j < i; j++) { for (unsigned k = 0; k < 4; k++) { if (buffer[j][k] != answer[j][k]) { |