Hi,
On 2023-04-03 12:00:30 -0700, Andres Freund wrote:
> It's not great, I agree. I tried to make it easier to read in this version by
> a) changing GetVisibilityMapPins() as I proposed
> b) added a new variable "recheckVmPins", that gets set in
> if (unlockedTargetBuffer)
> and
> if (otherBuffer != InvalidBuffer)
> c) reformulated comments
I pushed this version a couple hours ago, after a bit more polishing.
Greetings,
Andres Freund