On Tue, Aug 14, 2018 at 12:45 AM Stefan Beller <sbeller@xxxxxxxxxx> wrote: > +static int module_update_module_mode(int argc, const char **argv, const char *prefix) > +{ > + const char *path, *update = NULL; > + int just_cloned; > + struct submodule_update_strategy update_strategy = { .type = SM_UPDATE_CHECKOUT }; > + > + if (argc < 3 || argc > 4) > + die("submodule--helper update-module-clone expects <just-cloned> <path> [<update>]"); Maybe _() ? -- Duy