Support '*' and comma separated list for exec/eval -client switch

Closes #5326

Author: mawww

doc/pages/execeval.asciidoc

@@ -18,8 +18,9 @@ are then restored when the keys have been executed: */*, *"*, *|*, *^*,
 
 == Switches for both commands
 
-*-client* <name>::
-    Execute in the context of the client *name*.
+*-client* <names>::
+    Execute in the context of each client specified in the comma separated
+    list *names*. '\*' can be used as *names* to iterate overall clients.
 
 *-try-client* <name>::
     Execute in the context of the client *name* if such client exists,
@@ -37,7 +38,7 @@ are then restored when the keys have been executed: */*, *"*, *|*, *^*,
 
 *-buffer* <names>::
     Execute in the context of each buffer specified in the comma separated
-    list *names*. '\*' can be used as a *name* to iterate over all non-debug
+    list *names*. '\*' can be used as *names* to iterate over all non-debug
     buffers.
     (See <<buffers#debug-buffers, `:doc buffers`>>)
 

src/commands.cc

@@ -2038,7 +2038,7 @@ template<size_t... P>
 ParameterDesc make_context_wrap_params_impl(Array<HashItem<String, SwitchDesc>, sizeof...(P)>&& additional_params,
                                             std::index_sequence<P...>)
 {
-    return { { { "client",     { {client_arg_completer},  "run in given client context" } },
+    return { { { "client",     { {client_arg_completer},  "run in the client context for each client in the given comma separatd list" } },
                { "try-client", { {client_arg_completer},  "run in given client context if it exists, or else in the current one" } },
                { "buffer",     { {complete_buffer_name<false>},  "run in a disposable context for each given buffer in the comma separated list argument" } },
                { "draft",      { {}, "run in a disposable context" } },
@@ -2188,14 +2188,25 @@ void context_wrap(const ParametersParser& parser, Context& context, StringView d
     };
 
     ClientManager& cm = ClientManager::instance();
-    if (auto client_name = parser.get_switch("client"))
-        context_wrap_for_context(cm.get_client(*client_name).context());
-    else if (auto client_name = parser.get_switch("try-client"))
+    if (auto client_names = parser.get_switch("client"))
     {
-        if (Client* client = cm.get_client_ifp(*client_name))
-            context_wrap_for_context(client->context());
+        if (*client_names == "*")
+        {
+            for (auto&& client : ClientManager::instance()
+                               | transform(&std::unique_ptr<Client>::get)
+                               | gather<Vector<SafePtr<Client>>>()) // gather as we might be mutating the client list in the loop.
+                context_wrap_for_context(client->context());
+        }
         else
-            context_wrap_for_context(context);
+            for (auto&& name : *client_names
+                             | split<StringView>(',', '\\')
+                             | transform(unescape<',', '\\'>))
+                context_wrap_for_context(ClientManager::instance().get_client(name).context());
+    }
+    else if (auto client_name = parser.get_switch("try-client"))
+    {
+        Client* client = cm.get_client_ifp(*client_name);
+        context_wrap_for_context(client ? client->context() : context);
     }
     else
         context_wrap_for_context(context);