@@ -2484,7 +2484,7 @@ static int run_update_command(struct update_data *ud, int subforce, struct strin
return 0;
}
-static int do_run_update_procedure(struct update_data *ud, struct string_list *err)
+static int run_update_procedure(struct update_data *ud, struct string_list *err)
{
int subforce = is_null_oid(&ud->suboid) || ud->force;
@@ -2646,7 +2646,7 @@ static int push_check(int argc, const char **argv, const char *prefix)
return 0;
}
-static void do_ensure_core_worktree(const char *path)
+static void ensure_core_worktree(const char *path)
{
const struct submodule *sub;
const char *cw;
@@ -2910,7 +2910,7 @@ static int update_submodule(struct update_data *update_data)
char *prefixed_path;
struct string_list err = STRING_LIST_INIT_DUP;
- do_ensure_core_worktree(update_data->sm_path);
+ ensure_core_worktree(update_data->sm_path);
if (update_data->recursive_prefix)
prefixed_path = xstrfmt("%s%s", update_data->recursive_prefix,
@@ -2950,7 +2950,7 @@ static int update_submodule(struct update_data *update_data)
}
if (!oideq(&update_data->oid, &update_data->suboid) || update_data->force)
- if (do_run_update_procedure(update_data, &err))
+ if (run_update_procedure(update_data, &err))
return 1;
if (update_data->recursive) {
These two functions were prefixed with 'do' before the shell-to-C conversion because they were utility functions meant to be called by their non-prefixed counterpart. Since those callers don't exist anymore, and these functions can now be used directly, let's rename them to signal this fact. Mentored-by: Christian Couder <christian.couder@gmail.com> Mentored-by: Shourya Shukla <periperidip@gmail.com> Signed-off-by: Atharva Raykar <raykar.ath@gmail.com> --- builtin/submodule--helper.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)