diff options
author | mtklein <mtklein@chromium.org> | 2015-03-30 10:50:27 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-03-30 10:50:27 -0700 |
commit | c9adb05b64fa0bfadf9d1a782afcda470da68c9e (patch) | |
tree | 6413cc149b70ae36181e9f0789246b9db24447f0 /include/effects | |
parent | 23ac62c83a49d675a38f1c20462b5537f3c8af01 (diff) |
Refactor Sk2x<T> + Sk4x<T> into SkNf<N,T> and SkNi<N,T>
The primary feature this delivers is SkNf and SkNd for arbitrary power-of-two N. Non-specialized types or types larger than 128 bits should now Just Work (and we can drop in a specialization to make them faster). Sk4s is now just a typedef for SkNf<4, SkScalar>; Sk4d is SkNf<4, double>, Sk2f SkNf<2, float>, etc.
This also makes implementing new specializations easier and more encapsulated. We're now using template specialization, which means the specialized versions don't have to leak out so much from SkNx_sse.h and SkNx_neon.h.
This design leaves us room to grow up, e.g to SkNf<8, SkScalar> == Sk8s, and to grown down too, to things like SkNi<8, uint16_t> == Sk8h.
To simplify things, I've stripped away most APIs (swizzles, casts, reinterpret_casts) that no one's using yet. I will happily add them back if they seem useful.
You shouldn't feel bad about using any of the typedef Sk4s, Sk4f, Sk4d, Sk2s, Sk2f, Sk2d, Sk4i, etc. Here's how you should feel:
- Sk4f, Sk4s, Sk2d: feel awesome
- Sk2f, Sk2s, Sk4d: feel pretty good
No public API changes.
TBR=reed@google.com
BUG=skia:3592
Review URL: https://codereview.chromium.org/1048593002
Diffstat (limited to 'include/effects')
-rw-r--r-- | include/effects/SkColorMatrixFilter.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/effects/SkColorMatrixFilter.h b/include/effects/SkColorMatrixFilter.h index 27a2be835b..47092739f2 100644 --- a/include/effects/SkColorMatrixFilter.h +++ b/include/effects/SkColorMatrixFilter.h @@ -45,7 +45,7 @@ protected: private: SkColorMatrix fMatrix; - float fTranspose[SkColorMatrix::kCount]; // for Sk4f + float fTranspose[SkColorMatrix::kCount]; // for Sk4s typedef void (*Proc)(const State&, unsigned r, unsigned g, unsigned b, unsigned a, int32_t result[4]); |