Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: > Stefan Beller <stefanbeller@xxxxxxxxx> writes: > >> + if (is_bare && !allow_bare) { >> + unlink(name); >> + fprintf(stderr, "corrupt mailbox\n"); >> + exit(1); >> + } > > Not directly related to your patch, but shouldn't this be using > > die(_("corrupt mailbox")); > > instead? Doesn't the exit status of mailsplit matter to its existing invokers? -- 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