This used to say "nsubxcnt isn't decreased when subtransactions
abort", but there's no variable called nsubxcnt. Commit
8548ddc61b
changed it to "subxcnt", among other typo fixes, but that was wrong
too: the comment actually talks about txn->nsubtxns. That's the field
that's incremented but never decremented and is used for the
allocation earlier in the function.
snap->subxip[i++] = txn->xid;
/*
- * subxcnt isn't decreased when subtransactions abort, so count manually.
- * Since it's an upper boundary it is safe to use it for the allocation
- * above.
+ * txn->nsubtxns isn't decreased when subtransactions abort, so count
+ * manually. Since it's an upper boundary it is safe to use it for the
+ * allocation above.
*/
snap->subxcnt = 1;