On Thu, Apr 5, 2018 at 9:50 AM, Christian Couder <christian.couder@xxxxxxxxx> wrote: > > So maybe we can get rid of `result` and have something like: > > if (argc < 1) { > error(_("at least one argument is required")); > usage_with_options(git_stash_helper_usage, options); Maybe we could also simplify these 2 lines by using usage_msg_opt(). > } > > if (!strcmp(argv[0], "apply")) > return apply_stash(argc, argv, prefix); > > error(_("unknown subcommand: %s"), argv[0]); > usage_with_options(git_stash_helper_usage, options); > }