On 14/06/2024 16:56, Maxim Orlov wrote:
+static inline int
+MXOffsetToFlagsOffset(MultiXactOffset offset)
+{
+       int             flagsoff;
+
+       offset /= MULTIXACT_MEMBERS_PER_MEMBERGROUP;
+       offset %= MULTIXACT_MEMBERGROUPS_PER_PAGE;
+       flagsoff = offset * MULTIXACT_MEMBERGROUP_SIZE;
+
+       return flagsoff;
+}

I found this reuse of the 'offset' variable a bit confusing, so I added separate local variables for each step.

Committed with that change, thanks!

--
Heikki Linnakangas
Neon (https://neon.tech)



Reply via email to