Stop passing mutable references to completions around

Replace uses of vector<completion_t>& with vector<completion_t>*
This makes it clear at the call site that the object may be mutated.

1 files changed, 3 insertions, 1 deletions

diff --git a/src/builtin.cpp b/src/builtin.cpp
index b31fb6a1..51aa445a 100644
--- a/src/builtin.cpp
+++ b/src/builtin.cpp
@@ -4226,8 +4226,10 @@ wcstring_list_t builtin_get_names(void)
     return result;
 }
 
-void builtin_get_names(std::vector<completion_t> &list)
+void builtin_get_names(std::vector<completion_t> *list)
 {
+    assert(list != NULL);
+    list->reserve(list->size() + BUILTIN_COUNT);
     for (size_t i=0; i < BUILTIN_COUNT; i++)
     {
         append_completion(list, builtin_datas[i].name);