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)