[Lldb-commits] [PATCH] D84440: [llb] Remove the user-defined copy-ctor in ConstString

[Shafik Yaghmour via Phabricator via lldb-commits]

shafik added inline comments.


================
Comment at: lldb/include/lldb/Utility/ConstString.h:45
   /// Initializes the string to an empty string.
   ConstString() : m_string(nullptr) {}
 
----------------
Since your touching this type, why not modernize it and use an in class member initializer for `m_string` Then default the constructor.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84440/new/

https://reviews.llvm.org/D84440