diff options
author | halcanary <halcanary@google.com> | 2016-04-22 08:03:03 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-04-22 08:03:03 -0700 |
commit | 606cadd5aac62299ef2e277709b3684cae2bf96c (patch) | |
tree | 40c5d32fc577c2eec86e6215134b17ea29e12cba /src/core/SkString.cpp | |
parent | ada338ab4edbe0ac644663aca6da205122fa5dac (diff) |
SkStringPrintf and SkString::printf now are no longer limted by a static buffer
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1403803002
Review URL: https://codereview.chromium.org/1403803002
Diffstat (limited to 'src/core/SkString.cpp')
-rw-r--r-- | src/core/SkString.cpp | 61 |
1 files changed, 52 insertions, 9 deletions
diff --git a/src/core/SkString.cpp b/src/core/SkString.cpp index 24b1b8fb62..9331665dcb 100644 --- a/src/core/SkString.cpp +++ b/src/core/SkString.cpp @@ -33,6 +33,56 @@ static const size_t kBufferSize = 1024; va_end(args); \ } while (0) +#ifdef SK_BUILD_FOR_WIN +#define V_SKSTRING_PRINTF(output, format) \ + do { \ + va_list args; \ + va_start(args, format); \ + char buffer[kBufferSize]; \ + int length = _vsnprintf_s(buffer, sizeof(buffer), \ + _TRUNCATE, format, args); \ + va_end(args); \ + if (length >= 0 && length < (int)sizeof(buffer)) { \ + output.set(buffer, length); \ + break; \ + } \ + va_start(args, format); \ + length = _vscprintf(format, args); \ + va_end(args); \ + output.resize((size_t)length); \ + va_start(args, format); \ + SkDEBUGCODE(int check = ) _vsnprintf_s(output.writable_str(), \ + length + 1, _TRUNCATE, \ + format, args); \ + va_end(args); \ + SkASSERT(check == length); \ + SkASSERT(output[length] == '\0'); \ + } while (false) +#else +#define V_SKSTRING_PRINTF(output, format) \ + do { \ + va_list args; \ + va_start(args, format); \ + char buffer[kBufferSize]; \ + int length = vsnprintf(buffer, sizeof(buffer), format, args); \ + va_end(args); \ + if (length < 0) { \ + break; \ + } \ + if (length < (int)sizeof(buffer)) { \ + output.set(buffer, length); \ + break; \ + } \ + output.resize((size_t)length); \ + va_start(args, format); \ + SkDEBUGCODE(int check = ) vsnprintf(output.writable_str(), \ + length + 1, format, args); \ + va_end(args); \ + SkASSERT(check == length); \ + SkASSERT(output[length] == '\0'); \ + } while (false) +#endif + /////////////////////////////////////////////////////////////////////////////// bool SkStrEndsWith(const char string[], const char suffixStr[]) { @@ -513,11 +563,7 @@ void SkString::insertScalar(size_t offset, SkScalar value) { } void SkString::printf(const char format[], ...) { - char buffer[kBufferSize]; - int length; - ARGS_TO_BUFFER(format, buffer, kBufferSize, length); - - this->set(buffer, length); + V_SKSTRING_PRINTF((*this), format); } void SkString::appendf(const char format[], ...) { @@ -593,10 +639,7 @@ void SkString::swap(SkString& other) { SkString SkStringPrintf(const char* format, ...) { SkString formattedOutput; - char buffer[kBufferSize]; - SK_UNUSED int length; - ARGS_TO_BUFFER(format, buffer, kBufferSize, length); - formattedOutput.set(buffer); + V_SKSTRING_PRINTF(formattedOutput, format); return formattedOutput; } |