Concept (N)ACK issues

[maflcko]

Parsing will never get 100% accuracy, unless everyone is forced to use an exact format.

Here is a collection of parsing issues.

Misdetection

• HACK Concept ACK? #26 (comment)
• not a NACK, retract my NACK Switch hardened derivation marker to h bitcoin/bitcoin#26076 (comment), Remove -mempoolfullrbf option bitcoin/bitcoin#26525 (comment)

They should be fixed by providing a fresh review, or by downvoting the DrahtBot comment. ("If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.")

Review going stale

• ACK hash, then ACK kernel: De-globalize validation caches bitcoin/bitcoin#30141 (comment)

Workarounds:

• Edit the later comments by this user after their review to not contain ACK, or
• Have the user resubmit their review

An alternative code fix would be to ignore ACK after an ACK hash. However, this may interfere with the stale-ACK detection, so it should only kick in when ACK hash is not stale.

[maflcko]

An alternative code fix would be to ignore ACK after an ACK hash. However, this may interfere with the stale-ACK detection, so it should only kick in when ACK hash is not stale.

Fixed it in cf46398.

This also means that a ACK hash, followed by NACK may or may not be misdetected as ACK.

[ryanofsky]

Thanks for fixing!

This also means that a ACK hash, followed by NACK may or may not be misdetected as ACK.

This doesn't necessarily sound too bad if the PR changed and it shows up a stale ACK. Maybe worse in some other situations, though, but it is hard to be perfect.