diff options
| author |  halcanary <halcanary@google.com> | 2016-04-11 19:41:48 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-04-11 19:41:48 -0700 |
| commit | cbc060a70007bcdcc180d74b552f30104c88a805 (patch) | |
| tree | 8c1c9a9ca0796e27d1c8dc1e979acf522204dfd9 /src/core/SkStream.cpp | |
| parent | 74b8d323323c8533e3e5cc7719e0bd127aacd829 (diff) | |
SkWStream::writeText inlined.
Motivation: This function is used throughout SkPDF.
Note that the compiler can usually inline the result of strlen() for literal strings.
Before:
out/Release/nanobench -m WStreamWriteText -q
Timer overhead: 24.2ns
! -> high variance, ? -> moderate variance
micros bench
6.10 WStreamWriteText nonrendering
After:
out/Release/nanobench -m WStreamWriteText -q
Timer overhead: 23.9ns
! -> high variance, ? -> moderate variance
micros bench
2.51 WStreamWriteText nonrendering
PDF runtime change: -0.8% ±0.04%.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1844343004
Review URL: https://codereview.chromium.org/1844343004
Diffstat (limited to 'src/core/SkStream.cpp')
| -rw-r--r-- | src/core/SkStream.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/core/SkStream.cpp b/src/core/SkStream.cpp index 1d1c9b971f..6dbde4dc6d 100644 --- a/src/core/SkStream.cpp +++ b/src/core/SkStream.cpp @@ -78,12 +78,6 @@ void SkWStream::flush() { } -bool SkWStream::writeText(const char text[]) -{ - SkASSERT(text); - return this->write(text, strlen(text)); -} - bool SkWStream::writeDecAsText(int32_t dec) { char buffer[SkStrAppendS32_MaxSize]; |