================
@@ -305,6 +305,42 @@ void CommandObject::HandleCompletion(CompletionRequest 
&request) {
   }
 }
 
+void
+CommandObject::HandleArgumentCompletion(CompletionRequest &request,
+                           OptionElementVector &opt_element_vector) {
+  size_t num_arg_entries = GetNumArgumentEntries();
+  if (num_arg_entries != 1)
+    return;
+
+  CommandArgumentEntry *entry_ptr = GetArgumentEntryAtIndex(0);
+  if (!entry_ptr)
+    return; // Maybe this should be an assert, this shouldn't be possible.
----------------
bulbazord wrote:

If it shouldn't be possible, I think an assert is warranted here. I would say 
add the assertion above the early return (and keep the early return since the 
assertion will be compiled out in release builds).

https://github.com/llvm/llvm-project/pull/82085
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to