Author: teemperor Date: Wed Aug 28 02:02:32 2019 New Revision: 370174 URL: http://llvm.org/viewvc/llvm-project?rev=370174&view=rev Log: [lldb][NFC] Update documentation of Handle[Argument]Completion
We no longer have return values or any of the mentioned arguments in these functions since the introduction of CompletionRequest. Modified: lldb/trunk/include/lldb/Interpreter/CommandObject.h Modified: lldb/trunk/include/lldb/Interpreter/CommandObject.h URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/include/lldb/Interpreter/CommandObject.h?rev=370174&r1=370173&r2=370174&view=diff ============================================================================== --- lldb/trunk/include/lldb/Interpreter/CommandObject.h (original) +++ lldb/trunk/include/lldb/Interpreter/CommandObject.h Wed Aug 28 02:02:32 2019 @@ -228,25 +228,15 @@ public: /// /// \param[in/out] request /// The completion request that needs to be answered. - /// - /// FIXME: This is the wrong return value, since we also need to make a - /// distinction between - /// total number of matches, and the window the user wants returned. virtual void HandleCompletion(CompletionRequest &request); - /// The input array contains a parsed version of the line. The insertion - /// point is given by cursor_index (the index in input of the word containing - /// the cursor) and cursor_char_position (the position of the cursor in that - /// word.) + /// The input array contains a parsed version of the line. + /// /// We've constructed the map of options and their arguments as well if that /// is helpful for the completion. /// /// \param[in/out] request /// The completion request that needs to be answered. - /// - /// FIXME: This is the wrong return value, since we also need to make a - /// distinction between - /// total number of matches, and the window the user wants returned. virtual void HandleArgumentCompletion(CompletionRequest &request, OptionElementVector &opt_element_vector) {} _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits