Subject: Re: [PATCH 2/2] close_lock_file(): new function in the lockfile API

From: Junio C Hamano <[email-masked]>
Date: Wed, 16 Jan 2008 14:55:41 -0800

Brandon Casey <[email-masked]> writes:

> Mainly, I prefer to not modify the data structures when a failure occurs.

Ok.  Is the rest of your patch that fixes callers Ok with that
semantics?  If so, I'd agree that is probably cleaner.  I'll
scrap the one we are discussing, resurrecting only the api
documentation part, and replace it with the lockfile.c changes
from your patch, along with the fixes to callers.