Remove poorly worded and duplicated comment

Move line of code to avoid need for duplicated comment

Brought to attention by Masahiko Sawada

diff --git a/src/backend/utils/time/snapmgr.c b/src/backend/utils/time/snapmgr.c
index 3a3a25a0c36ba5577ca53694f34202b01cec9e73..5fa665eafc4a4859e81940bd796113aa4024ba40 100644 (file)
--- a/src/backend/utils/time/snapmgr.c
+++ b/src/backend/utils/time/snapmgr.c
@@ -2039,14 +2039,6 @@ SerializeSnapshot(Snapshot snapshot, char *start_address)
    serialized_snapshot.whenTaken = snapshot->whenTaken;
    serialized_snapshot.lsn = snapshot->lsn;
 
-   /*
-    * Ignore the SubXID array if it has overflowed, unless the snapshot was
-    * taken during recovey - in that case, top-level XIDs are in subxip as
-    * well, and we mustn't lose them.
-    */
-   if (serialized_snapshot.suboverflowed && !snapshot->takenDuringRecovery)
-       serialized_snapshot.subxcnt = 0;
-
    /* Copy struct to possibly-unaligned buffer */
    memcpy(start_address,
           &serialized_snapshot, sizeof(SerializedSnapshotData));
@@ -2063,6 +2055,9 @@ SerializeSnapshot(Snapshot snapshot, char *start_address)
     * snapshot taken during recovery; all the top-level XIDs are in subxip as
     * well in that case, so we mustn't lose them.
     */
+   if (serialized_snapshot.suboverflowed && !snapshot->takenDuringRecovery)
+       serialized_snapshot.subxcnt = 0;
+
    if (serialized_snapshot.subxcnt > 0)
    {
        Size        subxipoff = sizeof(SerializedSnapshotData) +