Author: bogner Date: Mon Oct 17 01:17:56 2016 New Revision: 284362 URL: http://llvm.org/viewvc/llvm-project?rev=284362&view=rev Log: Interpreter: Don't return StringRef from functions whose return value is never used
StringRef is passed through all of these APIs but never actually used. Just remove it from the API for now and if people want to use it they can add it back. Modified: lldb/trunk/include/lldb/Interpreter/Args.h lldb/trunk/source/Interpreter/Args.cpp Modified: lldb/trunk/include/lldb/Interpreter/Args.h URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/include/lldb/Interpreter/Args.h?rev=284362&r1=284361&r2=284362&view=diff ============================================================================== --- lldb/trunk/include/lldb/Interpreter/Args.h (original) +++ lldb/trunk/include/lldb/Interpreter/Args.h Mon Oct 17 01:17:56 2016 @@ -193,12 +193,8 @@ public: /// /// @param[in] quote_char /// If the argument was originally quoted, put in the quote char here. - /// - /// @return - /// The NULL terminated C string of the copy of \a arg_cstr. //------------------------------------------------------------------ - llvm::StringRef AppendArgument(llvm::StringRef arg_str, - char quote_char = '\0'); + void AppendArgument(llvm::StringRef arg_str, char quote_char = '\0'); void AppendArguments(const Args &rhs); @@ -219,8 +215,8 @@ public: /// @return /// The NULL terminated C string of the copy of \a arg_cstr. //------------------------------------------------------------------ - llvm::StringRef InsertArgumentAtIndex(size_t idx, llvm::StringRef arg_str, - char quote_char = '\0'); + void InsertArgumentAtIndex(size_t idx, llvm::StringRef arg_str, + char quote_char = '\0'); //------------------------------------------------------------------ /// Replaces the argument value at index \a idx to \a arg_cstr @@ -234,13 +230,9 @@ public: /// /// @param[in] quote_char /// If the argument was originally quoted, put in the quote char here. - /// - /// @return - /// The NULL terminated C string of the copy of \a arg_cstr if - /// \a idx was a valid index, NULL otherwise. //------------------------------------------------------------------ - llvm::StringRef ReplaceArgumentAtIndex(size_t idx, llvm::StringRef arg_str, - char quote_char = '\0'); + void ReplaceArgumentAtIndex(size_t idx, llvm::StringRef arg_str, + char quote_char = '\0'); //------------------------------------------------------------------ /// Deletes the argument value at index @@ -289,11 +281,8 @@ public: /// /// @param[in] quote_char /// If the argument was originally quoted, put in the quote char here. - /// - /// @return - /// A pointer to the copy of \a arg_cstr that was made. //------------------------------------------------------------------ - llvm::StringRef Unshift(llvm::StringRef arg_str, char quote_char = '\0'); + void Unshift(llvm::StringRef arg_str, char quote_char = '\0'); //------------------------------------------------------------------ /// Parse the arguments in the contained arguments. Modified: lldb/trunk/source/Interpreter/Args.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/Args.cpp?rev=284362&r1=284361&r2=284362&view=diff ============================================================================== --- lldb/trunk/source/Interpreter/Args.cpp (original) +++ lldb/trunk/source/Interpreter/Args.cpp Mon Oct 17 01:17:56 2016 @@ -332,8 +332,8 @@ void Args::Shift() { m_entries.erase(m_entries.begin()); } -llvm::StringRef Args::Unshift(llvm::StringRef arg_str, char quote_char) { - return InsertArgumentAtIndex(0, arg_str, quote_char); +void Args::Unshift(llvm::StringRef arg_str, char quote_char) { + InsertArgumentAtIndex(0, arg_str, quote_char); } void Args::AppendArguments(const Args &rhs) { @@ -361,30 +361,28 @@ void Args::AppendArguments(const char ** m_argv.push_back(nullptr); } -llvm::StringRef Args::AppendArgument(llvm::StringRef arg_str, char quote_char) { - return InsertArgumentAtIndex(GetArgumentCount(), arg_str, quote_char); +void Args::AppendArgument(llvm::StringRef arg_str, char quote_char) { + InsertArgumentAtIndex(GetArgumentCount(), arg_str, quote_char); } -llvm::StringRef Args::InsertArgumentAtIndex(size_t idx, llvm::StringRef arg_str, - char quote_char) { +void Args::InsertArgumentAtIndex(size_t idx, llvm::StringRef arg_str, + char quote_char) { assert(m_argv.size() == m_entries.size() + 1); assert(m_argv.back() == nullptr); if (idx > m_entries.size()) - return llvm::StringRef(); + return; m_entries.emplace(m_entries.begin() + idx, arg_str, quote_char); m_argv.insert(m_argv.begin() + idx, m_entries[idx].data()); - return m_entries[idx].ref; } -llvm::StringRef Args::ReplaceArgumentAtIndex(size_t idx, - llvm::StringRef arg_str, - char quote_char) { +void Args::ReplaceArgumentAtIndex(size_t idx, llvm::StringRef arg_str, + char quote_char) { assert(m_argv.size() == m_entries.size() + 1); assert(m_argv.back() == nullptr); if (idx >= m_entries.size()) - return llvm::StringRef(); + return; if (arg_str.size() > m_entries[idx].ref.size()) { m_entries[idx] = ArgEntry(arg_str, quote_char); @@ -395,8 +393,6 @@ llvm::StringRef Args::ReplaceArgumentAtI m_entries[idx].ptr[arg_str.size()] = 0; m_entries[idx].ref = m_entries[idx].ref.take_front(arg_str.size()); } - - return m_entries[idx].ref; } void Args::DeleteArgumentAtIndex(size_t idx) { _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits