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)
