bip159: Clarify that there is only one threshold #617

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:Mf1712-bip159rework changing 1 files +5 −7

MarcoFalke commented at 6:56 PM on December 19, 2017: member
@jonasschnelli

Changes:
- The last section mentions "thresholds", while only a single threshold of 244 is specified in the bip. Clarify by changing to "threshold"
- Remove internal Bitcoin Core implementation detail. This is not relevant for a bip.
- Remove word "additional" when referring to safety buffer, since the safety buffer is the only buffer and not an additional one.
- Remove empty section "references"
- Add missing link to signaling implementation.
bip159: Clarify that there is only one threshold fa810b3bff
jonasschnelli commented at 6:57 PM on December 19, 2017: contributor

Thanks! ACK f455a24da7b098d390b6405ae19826d0c5684c3e
bip159: Add missing link to implementation fa9a4f307e
MarcoFalke commented at 7:01 PM on December 19, 2017: member

Added a link to the signaling implementation after your review.
jonasschnelli commented at 7:06 PM on December 19, 2017: contributor

Also ACK fa9a4f307eb241b69f04681c3da35b7f69cf9fd5
luke-jr merged this on Dec 19, 2017
luke-jr closed this on Dec 19, 2017
MarcoFalke deleted the branch on Dec 19, 2017

Contributors