Make CommandReturnObject methods accept StringRefs.

llvm-svn: 286728

1 files changed, 23 insertions, 20 deletions

diff --git a/lldb/source/Interpreter/CommandReturnObject.cpp b/lldb/source/Interpreter/CommandReturnObject.cpp
index c289fd793a8..53f3bd2d8d6 100644
--- a/lldb/source/Interpreter/CommandReturnObject.cpp
+++ b/lldb/source/Interpreter/CommandReturnObject.cpp
@@ -84,30 +84,31 @@ void CommandReturnObject::AppendWarningWithFormat(const char *format, ...) {
   GetErrorStream().Printf("warning: %s", sstrm.GetData());
 }
 
-void CommandReturnObject::AppendMessage(const char *in_string) {
-  if (!in_string)
+void CommandReturnObject::AppendMessage(llvm::StringRef in_string) {
+  if (in_string.empty())
     return;
-  GetOutputStream().Printf("%s\n", in_string);
+  GetOutputStream() << in_string << "\n";
 }
 
-void CommandReturnObject::AppendWarning(const char *in_string) {
-  if (!in_string || *in_string == '\0')
+void CommandReturnObject::AppendWarning(llvm::StringRef in_string) {
+  if (in_string.empty())
     return;
-  GetErrorStream().Printf("warning: %s\n", in_string);
+  GetErrorStream() << "warning: " << in_string << "\n";
 }
 
 // Similar to AppendWarning, but do not prepend 'warning: ' to message, and
 // don't append "\n" to the end of it.
 
-void CommandReturnObject::AppendRawWarning(const char *in_string) {
-  if (in_string && in_string[0])
-    GetErrorStream().PutCString(in_string);
+void CommandReturnObject::AppendRawWarning(llvm::StringRef in_string) {
+  if (in_string.empty())
+    return;
+  GetErrorStream() << in_string;
 }
 
-void CommandReturnObject::AppendError(const char *in_string) {
-  if (!in_string || *in_string == '\0')
+void CommandReturnObject::AppendError(llvm::StringRef in_string) {
+  if (in_string.empty())
     return;
-  GetErrorStream().Printf("error: %s\n", in_string);
+  GetErrorStream() << "error: " << in_string << "\n";
 }
 
 void CommandReturnObject::SetError(const Error &error,
@@ -118,19 +119,21 @@ void CommandReturnObject::SetError(const Error &error,
   SetError(error_cstr);
 }
 
-void CommandReturnObject::SetError(const char *error_cstr) {
-  if (error_cstr) {
-    AppendError(error_cstr);
-    SetStatus(eReturnStatusFailed);
-  }
+void CommandReturnObject::SetError(llvm::StringRef error_str) {
+  if (error_str.empty())
+    return;
+
+  AppendError(error_str);
+  SetStatus(eReturnStatusFailed);
 }
 
 // Similar to AppendError, but do not prepend 'Error: ' to message, and
 // don't append "\n" to the end of it.
 
-void CommandReturnObject::AppendRawError(const char *in_string) {
-  if (in_string && in_string[0])
-    GetErrorStream().PutCString(in_string);
+void CommandReturnObject::AppendRawError(llvm::StringRef in_string) {
+  if (in_string.empty())
+    return;
+  GetErrorStream() << in_string;
 }
 
 void CommandReturnObject::SetStatus(ReturnStatus status) { m_status = status; }