diff options
author | 2017-05-11 10:59:22 -0400 | |
---|---|---|
committer | 2017-05-12 11:38:49 +0000 | |
commit | 5c7960be57010bf61db3d4ce879a3194687b5af9 (patch) | |
tree | 9ccdfd2230214173d19db29f206d1af2197f2787 /src/effects/gradients/SkSweepGradient.cpp | |
parent | df3a371c904c2e3e1d3d9201b7dfc0d080e5f12a (diff) |
refactor gradient stage names
This is just a name refactor and I'm happy to delay it until we're done
with the current wave of gradient CLs. The main ideas:
- we use the "linear_gradient" stages for all gradients,
so cut the "linear" and just call them "gradient";
- remind ourselves that the 2-stop stage requires even spacing, i.e.
stops at 0 and 1. This name should harmonize with Herb's new
general evenly spaced gradient stage, currently
"evenly_spaced_linear_gradient", and after it lands and I rebase,
"evenly_spaced_gradient"
- remind ourselves which polar coordinate xy_to_polar_unit returns,
the angle.
Change-Id: I0fd0c8bd4c1ead7d2d0fff45a199d318b71f34ac
Reviewed-on: https://skia-review.googlesource.com/16500
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Reed <reed@google.com>
Diffstat (limited to 'src/effects/gradients/SkSweepGradient.cpp')
-rw-r--r-- | src/effects/gradients/SkSweepGradient.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/effects/gradients/SkSweepGradient.cpp b/src/effects/gradients/SkSweepGradient.cpp index b3a24cd16e..df571e9ea8 100644 --- a/src/effects/gradients/SkSweepGradient.cpp +++ b/src/effects/gradients/SkSweepGradient.cpp @@ -315,7 +315,7 @@ bool SkSweepGradient::adjustMatrixAndAppendStages(SkArenaAlloc* alloc, SkMatrix* matrix, SkRasterPipeline* p) const { matrix->postTranslate(-fCenter.fX, -fCenter.fY); - p->append(SkRasterPipeline::xy_to_polar_unit); + p->append(SkRasterPipeline::xy_to_unit_angle); return true; } |