Re: [PATCH] do not pretend sha1write returns errors

2013-12-22 Thread Thomas Rast
Jeff King writes: > The sha1write function returns an int, but it will always be > "0". The failure-prone parts of the function happen in the > "flush" callback, which cannot pass an error back to us. So > we just end up calling die() during the flush. > > Let's just drop the return value altoget

[PATCH] do not pretend sha1write returns errors

2013-12-21 Thread Jeff King
The sha1write function returns an int, but it will always be "0". The failure-prone parts of the function happen in the "flush" callback, which cannot pass an error back to us. So we just end up calling die() during the flush. Let's just drop the return value altogether, as it only confuses caller