Andres Freund <and...@anarazel.de> writes:
> However, jsonexpr/EEOP_JSONEXPR is 296 bytes, and
> hashedscalararrayop/EEOP_HASHED_SCALARARRAYOP is 64 bytes, even though the
> limit is 40 bytes.
Oops.
> Maybe it's worth sticking a StaticAssert() for the struct size
> somewhere.
Indeed. I thought we had one already.
> I'm a bit wary about that being too noisy, there are some machines with
> odd alignment requirements. Perhaps worth restricting the assertion to
> x86-64 + armv8 or such?
I'd put it in first and only reconsider if it shows unfixable problems.
regards, tom lane
