From 80b103bf53ee1249823e6bb3bed9cb2bc15ca67a Mon Sep 17 00:00:00 2001 From: tsepez Date: Thu, 14 Apr 2016 17:39:09 -0700 Subject: Still more comments about strings TBR=thestig@chromium.org Review URL: https://codereview.chromium.org/1888843004 --- core/fxcrt/include/fx_string.h | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/core/fxcrt/include/fx_string.h b/core/fxcrt/include/fx_string.h index 94df35c8e7..2fabee39cf 100644 --- a/core/fxcrt/include/fx_string.h +++ b/core/fxcrt/include/fx_string.h @@ -34,18 +34,19 @@ class CFX_ByteStringC { m_Length = size; } + // Deliberately implicit to avoid calling on every string literal. CFX_ByteStringC(const FX_CHAR* ptr) { m_Ptr = (const uint8_t*)ptr; m_Length = ptr ? FXSYS_strlen(ptr) : 0; } + // Deliberately implicit to avoid calling on every string literal. // |ch| must be an lvalue that outlives the the CFX_ByteStringC. However, // the use of char rvalues are not caught at compile time. They are // implicitly promoted to CFX_ByteString (see below) and then the // CFX_ByteStringC is constructed from the CFX_ByteString via the alternate // constructor below. The CFX_ByteString then typically goes out of scope // and |m_Ptr| may be left pointing to invalid memory. Beware. - // TODO(tsepez): Mark single-argument string constructors as explicit. CFX_ByteStringC(FX_CHAR& ch) { m_Ptr = (const uint8_t*)&ch; m_Length = 1; @@ -357,11 +358,19 @@ class CFX_WideStringC { m_Length = 0; } + // Deliberately implicit to avoid calling on every string literal. CFX_WideStringC(const FX_WCHAR* ptr) { m_Ptr = ptr; m_Length = ptr ? FXSYS_wcslen(ptr) : 0; } + // Deliberately implicit to avoid calling on every string literal. + // |ch| must be an lvalue that outlives the the CFX_WideStringC. However, + // the use of char rvalues are not caught at compile time. They are + // implicitly promoted to CFX_WideString (see below) and then the + // CFX_WideStringC is constructed from the CFX_WideString via the alternate + // constructor below. The CFX_WideString then typically goes out of scope + // and |m_Ptr| may be left pointing to invalid memory. Beware. CFX_WideStringC(FX_WCHAR& ch) { m_Ptr = &ch; m_Length = 1; -- cgit v1.2.3