Jeff King <peff@xxxxxxxx> writes:
>> I would however agree that when we do mean 255 we should probably write
>> 255, not (-1). It is easier to document things that way.
>
> I started to fix the callsites that Stephan mentioned, but it really is
> convenient to be able to 'return error("foo")' (or even return
> func_that_calls_error(), and tracking down deep calls is time consuming
> and error prone). So maybe we should just enhance the change from
> 2488df84 and special case "-1" into "1"?
Didn't the patch to testsuite that triggered this thread talk about "small
negative integer" not "-1"? I suspect there might be other negative
return values from cmd_foo(), although I haven't checked.
Is it that somebody do not want 255 exit value, or anything that has 7th
bit set? 2488df8 (builtin run_command: do not exit with -1., 2007-11-13)
suggests otherwise at least for Windows runtime, so what we currently have
that does extra truncation ourselves might be sufficient.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
