Skip to content

Commit e6ae4d6

Browse files
committed
Merge branch 'rs/simplify-submodule-helper-super-prefix-invocation'
Code clean-up. * rs/simplify-submodule-helper-super-prefix-invocation: submodule--helper: use strvec_pushf() for --super-prefix
2 parents 7c01dcd + 4b837f8 commit e6ae4d6

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

builtin/submodule--helper.c

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -376,8 +376,7 @@ static void runcommand_in_submodule_cb(const struct cache_entry *list_item,
376376

377377
strvec_pushl(&cpr.args, "submodule--helper", "foreach", "--recursive",
378378
NULL);
379-
strvec_pushl(&cpr.args, "--super-prefix", NULL);
380-
strvec_pushf(&cpr.args, "%s/", displaypath);
379+
strvec_pushf(&cpr.args, "--super-prefix=%s/", displaypath);
381380

382381
if (info->quiet)
383382
strvec_push(&cpr.args, "--quiet");
@@ -702,8 +701,7 @@ static void status_submodule(const char *path, const struct object_id *ce_oid,
702701

703702
strvec_pushl(&cpr.args, "submodule--helper", "status",
704703
"--recursive", NULL);
705-
strvec_push(&cpr.args, "--super-prefix");
706-
strvec_pushf(&cpr.args, "%s/", displaypath);
704+
strvec_pushf(&cpr.args, "--super-prefix=%s/", displaypath);
707705

708706
if (flags & OPT_CACHED)
709707
strvec_push(&cpr.args, "--cached");
@@ -1304,9 +1302,7 @@ static void sync_submodule(const char *path, const char *prefix,
13041302

13051303
strvec_pushl(&cpr.args, "submodule--helper", "sync",
13061304
"--recursive", NULL);
1307-
strvec_push(&cpr.args, "--super-prefix");
1308-
strvec_pushf(&cpr.args, "%s/", displaypath);
1309-
1305+
strvec_pushf(&cpr.args, "--super-prefix=%s/", displaypath);
13101306

13111307
if (flags & OPT_QUIET)
13121308
strvec_push(&cpr.args, "--quiet");
@@ -2534,10 +2530,9 @@ static void update_data_to_args(const struct update_data *update_data,
25342530
enum submodule_update_type update_type = update_data->update_default;
25352531

25362532
strvec_pushl(args, "submodule--helper", "update", "--recursive", NULL);
2537-
if (update_data->displaypath) {
2538-
strvec_push(args, "--super-prefix");
2539-
strvec_pushf(args, "%s/", update_data->displaypath);
2540-
}
2533+
if (update_data->displaypath)
2534+
strvec_pushf(args, "--super-prefix=%s/",
2535+
update_data->displaypath);
25412536
strvec_pushf(args, "--jobs=%d", update_data->max_jobs);
25422537
if (update_data->quiet)
25432538
strvec_push(args, "--quiet");

0 commit comments

Comments
 (0)