On Mon, Nov 30, 2015 at 05:47:42PM -0500, David Turner wrote: > In verify_pack, a caller-supplied verification function is called. > The function returns an int. If that return value is non-zero, > verify_pack should fail. > > The only caller of verify_pack is in builtin/fsck.c, whose verify_fn > returns a meaningful error code (which was then ignored). Now, fsck > might return a different error code (with more detail). This would > happen in the unlikely event that a commit or tree that is a valid git > object but not a valid instance of its type gets into a pack. > > Signed-off-by: David Turner <dturner@xxxxxxxxxxxxxxxx> > --- > pack-check.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Thanks, this looks good to me. -Peff -- 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