I have updated all instances of "ACK Value" and "ACK Counter Value" within the BIP to be "ACK-counter".
Updates made to align with term used through out the BIP: Line 213 - adding explicit mention of "ACK-counter" Line 229 - adding explicit mention of "ACK-counter" Line 257 - replaced "ACK-value" with "ACK-counter" Line 288 - replaced "ACK value" with "ACK-counter"
I've assumed that all mentions of ACK value and ACK counter are synonymous and therefore have updated all instances of this within the BIP to be "ACK-counter". I've noticed that M4 shows the different terms on two different lines (Line 256 and Line 257), which has made me think that potentially these terms aren't synonymous.
Updates made to align with term used through out the BIP:
Line 213 - adding explicit mention of "ACK-counter"
Line 229 - adding explicit mention of "ACK-counter"
Line 257 - replaced "ACK-value" with "ACK-counter"
Line 288 - replaced "ACK value" with "ACK-counter"
I don't have a problem with it.
It seems like a very small change to me. But the author of the pull request insists that it would have been easier for him to understand, had it been worded like this. So it is fine by me.