Make the Error class constructor protected

This is forcing to use Error::success(), which is in a wide majority
of cases a lot more readable.

Differential Revision: https://reviews.llvm.org/D26481

llvm-svn: 286561

1 files changed, 2 insertions, 2 deletions

diff --git a/lldb/source/Interpreter/OptionValueString.cpp b/lldb/source/Interpreter/OptionValueString.cpp
index e61ead081b8..b5ce38ce4fd 100644
--- a/lldb/source/Interpreter/OptionValueString.cpp
+++ b/lldb/source/Interpreter/OptionValueString.cpp
@@ -133,7 +133,7 @@ Error OptionValueString::SetCurrentValue(llvm::StringRef value) {
       return error;
   }
   m_current_value.assign(value);
-  return Error();
+  return Error::success();
 }
 
 Error OptionValueString::AppendToCurrentValue(const char *value) {
@@ -148,5 +148,5 @@ Error OptionValueString::AppendToCurrentValue(const char *value) {
     } else
       m_current_value.append(value);
   }
-  return Error();
+  return Error::success();
 }