diff options
author | Ben Wagner <bungeman@google.com> | 2017-08-16 10:53:04 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-08-16 15:37:54 +0000 |
commit | b4aab9ae6d27c446af8302b79d15b832c816c633 (patch) | |
tree | 27e7f87a7a5e3d61990bc2742d9b9f167d26e0c3 /src/core/SkFindAndPlaceGlyph.h | |
parent | edf3dc094b0fc55bfc932539da1127484051b474 (diff) |
Replace SkFAIL with SK_ABORT.
SkFAIL is a legacy macro which is just SK_ABORT. This CL mechanically
changes uses of SkFAIL to SK_ABORT in preparation for its removal. The
related sk_throw macro will be changed independently, due to needing to
actually clean up its users.
Change-Id: Id70b5c111a02d2458dc60c8933f444df27d9cebb
Reviewed-on: https://skia-review.googlesource.com/35284
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'src/core/SkFindAndPlaceGlyph.h')
-rw-r--r-- | src/core/SkFindAndPlaceGlyph.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/core/SkFindAndPlaceGlyph.h b/src/core/SkFindAndPlaceGlyph.h index 5d7c46bf27..a11c4ecc7d 100644 --- a/src/core/SkFindAndPlaceGlyph.h +++ b/src/core/SkFindAndPlaceGlyph.h @@ -150,7 +150,7 @@ private: case SkPaint::kGlyphID_TextEncoding: return arena->make<GlyphIdGlyphFinder>(cache); } - SkFAIL("Should not get here."); + SK_ABORT("Should not get here."); return nullptr; } @@ -262,7 +262,7 @@ private: SkFloatToScalar(glyph.fAdvanceY)}; } // Even though the entire enum is covered above, MVSC doesn't think so. Make it happy. - SkFAIL("Should never get here."); + SK_ABORT("Should never get here."); return {0.0f, 0.0f}; } @@ -281,7 +281,7 @@ private: case kNone_SkAxisAlignment: return {kSubpixelRounding, kSubpixelRounding}; } - SkFAIL("Should not get here."); + SK_ABORT("Should not get here."); return {0.0f, 0.0f}; } @@ -300,7 +300,7 @@ private: return {SkScalarToFixed(SkScalarFraction(position.fX) + kSubpixelRounding), SkScalarToFixed(SkScalarFraction(position.fY) + kSubpixelRounding)}; } - SkFAIL("Should not get here."); + SK_ABORT("Should not get here."); return {0, 0}; } @@ -325,7 +325,7 @@ private: // See GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60277 virtual SkPoint findAndPositionGlyph( const char** text, SkPoint position, ProcessOneGlyph&& processOneGlyph) { - SkFAIL("Should never get here."); + SK_ABORT("Should never get here."); return {0.0f, 0.0f}; } }; @@ -435,7 +435,7 @@ private: return arena->make<GlyphFindAndPlaceSubpixel< ProcessOneGlyph, kTextAlignment, kY_SkAxisAlignment>>(glyphFinder); } - SkFAIL("Should never get here."); + SK_ABORT("Should never get here."); return nullptr; } |