On Sun, Dec 29, 2019 at 11:33:34AM +0100, Peter Eisentraut wrote: > Attached are two patches to remove these two symbols. STATUS_FOUND can be > replaced by a simple bool. STATUS_WAITING is replaced by a separate enum.
Patch 0001 looks good to me, but I got to wonder why the check after waitMask in LockAcquireExtended() is not done directly in LockCheckConflicts(). Regarding patch 0002, I am not sure that the addition of ProcWaitStatus brings much though in terms of code readability. -- Michael
signature.asc
Description: PGP signature
