Subject: Re: [PATCH 2/4] remote: print an error if refspec cannot be removed

From: Junio C Hamano <[email-masked]>
Date: Thu, 12 Sep 2024 09:22:13 -0700

Patrick Steinhardt <[email-masked]> 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();
	}