diff options
author | Ben Wagner <bungeman@google.com> | 2018-07-10 14:20:15 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-10 20:03:46 +0000 |
commit | 5d948228d91ae8ff2c9602ff45a9e8661665ed46 (patch) | |
tree | 5befeac595bc114a5854a37ab406e20dfd560cbd /src/core/SkReadBuffer.cpp | |
parent | 1994f20d79055b9279b68236176073cf07db293f (diff) |
Make SkTypefacePlayback use smart pointers.
This clarifies the ownership of the typefaces as well as removes a use
of SkRefCnt_SafeAssign which is currently a code smell.
Change-Id: I8fec541f71f555c2182b77870979ece87b501901
Reviewed-on: https://skia-review.googlesource.com/140249
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'src/core/SkReadBuffer.cpp')
-rw-r--r-- | src/core/SkReadBuffer.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkReadBuffer.cpp b/src/core/SkReadBuffer.cpp index 7e58e52e67..e1c9d02ec0 100644 --- a/src/core/SkReadBuffer.cpp +++ b/src/core/SkReadBuffer.cpp @@ -379,7 +379,7 @@ sk_sp<SkTypeface> SkReadBuffer::readTypeface() { if (!this->validate(index <= fTFCount)) { return nullptr; } - return sk_ref_sp(fTFArray[index - 1]); + return fTFArray[index - 1]; } else { // custom size_t size = sk_negate_to_size_t(index); const void* data = this->skip(size); |