add replace_char() method to SkString API (skbug.com/2651)

include/core/SkString.h

 
 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #ifndef SkString_DEFINED
@@ skipping 172 matching lines @@
     void appendScalar(SkScalar value) { this->insertScalar((size_t)-1, value); }
 
     void prepend(const SkString& str) { this->insert(0, str); }
     void prepend(const char text[]) { this->insert(0, text); }
     void prepend(const char text[], size_t len) { this->insert(0, text, len); }
     void prependUnichar(SkUnichar uni) { this->insertUnichar(0, uni); }
     void prependS32(int32_t value) { this->insertS32(0, value); }
     void prependS64(int32_t value, int minDigits = 0) { this->insertS64(0, value, minDigits); }
     void prependHex(uint32_t value, int minDigits = 0) { this->insertHex(0, value, minDigits); }
     void prependScalar(SkScalar value) { this->insertScalar((size_t)-1, value); }
 

[epoger, 2014/06/11 11:01:33]

Please add unittests in
https://skia.googlesource.com/skia/+/master/tests/StringTest.cpp

[rs.prinja, 2014/06/11 12:08:03]

Done.

+    /* Replace all occurrences of oldChar with newChar */
+    void replace_char(const char oldChar, const char newChar);

[epoger, 2014/06/11 11:01:33]

For consistency, please call this either replace() or replaceChar().  I'm not
sure which is better.

[rs.prinja, 2014/06/11 12:08:03]

I'm calling it replaceChar for the time being. Will reupload if replace() or
some other name sounds better to everyone.

+
     void printf(const char format[], ...) SK_PRINTF_LIKE(2, 3);
     void appendf(const char format[], ...) SK_PRINTF_LIKE(2, 3);
     void appendVAList(const char format[], va_list);
     void prependf(const char format[], ...) SK_PRINTF_LIKE(2, 3);
 
     void remove(size_t offset, size_t length);
 
     SkString& operator+=(const SkString& s) { this->append(s); return *this; }
     SkString& operator+=(const char text[]) { this->append(text); return *this; }
     SkString& operator+=(const char c) { this->append(&c, 1); return *this; }
@@ skipping 34 matching lines @@
 // Specialized to take advantage of SkString's fast swap path. The unspecialized function is
 // declared in SkTypes.h and called by SkTSort.
 template <> inline void SkTSwap(SkString& a, SkString& b) {
     a.swap(b);
 }
 
 // Split str on any characters in delimiters into out.  (Think, strtok with a sane API.)
 void SkStrSplit(const char* str, const char* delimiters, SkTArray<SkString>* out);
 
 #endif