From 0bf4aa4d0148a03c030b4d8e25dd5053dd2c336e Mon Sep 17 00:00:00 2001
From: Jacob Champion <jacob.champion@enterprisedb.com>
Date: Fri, 30 Jan 2026 10:43:22 -0800
Subject: [PATCH v6 4/4] squash! doc: Expand upon protocol versions and
 extensions

See screenshot-2. I don't like the way this looks.
---
 doc/src/sgml/protocol.sgml | 52 +++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 31 deletions(-)

diff --git a/doc/src/sgml/protocol.sgml b/doc/src/sgml/protocol.sgml
index 89503571e9e..8a38cbab812 100644
--- a/doc/src/sgml/protocol.sgml
+++ b/doc/src/sgml/protocol.sgml
@@ -327,33 +327,14 @@
     shows the current list of protocol extension parameters.
    </para>
 
-   <table id="protocol-extensions-table" rowheader="firstcol">
-    <title>Protocol Extensions</title>
-
-    <tgroup cols="5">
-     <!-- For spanning, below. -->
-     <colspec colname="first" colsep="0"/>
-     <colspec colname="name" colnum="2"/>
-     <colspec colname="vals" colnum="3"/>
-     <colspec colname="support" colnum="4"/>
-     <colspec colname="last" colnum="5"/>
-
-     <!--
-       "all" spans the width of the table; it is used for the section headers.
-       "reserved" spans the Values and Supported columns, for the Reserved
-       names which define neither.
-     -->
-     <spanspec spanname="all" namest="first" nameend="last"/>
-     <spanspec spanname="reserved" namest="vals" nameend="support"/>
+   <table id="protocol-extensions-table">
+    <title>Supported Protocol Extensions</title>
+
+    <tgroup cols="4">
+     <colspec colname="last" colnum="4"/>
 
      <thead>
       <row>
-       <!--
-         Row header. Use namest="name" to skip to the next column.
-         XXX this causes a thin "indent" effect for the other rows
-       -->
-       <entry/>
-
        <entry>Parameter&nbsp;Name</entry>
        <entry>Values</entry>
        <entry>Supported&nbsp;by</entry>
@@ -362,19 +343,29 @@
      </thead>
 
      <tbody>
-      <row><entry spanname="all">Defined</entry></row>
-
       <row>
        <entry namest="last" align="center" valign="middle">
         <emphasis>(No supported protocol extensions are currently defined.)</emphasis>
        </entry>
       </row>
+     </tbody>
+    </tgroup>
+   </table>
+
+   <table id="reserved-protocol-extensions-table">
+    <title>Reserved Protocol Extensions</title>
 
-      <row><entry spanname="all">Reserved</entry></row>
+    <tgroup cols="2">
+     <thead>
+      <row>
+       <entry>Parameter&nbsp;Name</entry>
+       <entry>Description</entry>
+      </row>
+     </thead>
 
+     <tbody>
       <row>
-      <entry namest="name"><literal>_pq_.<replaceable>[name]</replaceable></literal></entry>
-      <entry spanname="reserved">-</entry>
+      <entry><literal>_pq_.<replaceable>[name]</replaceable></literal></entry>
       <entry>Any other parameter names beginning with <literal>_pq_.</literal>,
         that are not defined above, are reserved for future protocol expansion.
         Servers <emphasis>must</emphasis> reject any that are received from a
@@ -385,8 +376,7 @@
       </row>
 
       <row>
-      <entry namest="name"><literal>_pq_.test_protocol_negotiation</literal></entry>
-      <entry spanname="reserved">-</entry>
+      <entry><literal>_pq_.test_protocol_negotiation</literal></entry>
       <entry>Reserved for protocol greasing. libpq may send this extension to
         test that servers and middleware properly implement protocol extension
         negotiation. Servers <emphasis>must not</emphasis> add special-case
-- 
2.34.1