Patrick Steinhardt <ps@xxxxxx> writes: > I don't think we want to return the error code from `error()`, do we? > `set_branches()` is wired up as a subcommand, so we'd ultimately do > `exit(-1)` instead of `exit(1)` if we returned the `error()` code here. Hmph, I thought there was somebody doing !! to canonicalize the return value to exit status in the call chain. ... goes and looks again ... After finding the subcommand in fn, cmd_remote() ends with if (fn) { return !!fn(argc, argv, prefix); } else { ... return !!show_all(); }