Use CFX_RetainPtr to ref count CFX_ByteString

core/fxcrt/include/fx_string.h

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #ifndef CORE_FXCRT_INCLUDE_FX_STRING_H_
 #define CORE_FXCRT_INCLUDE_FX_STRING_H_
 
 #include <stdint.h>  // For intptr_t.
 #include <algorithm>
 
+#include "core/fxcrt/include/cfx_retain_ptr.h"
 #include "core/fxcrt/include/fx_memory.h"
 #include "core/fxcrt/include/fx_system.h"
 
 class CFX_BinaryBuf;
 class CFX_ByteString;
 class CFX_WideString;
 
 // An immutable string with caller-provided storage which must outlive the
 // string itself.
 class CFX_ByteStringC {
@@ skipping 62 matching lines @@
            FXSYS_memcmp(other.m_Ptr, m_Ptr, m_Length) == 0;
   }
   bool operator!=(const char* ptr) const { return !(*this == ptr); }
   bool operator!=(const CFX_ByteStringC& other) const {
     return !(*this == other);
   }
 
   uint32_t GetID(FX_STRSIZE start_pos = 0) const;
 
   const uint8_t* GetPtr() const { return m_Ptr; }
-
   const FX_CHAR* GetCStr() const { return (const FX_CHAR*)m_Ptr; }
 
   FX_STRSIZE GetLength() const { return m_Length; }
