diff options
author | benjaminwagner <benjaminwagner@google.com> | 2016-02-24 06:51:52 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-24 06:51:52 -0800 |
commit | 7ea5cb18389db11a94175de95c9db2b44972630c (patch) | |
tree | 3e87b6ec8c453081a1666fca96dfb5bfbbda2141 /src/core/SkPaint.cpp | |
parent | 8f7b0b2d809510d4af4e6ff6f731bac78eded6c4 (diff) |
Cleanups related to SkFixed.
Remove SK_FixedNaN. It does not seem to be supported or used anywhere in Skia, Chromium, Android, or Google3, (except accidentally by TwoPtRadial::kDontDrawT). I think supporting it would erase any benefit of SkFixed over float.
Remove SkBitmapFilter::lookup. It does not appear to be used anywhere in Skia, Chromium, Android, or Google3.
Fix a bug in SkPaint::breakText that limited it to ~5kB of text. Now it can handle more than 1GB.
BUG=skia:4632
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1683743005
Review URL: https://codereview.chromium.org/1683743005
Diffstat (limited to 'src/core/SkPaint.cpp')
-rw-r--r-- | src/core/SkPaint.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkPaint.cpp b/src/core/SkPaint.cpp index d3384a628d..638e251b36 100644 --- a/src/core/SkPaint.cpp +++ b/src/core/SkPaint.cpp @@ -925,7 +925,7 @@ size_t SkPaint::breakText(const void* textD, size_t length, SkScalar maxWidth, GlyphCacheProc glyphCacheProc = paint.getGlyphCacheProc(false); const int xyIndex = paint.isVerticalText() ? 1 : 0; // use 64bits for our accumulator, to avoid overflowing 16.16 - Sk48Dot16 max = SkScalarToFixed(maxWidth); + Sk48Dot16 max = SkScalarTo48Dot16(maxWidth); Sk48Dot16 width = 0; SkAutoKern autokern; |