mirror of
https://github.com/git/git.git
synced 2024-12-13 11:54:56 +08:00
6def0ff878
Migrate those run-command API users that assign directly to the "argv"
member to use a strvec_pushv() of "args" instead.
In these cases it did not make sense to further refactor these
callers, e.g. daemon.c could be made to construct the arguments closer
to handle(), but that would require moving the construction from its
cmd_main() and pass "argv" through two intermediate functions.
It would be possible for a change like this to introduce a regression
if we were doing:
cp.argv = argv;
argv[1] = "foo";
And changed the code, as is being done here, to:
strvec_pushv(&cp.args, argv);
argv[1] = "foo";
But as viewing this change with the "-W" flag reveals none of these
functions modify variable that's being pushed afterwards in a way that
would introduce such a logic error.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
21 lines
447 B
C
21 lines
447 B
C
#include "test-tool.h"
|
|
#include "cache.h"
|
|
#include "run-command.h"
|
|
|
|
int cmd__subprocess(int argc, const char **argv)
|
|
{
|
|
struct child_process cp = CHILD_PROCESS_INIT;
|
|
int nogit = 0;
|
|
|
|
setup_git_directory_gently(&nogit);
|
|
if (nogit)
|
|
die("No git repo found");
|
|
if (argc > 1 && !strcmp(argv[1], "--setup-work-tree")) {
|
|
setup_work_tree();
|
|
argv++;
|
|
}
|
|
cp.git_cmd = 1;
|
|
strvec_pushv(&cp.args, (const char **)argv + 1);
|
|
return run_command(&cp);
|
|
}
|