Author: teemperor Date: Thu Sep 26 00:06:05 2019 New Revision: 372946 URL: http://llvm.org/viewvc/llvm-project?rev=372946&view=rev Log: [lldb][NFC] Use AppendEmptyArgument in CompletionRequest constructor
We now have a utility function for this purpose. (Also fixing the typo in the related comment while I'm at it.) Modified: lldb/trunk/source/Utility/CompletionRequest.cpp Modified: lldb/trunk/source/Utility/CompletionRequest.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Utility/CompletionRequest.cpp?rev=372946&r1=372945&r2=372946&view=diff ============================================================================== --- lldb/trunk/source/Utility/CompletionRequest.cpp (original) +++ lldb/trunk/source/Utility/CompletionRequest.cpp Thu Sep 26 00:06:05 2019 @@ -35,14 +35,10 @@ CompletionRequest::CompletionRequest(llv // The cursor is after a space but the space is not part of the argument. // Let's add an empty fake argument to the end to make sure the completion - // code Note: The space could be part of the last argument when it's quoted. + // code. Note: The space could be part of the last argument when it's quoted. if (partial_command.endswith(" ") && - !GetCursorArgumentPrefix().endswith(" ")) { - m_parsed_line.AppendArgument(llvm::StringRef()); - // Set the cursor to the start of the fake argument. - m_cursor_index++; - m_cursor_char_position = 0; - } + !GetCursorArgumentPrefix().endswith(" ")) + AppendEmptyArgument(); } std::string CompletionResult::Completion::GetUniqueKey() const { _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits