This revision was automatically updated to reflect the committed changes. Closed by commit rL334860: Revert "[lldb-mi] Add overload method for setting an error" (authored by apolyakov, committed by ). Herald added a subscriber: llvm-commits.
Changed prior to commit: https://reviews.llvm.org/D48212?vs=151482&id=151553#toc Repository: rL LLVM https://reviews.llvm.org/D48212 Files: lldb/trunk/tools/lldb-mi/MICmdBase.cpp lldb/trunk/tools/lldb-mi/MICmdBase.h Index: lldb/trunk/tools/lldb-mi/MICmdBase.cpp =================================================================== --- lldb/trunk/tools/lldb-mi/MICmdBase.cpp +++ lldb/trunk/tools/lldb-mi/MICmdBase.cpp @@ -214,23 +214,6 @@ //++ //------------------------------------------------------------------------------------ -// Details: Short cut function to enter error information into the command's -// metadata object and set the command's error status. -// Type: Method. -// Args: error - (R) Command result description. -// Return: None. -// Throws: None. -//-- -void CMICmdBase::SetError(const lldb::SBError &error) { - const char *error_cstr = error.GetCString(); - if (error_cstr) - SetError(error_cstr); - else - SetError("unknown error"); -} - -//++ -//------------------------------------------------------------------------------------ // Details: Ask a command to provide its unique identifier. // Type: Method. // Args: A unique identifier for this command class. Index: lldb/trunk/tools/lldb-mi/MICmdBase.h =================================================================== --- lldb/trunk/tools/lldb-mi/MICmdBase.h +++ lldb/trunk/tools/lldb-mi/MICmdBase.h @@ -12,8 +12,6 @@ // C Includes // C++ Includes // Other libraries and framework includes -#include "lldb/API/SBError.h" - // Project includes #include "MICmdArgSet.h" #include "MICmdData.h" @@ -82,7 +80,6 @@ // Methods: protected: void SetError(const CMIUtilString &rErrMsg); - void SetError(const lldb::SBError &error); template <class T> T *GetOption(const CMIUtilString &vStrOptionName); bool ParseValidateCmdOptions();
Index: lldb/trunk/tools/lldb-mi/MICmdBase.cpp =================================================================== --- lldb/trunk/tools/lldb-mi/MICmdBase.cpp +++ lldb/trunk/tools/lldb-mi/MICmdBase.cpp @@ -214,23 +214,6 @@ //++ //------------------------------------------------------------------------------------ -// Details: Short cut function to enter error information into the command's -// metadata object and set the command's error status. -// Type: Method. -// Args: error - (R) Command result description. -// Return: None. -// Throws: None. -//-- -void CMICmdBase::SetError(const lldb::SBError &error) { - const char *error_cstr = error.GetCString(); - if (error_cstr) - SetError(error_cstr); - else - SetError("unknown error"); -} - -//++ -//------------------------------------------------------------------------------------ // Details: Ask a command to provide its unique identifier. // Type: Method. // Args: A unique identifier for this command class. Index: lldb/trunk/tools/lldb-mi/MICmdBase.h =================================================================== --- lldb/trunk/tools/lldb-mi/MICmdBase.h +++ lldb/trunk/tools/lldb-mi/MICmdBase.h @@ -12,8 +12,6 @@ // C Includes // C++ Includes // Other libraries and framework includes -#include "lldb/API/SBError.h" - // Project includes #include "MICmdArgSet.h" #include "MICmdData.h" @@ -82,7 +80,6 @@ // Methods: protected: void SetError(const CMIUtilString &rErrMsg); - void SetError(const lldb::SBError &error); template <class T> T *GetOption(const CMIUtilString &vStrOptionName); bool ParseValidateCmdOptions();
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits