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