Junio C Hamano <gitster@xxxxxxxxx> writes: > Miriam Rubio <mirucam@xxxxxxxxx> writes: > >> + if (res < 0 || 128 <= res) { >> + error(_("bisect run failed: exit code %d from" >> + " '%s' is < 0 or >= 128"), res, command.buf); > > Good now. Oh, while asking for better test coverage, it is somewhat surprising that the broken error condition check in v2 was never noticed. Can we add a test for this, too? Thanks.