I think the patch maybe is not quite correct for all the flags.
For example,
@@ -7607,44 +7615,44 @@ are available since protocol version 3.
<varlistentry>
<term>Int8</term>
<listitem><para>
- Flags; currently unused (must be 0).
+ Flags (uint8); currently unused.
</para></listitem>
</varlistentry>
AFAIK, even though the flags are "unused", the code still insists that
most (or all? Please check the code) of these flag values MUST be 0,
so I think that this zero value requirement ought to be indicated in
the docs using the "Int8(0)" convention [1]. For example,
BEFORE
Int8
Flags (uint8); currently unused.
AFTER
Int8(0)
Flags (uint8); currently unused.
------
[1] https://www.postgresql.org/docs/devel/protocol-message-types.html
Kind Regards,
Peter Smith.
Fujitsu Australia.