Re: [PATCHES] Refactoring lock.c

2005-02-03 Thread Heikki Linnakangas
On Thu, 3 Feb 2005, Neil Conway wrote: On Wed, 2005-02-02 at 21:41 +0200, Heikki Linnakangas wrote: There's two almost identical pieces of code in LockRelease and LockReleaseAll that do the opposite of GrantLock. Here's a small patch that replaces those pieces with a static UnGrantLock function.

Re: [PATCHES] Refactoring lock.c

2005-02-03 Thread Neil Conway
Heikki Linnakangas wrote: There's two almost identical pieces of code in LockRelease and LockReleaseAll that do the opposite of GrantLock. Here's a small patch that replaces those pieces with a static UnGrantLock function. Applied to HEAD with a few trivial editorial fixes. Thanks for the

[PATCHES] Refactoring lock.c

2005-02-02 Thread Heikki Linnakangas
Hi, There's two almost identical pieces of code in LockRelease and LockReleaseAll that do the opposite of GrantLock. Here's a small patch that replaces those pieces with a static UnGrantLock function. This is preparation for the two-phase commit patch, since that introduces more calls to

Re: [PATCHES] Refactoring lock.c

2005-02-02 Thread Neil Conway
On Wed, 2005-02-02 at 21:41 +0200, Heikki Linnakangas wrote: There's two almost identical pieces of code in LockRelease and LockReleaseAll that do the opposite of GrantLock. Here's a small patch that replaces those pieces with a static UnGrantLock function. LockReleaseAll() did not update