Make down-conversion explicit from CFX_ByteString to CFX_ByteStringC.

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.
@@ skipping 43 matching lines @@
   CFX_ByteStringC(const FX_CHAR* ptr, FX_STRSIZE len) {
     m_Ptr = (const uint8_t*)ptr;
     m_Length = (len == -1) ? FXSYS_strlen(ptr) : len;
   }
 
   CFX_ByteStringC(const CFX_ByteStringC& src) {
     m_Ptr = src.m_Ptr;
     m_Length = src.m_Length;
   }
 
-  CFX_ByteStringC(const CFX_ByteString& src);
-
   CFX_ByteStringC& operator=(const FX_CHAR* src) {
     m_Ptr = (const uint8_t*)src;
     m_Length = m_Ptr ? FXSYS_strlen(src) : 0;
     return *this;
   }
 
   CFX_ByteStringC& operator=(const CFX_ByteStringC& src) {
     m_Ptr = src.m_Ptr;
     m_Length = src.m_Length;
     return *this;
@@ skipping 84 matching lines @@
   ~CFX_ByteString();
 
   // Deprecated -- use clear().
   void Empty() { m_pData.Reset(); }
   void clear() { m_pData.Reset(); }
 
   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.
+  // Note: |this| must outlive the use of the result.
   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*.
+  // Note: |this| must outlive the use of the result.
   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;
   }
 
+  // Explicit conversion to CFX_ByteStringC.
+  // Note: |this| must outlive the use of the result.
+  CFX_ByteStringC AsByteStringC() const {
+    return CFX_ByteStringC(raw_str(), GetLength());
+  }
+
   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 EqualNoCase(const CFX_ByteStringC& str) const;
 
   bool operator==(const char* ptr) const;
   bool operator==(const CFX_ByteStringC& str) const;
   bool operator==(const CFX_ByteString& other) const;
 
@@ skipping 126 matching lines @@
   void AllocCopy(CFX_ByteString& dest,
                  FX_STRSIZE nCopyLen,
                  FX_STRSIZE nCopyIndex) const;
   void AssignCopy(const FX_CHAR* pSrcData, FX_STRSIZE nSrcLen);
   void Concat(const FX_CHAR* lpszSrcData, FX_STRSIZE nSrcLen);
 
   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;
 }
 
 inline bool operator==(const char* lhs, const CFX_ByteString& rhs) {
   return rhs == lhs;
 }
 inline bool operator==(const CFX_ByteStringC& lhs, const CFX_ByteString& rhs) {
@@ skipping 19 matching lines @@
   return CFX_ByteString(str1, str2);
 }
 inline CFX_ByteString operator+(const CFX_ByteStringC& str1, FX_CHAR ch) {
   return CFX_ByteString(str1, CFX_ByteStringC(ch));
 }
 inline CFX_ByteString operator+(FX_CHAR ch, const CFX_ByteStringC& str2) {
   return CFX_ByteString(ch, str2);
 }
 inline CFX_ByteString operator+(const CFX_ByteString& str1,
                                 const CFX_ByteString& str2) {
-  return CFX_ByteString(str1, str2);
+  return CFX_ByteString(str1.AsByteStringC(), str2.AsByteStringC());
 }
 inline CFX_ByteString operator+(const CFX_ByteString& str1, FX_CHAR ch) {
-  return CFX_ByteString(str1, CFX_ByteStringC(ch));
+  return CFX_ByteString(str1.AsByteStringC(), CFX_ByteStringC(ch));
 }
 inline CFX_ByteString operator+(FX_CHAR ch, const CFX_ByteString& str2) {
-  return CFX_ByteString(ch, str2);
+  return CFX_ByteString(ch, str2.AsByteStringC());
 }
 inline CFX_ByteString operator+(const CFX_ByteString& str1,
                                 const FX_CHAR* str2) {
-  return CFX_ByteString(str1, str2);
+  return CFX_ByteString(str1.AsByteStringC(), str2);
 }
 inline CFX_ByteString operator+(const FX_CHAR* str1,
                                 const CFX_ByteString& str2) {
-  return CFX_ByteString(str1, str2);
+  return CFX_ByteString(str1, str2.AsByteStringC());
 }
 inline CFX_ByteString operator+(const CFX_ByteString& str1,
                                 const CFX_ByteStringC& str2) {
-  return CFX_ByteString(str1, str2);
+  return CFX_ByteString(str1.AsByteStringC(), str2);
 }
 inline CFX_ByteString operator+(const CFX_ByteStringC& str1,
                                 const CFX_ByteString& str2) {
-  return CFX_ByteString(str1, str2);
+  return CFX_ByteString(str1, str2.AsByteStringC());
 }
 
 class CFX_WideStringC {
  public:
   typedef FX_WCHAR value_type;
 
   CFX_WideStringC() {
     m_Ptr = NULL;
     m_Length = 0;
   }
@@ skipping 342 matching lines @@
 CFX_ByteString FX_UTF8Encode(const FX_WCHAR* pwsStr, FX_STRSIZE len);
 inline CFX_ByteString FX_UTF8Encode(const CFX_WideStringC& wsStr) {
   return FX_UTF8Encode(wsStr.GetPtr(), wsStr.GetLength());
 }
 inline CFX_ByteString FX_UTF8Encode(const CFX_WideString& wsStr) {
   return FX_UTF8Encode(wsStr.c_str(), wsStr.GetLength());
 }
 
 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()));
+  return FX_atof(FX_UTF8Encode(wsStr.GetPtr(), wsStr.GetLength()).c_str());
 }
 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_