Re: [PATCH] read-cache: call verify_hdr() in a background thread

[Jonathan Nieder <jrnieder@xxxxxxxxx>]

Jeff Hostetler wrote:
> On 3/24/2017 12:35 PM, Jonathan Nieder wrote:

>> What happens if there is an error before the code reaches the end of
>> the function?  I think there needs to be a verify_hdr_finish call in
>> the 'unmap:' cleanup section.
>
> But the "unmap" section calls die().  Do need to join first ??
> (It's OK if we do, just asking.)

Good point.  Now I wonder why the 'unmap:' section exists at all.

	$ git log -Sunmap: read-cache.c 
	commit e83c5163316f89bfbde7d9ab23ca2e25604af290
	Author: Linus Torvalds <torvalds@xxxxxxxxxxxxxxx>
	Date:   Thu Apr 7 15:13:13 2005 -0700

	    Initial revision of "git", the information manager from hell

So the unmap: section has been there since day one.  At that point
it used 'return error(...)', so it made more sense.  Where did the
die() come from, then?

	commit 5d1a5c02e8ac1c16688ea4a44512245f25a49f8a
	Author: Linus Torvalds <torvalds@xxxxxxxx>
	Date:   Sat Oct 1 13:24:27 2005 -0700

	    [PATCH] Better error reporting for "git status"

I think it should be safe to eliminate the 'unmap' section altogether
and that that change just forgot to, but in any case it's orthogonal
to your patch.

Thanks and sorry for the confusion,
Jonathan