On Sun, Feb 24, 2008 at 11:59:15PM -0800, Junio C Hamano wrote: > mkoegler@xxxxxxxxxxxxxxxxx (Martin Koegler) writes: > > How should I define the return value of fsck_walk in the presence of > > multiple errors? > > Returning error is fine. That is not what I was talking about. > > I was talking about an early return in the code, that does not > callback once you find an error. That is not what I was talking about. We need to define the return code of fsck_walk in the case of multiple errors. What error (of all possible) should it return? What about this new behaviour: Callback return values: * 0 everything OK * > 0 error, but continue * < 0 error, but stop The return value of fsck_walk would be: * -1, if fsck_walk discoveres an error (eg. not parseable object) * first callback return value < 0 (and stops further processing in this case) * first callback return value > 0 (if not callback return value < 0) * 0, if no errors are discovered mfg Martin Kögler - 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