Re: remove some STATUS_* symbols

Michael Paquier <[email protected]> Fri, 10 Jan 2020 14:23:32 +0900

Newsgroups	gmane.comp.db.postgresql.devel.general
Message-ID	<[email protected]>

On Thu, Jan 09, 2020 at 11:15:08AM +0100, Peter Eisentraut wrote:
> You mean put he subsequent GrantLock() calls into LockCheckConflicts()? That
> would technically save some duplicate code, but it seems weird, because
> LockCheckConflicts() is notionally a read-only function that shouldn't
> change state.

Nah.  I was thinking about the first part of this "if" clause
LockCheckConflicts is part of here:
   if (lockMethodTable->conflictTab[lockmode] & lock->waitMask)
       status = STATUS_FOUND;
   else
       status = LockCheckConflicts(lockMethodTable, lockmode,
                                   lock, proclock);

But now that I look at it closely it messes up heavily with
ProcSleep() ;)
--
Michael

signature.asc (application/pgp-signature, 833 B)