-
   bool IsEmpty() const { return m_Length == 0; }
 
   uint8_t GetAt(FX_STRSIZE index) const { return m_Ptr[index]; }
 
   CFX_ByteStringC Mid(FX_STRSIZE index, FX_STRSIZE count = -1) const {
     if (index < 0) {
       index = 0;
     }
     if (index > m_Length) {
       return CFX_ByteStringC();
@@ skipping 25 matching lines @@
   return rhs != lhs;
 }
 #define FXBSTR_ID(c1, c2, c3, c4) ((c1 << 24) | (c2 << 16) | (c3 << 8) | (c4))
 
 // A mutable string with shared buffers using copy-on-write semantics that
 // avoids the cost of std::string's iterator stability guarantees.
 class CFX_ByteString {
  public:
   typedef FX_CHAR value_type;
 
-  CFX_ByteString() : m_pData(nullptr) {}
-
-  // Copy constructor.
-  CFX_ByteString(const CFX_ByteString& str);
-
-  // Move constructor.
-  inline CFX_ByteString(CFX_ByteString&& other) {
-    m_pData = other.m_pData;
-    other.m_pData = nullptr;
-  }
+  CFX_ByteString() {}
+  CFX_ByteString(const CFX_ByteString& other) : m_pData(other.m_pData) {}
+  CFX_ByteString(CFX_ByteString&& other) { m_pData.Swap(other.m_pData); }
 
   CFX_ByteString(char ch);
   CFX_ByteString(const FX_CHAR* ptr)
       : CFX_ByteString(ptr, ptr ? FXSYS_strlen(ptr) : 0) {}
 
   CFX_ByteString(const FX_CHAR* ptr, FX_STRSIZE len);
   CFX_ByteString(const uint8_t* ptr, FX_STRSIZE len);
 
   CFX_ByteString(const CFX_ByteStringC& bstrc);
   CFX_ByteString(const CFX_ByteStringC& bstrc1, const CFX_ByteStringC& bstrc2);
 
   ~CFX_ByteString();
 
+  void Empty() { m_pData.Reset(); }

[Lei Zhang, 2016/03/30 00:15:33]

I hope we can rename this to clear() to be closer to std::string.

[Tom Sepez, 2016/03/30 18:49:18]

Added clear().

+
   static CFX_ByteString FromUnicode(const FX_WCHAR* ptr, FX_STRSIZE len = -1);
-
   static CFX_ByteString FromUnicode(const CFX_WideString& str);
 
   // Explicit conversion to C-style string.
   const FX_CHAR* c_str() const { return m_pData ? m_pData->m_String : ""; }
 
   // Implicit conversion to C-style string -- deprecated.
   operator const FX_CHAR*() const { return m_pData ? m_pData->m_String : ""; }
 
   // Explicit conversion to uint8_t*.
   const uint8_t* raw_str() const {
     return m_pData ? reinterpret_cast<const uint8_t*>(m_pData->m_String)
                    : nullptr;
   }
 
   // Implicit conversiont to uint8_t* -- deprecated.
   operator const uint8_t*() const {
     return m_pData ? reinterpret_cast<const uint8_t*>(m_pData->m_String)
                    : nullptr;
   }
 
   FX_STRSIZE GetLength() const { return m_pData ? m_pData->m_nDataLength : 0; }
-
   bool IsEmpty() const { return !GetLength(); }
 
   int Compare(const CFX_ByteStringC& str) const;
 
   bool Equal(const char* ptr) const;
   bool Equal(const CFX_ByteStringC& str) const;
   bool Equal(const CFX_ByteString& other) const;
 
   bool EqualNoCase(const CFX_ByteStringC& str) const;
 
   bool operator==(const char* ptr) const { return Equal(ptr); }
   bool operator==(const CFX_ByteStringC& str) const { return Equal(str); }
   bool operator==(const CFX_ByteString& other) const { return Equal(other); }
 
   bool operator!=(const char* ptr) const { return !(*this == ptr); }
   bool operator!=(const CFX_ByteStringC& str) const { return !(*this == str); }
   bool operator!=(const CFX_ByteString& other) const {
     return !(*this == other);
   }
 
   bool operator<(const CFX_ByteString& str) const {
     int result = FXSYS_memcmp(c_str(), str.c_str(),
                               std::min(GetLength(), str.GetLength()));
     return result < 0 || (result == 0 && GetLength() < str.GetLength());
   }
 
-  void Empty();
-
   const CFX_ByteString& operator=(const FX_CHAR* str);
-
   const CFX_ByteString& operator=(const CFX_ByteStringC& bstrc);
-
   const CFX_ByteString& operator=(const CFX_ByteString& stringSrc);
-
   const CFX_ByteString& operator=(const CFX_BinaryBuf& buf);
 
   void Load(const uint8_t* str, FX_STRSIZE len);
 
   const CFX_ByteString& operator+=(FX_CHAR ch);
-
   const CFX_ByteString& operator+=(const FX_CHAR* str);
-
   const CFX_ByteString& operator+=(const CFX_ByteString& str);
-
   const CFX_ByteString& operator+=(const CFX_ByteStringC& bstrc);
 
   uint8_t GetAt(FX_STRSIZE nIndex) const {
     return m_pData ? m_pData->m_String[nIndex] : 0;
   }
 
   uint8_t operator[](FX_STRSIZE nIndex) const {
     return m_pData ? m_pData->m_String[nIndex] : 0;
   }
 
   void SetAt(FX_STRSIZE nIndex, FX_CHAR ch);
-
   FX_STRSIZE Insert(FX_STRSIZE index, FX_CHAR ch);
-
   FX_STRSIZE Delete(FX_STRSIZE index, FX_STRSIZE count = 1);
 
   void Format(const FX_CHAR* lpszFormat, ...);
-
   void FormatV(const FX_CHAR* lpszFormat, va_list argList);
 
   void Reserve(FX_STRSIZE len);
-
   FX_CHAR* GetBuffer(FX_STRSIZE len);
-
   void ReleaseBuffer(FX_STRSIZE len = -1);
 
   CFX_ByteString Mid(FX_STRSIZE first) const;
-
   CFX_ByteString Mid(FX_STRSIZE first, FX_STRSIZE count) const;
-
   CFX_ByteString Left(FX_STRSIZE count) const;
-
   CFX_ByteString Right(FX_STRSIZE count) const;
 
   FX_STRSIZE Find(const CFX_ByteStringC& lpszSub, FX_STRSIZE start = 0) const;
-
   FX_STRSIZE Find(FX_CHAR ch, FX_STRSIZE start = 0) const;
-
   FX_STRSIZE ReverseFind(FX_CHAR ch) const;
 
   void MakeLower();
-
   void MakeUpper();
 
   void TrimRight();
-
   void TrimRight(FX_CHAR chTarget);
-
   void TrimRight(const CFX_ByteStringC& lpszTargets);
 
   void TrimLeft();
-
   void TrimLeft(FX_CHAR chTarget);
-
   void TrimLeft(const CFX_ByteStringC& lpszTargets);
 
   FX_STRSIZE Replace(const CFX_ByteStringC& lpszOld,
                      const CFX_ByteStringC& lpszNew);
 
   FX_STRSIZE Remove(FX_CHAR ch);
 
   CFX_WideString UTF8Decode() const;
 
   uint32_t GetID(FX_STRSIZE start_pos = 0) const;
 
 #define FXFORMAT_SIGNED 1
 #define FXFORMAT_HEX 2
 #define FXFORMAT_CAPITAL 4
 
   static CFX_ByteString FormatInteger(int i, uint32_t flags = 0);
   static CFX_ByteString FormatFloat(FX_FLOAT f, int precision = 0);
 
  protected:
-  // To ensure ref counts do not overflow, consider the worst possible case:
-  // the entire address space contains nothing but pointers to this object.
-  // Since the count increments with each new pointer, the largest value is
-  // the number of pointers that can fit into the address space. The size of
-  // the address space itself is a good upper bound on it; we need not go
-  // larger.
   class StringData {
    public:
-    static StringData* Create(int nLen);
+    static StringData* Create(FX_STRSIZE nLen);
+    static StringData* Create(const StringData& other);
+    static StringData* Create(const FX_CHAR* pStr, FX_STRSIZE nLen);
+
     void Retain() { ++m_nRefs; }
     void Release() {
       if (--m_nRefs <= 0)
         FX_Free(this);
     }
 
+    bool CanOperateInPlace(FX_STRSIZE nTotalLen) {

[Lei Zhang, 2016/03/30 00:15:33]

const method

[Tom Sepez, 2016/03/30 18:49:18]

Done.

+      return m_nRefs <= 1 && nTotalLen <= m_nAllocLength;
+    }
+
+    void CopyContents(const StringData& other);
+    void CopyContents(const FX_CHAR* pStr, FX_STRSIZE nLen);
+    void CopyContentsAt(FX_STRSIZE offset,
+                        const FX_CHAR* pStr,
+                        FX_STRSIZE nLen);
+
+    // To ensure ref counts do not overflow, consider the worst possible case:
+    // the entire address space contains nothing but pointers to this object.
+    // Since the count increments with each new pointer, the largest value is
+    // the number of pointers that can fit into the address space. The size of
+    // the address space itself is a good upper bound on it.
     intptr_t m_nRefs;  // Would prefer ssize_t, but no windows support.
+
+    // |FX_STRSIZE| is currently typedef'd as in |int|.

[dsinclair, 2016/03/30 13:27:20]

nit: s/in/an

+    // TODO(palmer): It should be a |size_t|, or at least unsigned.
     FX_STRSIZE m_nDataLength;
     FX_STRSIZE m_nAllocLength;
+
+    // Not really 1, variable size.
     FX_CHAR m_String[1];
 
    private:
-    StringData(FX_STRSIZE dataLen, FX_STRSIZE allocLen)
-        : m_nRefs(1), m_nDataLength(dataLen), m_nAllocLength(allocLen) {
-      FXSYS_assert(dataLen >= 0);
-      FXSYS_assert(allocLen >= 0);
-      FXSYS_assert(dataLen <= allocLen);
-      m_String[dataLen] = 0;
-    }
+    StringData(FX_STRSIZE dataLen, FX_STRSIZE allocLen);
     ~StringData() = delete;
   };
 
+  void CopyBeforeWrite();
+  void AllocBeforeWrite(FX_STRSIZE nLen);
   void AllocCopy(CFX_ByteString& dest,
                  FX_STRSIZE nCopyLen,
                  FX_STRSIZE nCopyIndex) const;
-  void AssignCopy(FX_STRSIZE nSrcLen, const FX_CHAR* lpszSrcData);
-  void ConcatCopy(FX_STRSIZE nSrc1Len,
-                  const FX_CHAR* lpszSrc1Data,
-                  FX_STRSIZE nSrc2Len,
-                  const FX_CHAR* lpszSrc2Data);
-  void ConcatInPlace(FX_STRSIZE nSrcLen, const FX_CHAR* lpszSrcData);
-  void CopyBeforeWrite();
-  void AllocBeforeWrite(FX_STRSIZE nLen);
+  void AssignCopy(const FX_CHAR* pSrcData, FX_STRSIZE nSrcLen);
+  void Concat(const FX_CHAR* lpszSrcData, FX_STRSIZE nSrcLen);
 
-  StringData* m_pData;
-  friend class fxcrt_ByteStringConcatInPlace_Test;
+  CFX_RetainPtr<StringData> m_pData;
+  friend class fxcrt_ByteStringConcat_Test;
 };
+
 inline CFX_ByteStringC::CFX_ByteStringC(const CFX_ByteString& src) {
   m_Ptr = (const uint8_t*)src;
   m_Length = src.GetLength();
 }
 inline CFX_ByteStringC& CFX_ByteStringC::operator=(const CFX_ByteString& src) {
   m_Ptr = (const uint8_t*)src;
   m_Length = src.GetLength();
   return *this;
 }
 
@@ skipping 47 matching lines @@
   return CFX_ByteString(str1, str2);
 }
 inline CFX_ByteString operator+(const CFX_ByteString& str1,
                                 const CFX_ByteStringC& str2) {
   return CFX_ByteString(str1, str2);
 }
 inline CFX_ByteString operator+(const CFX_ByteStringC& str1,
                                 const CFX_ByteString& str2) {
   return CFX_ByteString(str1, str2);
 }
+
 class CFX_WideStringC {
  public:
   typedef FX_WCHAR value_type;
 
   CFX_WideStringC() {
     m_Ptr = NULL;
     m_Length = 0;
   }
 
   CFX_WideStringC(const FX_WCHAR* ptr) {
@@ skipping 37 matching lines @@
   }
   bool operator==(const CFX_WideStringC& str) const {
     return str.m_Length == m_Length && wmemcmp(str.m_Ptr, m_Ptr, m_Length) == 0;
   }
   bool operator!=(const wchar_t* ptr) const { return !(*this == ptr); }
   bool operator!=(const CFX_WideStringC& str) const { return !(*this == str); }
 
   const FX_WCHAR* GetPtr() const { return m_Ptr; }
 
   FX_STRSIZE GetLength() const { return m_Length; }
-
   bool IsEmpty() const { return m_Length == 0; }
 
   FX_WCHAR GetAt(FX_STRSIZE index) const { return m_Ptr[index]; }
 
   CFX_WideStringC Left(FX_STRSIZE count) const {
     if (count < 1) {
       return CFX_WideStringC();
     }
     if (count > m_Length) {
       count = m_Length;
@@ skipping 31 matching lines @@
     return result < 0 || (result == 0 && m_Length < that.m_Length);
   }
 
  protected:
   const FX_WCHAR* m_Ptr;
   FX_STRSIZE m_Length;
 
  private:
   void* operator new(size_t) throw() { return NULL; }
 };
+
 inline bool operator==(const wchar_t* lhs, const CFX_WideStringC& rhs) {
   return rhs == lhs;
 }
 inline bool operator!=(const wchar_t* lhs, const CFX_WideStringC& rhs) {
   return rhs != lhs;
 }
 #define FX_WSTRC(wstr) CFX_WideStringC(wstr, FX_ArraySize(wstr) - 1)
 
 // A mutable string with shared buffers using copy-on-write semantics that
 // avoids the cost of std::string's iterator stability guarantees.
 class CFX_WideString {
  public:
   typedef FX_WCHAR value_type;
 
   CFX_WideString() : m_pData(nullptr) {}
 
   // Copy constructor.
   CFX_WideString(const CFX_WideString& str);
 
   // Move constructor.
-  inline CFX_WideString(CFX_WideString&& other) {
+  CFX_WideString(CFX_WideString&& other) {
     m_pData = other.m_pData;
     other.m_pData = nullptr;
   }
 
   CFX_WideString(const FX_WCHAR* ptr)
       : CFX_WideString(ptr, ptr ? FXSYS_wcslen(ptr) : 0) {}
 
   CFX_WideString(const FX_WCHAR* ptr, FX_STRSIZE len);
-
   CFX_WideString(FX_WCHAR ch);
 
   CFX_WideString(const CFX_WideStringC& str);
-
   CFX_WideString(const CFX_WideStringC& str1, const CFX_WideStringC& str2);
 
   ~CFX_WideString();
 
   static CFX_WideString FromLocal(const CFX_ByteString& str);
-
   static CFX_WideString FromCodePage(const CFX_ByteString& str,
                                      uint16_t codepage);
 
   static CFX_WideString FromUTF8(const char* str, FX_STRSIZE len);
-
   static CFX_WideString FromUTF16LE(const unsigned short* str, FX_STRSIZE len);
 
   static FX_STRSIZE WStringLength(const unsigned short* str);
 
   // Explicit conversion to C-style wide string.
   const FX_WCHAR* c_str() const { return m_pData ? m_pData->m_String : L""; }
 
   // Implicit conversion to C-style wide string -- deprecated.
   operator const FX_WCHAR*() const { return m_pData ? m_pData->m_String : L""; }
 
   void Empty();
 
   bool IsEmpty() const { return !GetLength(); }
-
   FX_STRSIZE GetLength() const { return m_pData ? m_pData->m_nDataLength : 0; }
 
   const CFX_WideString& operator=(const FX_WCHAR* str);
-
   const CFX_WideString& operator=(const CFX_WideString& stringSrc);
-
   const CFX_WideString& operator=(const CFX_WideStringC& stringSrc);
 
   const CFX_WideString& operator+=(const FX_WCHAR* str);
-
   const CFX_WideString& operator+=(FX_WCHAR ch);
-
   const CFX_WideString& operator+=(const CFX_WideString& str);
-
   const CFX_WideString& operator+=(const CFX_WideStringC& str);
 
   bool operator==(const wchar_t* ptr) const { return Equal(ptr); }
   bool operator==(const CFX_WideStringC& str) const { return Equal(str); }
   bool operator==(const CFX_WideString& other) const { return Equal(other); }
 
   bool operator!=(const wchar_t* ptr) const { return !(*this == ptr); }
   bool operator!=(const CFX_WideStringC& str) const { return !(*this == str); }
   bool operator!=(const CFX_WideString& other) const {
     return !(*this == other);
   }
 
   bool operator<(const CFX_WideString& str) const {
     int result =
         wmemcmp(c_str(), str.c_str(), std::min(GetLength(), str.GetLength()));
     return result < 0 || (result == 0 && GetLength() < str.GetLength());
   }
 
   FX_WCHAR GetAt(FX_STRSIZE nIndex) const {
     return m_pData ? m_pData->m_String[nIndex] : 0;
   }
 
   FX_WCHAR operator[](FX_STRSIZE nIndex) const {
     return m_pData ? m_pData->m_String[nIndex] : 0;
   }
 
   void SetAt(FX_STRSIZE nIndex, FX_WCHAR ch);
 
   int Compare(const FX_WCHAR* str) const;
-
   int Compare(const CFX_WideString& str) const;
-
   int CompareNoCase(const FX_WCHAR* str) const;
 
   bool Equal(const wchar_t* ptr) const;
   bool Equal(const CFX_WideStringC& str) const;
   bool Equal(const CFX_WideString& other) const;
 
   CFX_WideString Mid(FX_STRSIZE first) const;
-
   CFX_WideString Mid(FX_STRSIZE first, FX_STRSIZE count) const;
-
   CFX_WideString Left(FX_STRSIZE count) const;
-
   CFX_WideString Right(FX_STRSIZE count) const;
 
   FX_STRSIZE Insert(FX_STRSIZE index, FX_WCHAR ch);
-
   FX_STRSIZE Delete(FX_STRSIZE index, FX_STRSIZE count = 1);
 
   void Format(const FX_WCHAR* lpszFormat, ...);
-
   void FormatV(const FX_WCHAR* lpszFormat, va_list argList);
 
   void MakeLower();
-
   void MakeUpper();
 
   void TrimRight();
-
   void TrimRight(FX_WCHAR chTarget);
-
   void TrimRight(const FX_WCHAR* lpszTargets);
 
   void TrimLeft();
-
   void TrimLeft(FX_WCHAR chTarget);
-
   void TrimLeft(const FX_WCHAR* lpszTargets);
 
   void Reserve(FX_STRSIZE len);
-
   FX_WCHAR* GetBuffer(FX_STRSIZE len);
-
   void ReleaseBuffer(FX_STRSIZE len = -1);
 
   int GetInteger() const;
-
   FX_FLOAT GetFloat() const;
 
   FX_STRSIZE Find(const FX_WCHAR* lpszSub, FX_STRSIZE start = 0) const;
-
   FX_STRSIZE Find(FX_WCHAR ch, FX_STRSIZE start = 0) const;
-
   FX_STRSIZE Replace(const FX_WCHAR* lpszOld, const FX_WCHAR* lpszNew);
-
   FX_STRSIZE Remove(FX_WCHAR ch);
 
   CFX_ByteString UTF8Encode() const;
-
   CFX_ByteString UTF16LE_Encode() const;
 
  protected:
   class StringData {
    public:
     static StringData* Create(int nLen);
     void Retain() { ++m_nRefs; }
     void Release() {
       if (--m_nRefs <= 0)
         FX_Free(this);
@@ skipping 106 matching lines @@
 }
 
 FX_FLOAT FX_atof(const CFX_ByteStringC& str);
 inline FX_FLOAT FX_atof(const CFX_WideStringC& wsStr) {
   return FX_atof(FX_UTF8Encode(wsStr.GetPtr(), wsStr.GetLength()));
 }
 void FX_atonum(const CFX_ByteStringC& str, FX_BOOL& bInteger, void* pData);
 FX_STRSIZE FX_ftoa(FX_FLOAT f, FX_CHAR* buf);
 
 #endif  // CORE_FXCRT_INCLUDE_FX_STRING_H_