unify capitalization of "bitcoin address" #9064

pull smatthewenglish wants to merge 1 commits into bitcoin:master from smatthewenglish:changeCaps changing 12 files +53 −53
  1. smatthewenglish commented at 3:15 PM on November 2, 2016: contributor

    in some places it was "Bitcoin address" in other places it was "bitcoin address".

    I've unified them to make them consistent with the idea that (capital "B") Bitcoin refers to the protocol.

    Of course I left it where it was the start of a sentence. Also I ignored everything in locale and release notes

  2. MarcoFalke commented at 3:24 PM on November 2, 2016: member

    Huh, personally, I'd call it "Bitcoin address" with a capital "B", so it is correct the way it was before.

    Also, we should make sure that translation strings are not touched unnecessarily. Please note that translators have to go through all of them, even if "nothing"/whitespace/capitals changed.

  3. MarcoFalke added the label Docs and Output on Nov 2, 2016
  4. smatthewenglish commented at 3:39 PM on November 2, 2016: contributor

    to me it doesn't matter but it seems it should be consistent.

    I was thinking that Bitcoin was the protocol itself so that if it was talking about one address to receive bitcoins then it would be lowercase.

    but anyway- in that case there are many instances of "bitcoin address"

    should we change those to "Bitcoin address"?

    I didn't touch anything in the locale directory, I guess that's the translator stuff isn't it?

  5. Leviathn commented at 4:46 PM on November 2, 2016: none

    One deposits bitcoin to a Bitcoin address - S-Matthew, revert this and change the "bitcoin address" if you want to get this through.

  6. converting "bitcoin address" to "Bitcoin address" 38d2054fe1
  7. smatthewenglish force-pushed on Nov 2, 2016
  8. smatthewenglish commented at 5:50 PM on November 2, 2016: contributor

    ok cool- so now I've changed all instances of "bitcoin address" to "Bitcoin address"

    I ignored everything in release notes since jonasschnelli said that's not consequential

  9. Xekyo approved
  10. Xekyo commented at 5:57 PM on November 8, 2016: member

    ACK (I've looked over the changed code.)

  11. MarcoFalke commented at 7:59 PM on December 2, 2016: member

    I tried to look up the "correct" capitalization, but there seem to be some discrepancy and no clear and "correct" way. I think it is fine to use both capitalizations exchangeably.

    I understand the need to unify the capitalization, but if that means we possibly need to change all of that back to the other way next year, I'd prefer to not change it at all.

    Closing for now. (Heh, maybe wait until there is a wikipedia article Capitalization of "Bitcoin", just like Capitalization of "Internet")

  12. MarcoFalke closed this on Dec 2, 2016

  13. smatthewenglish commented at 11:24 PM on December 2, 2016: contributor

    yeah, I feel you, but in my idea, like- Wikipedia is fully of a lot of crap- I know- since I put a bunch of it there. This GitHub code is sort of like the canonical reference, isn't it? So for instance- I feel like if the capitalization were firmly established here then others would adopt to the way we make it, and not really the other way around, you know what I mean?

  14. MarcoFalke commented at 10:08 AM on December 13, 2016: member

    We don't act as a reference for spelling (luckily), but I am not strongly against this change, if it at least makes our spelling consistent.

  15. MarcoFalke reopened this on Dec 13, 2016

  16. smatthewenglish commented at 10:11 PM on December 13, 2016: contributor

    Checkout the footnote on page 2 from Sarah Meiklejohn's Fistful of Bitcoins paper.

    Following established convention, we use the capitalized term Bitcoin
    when referring to the payment system and peer-to-peer network
    and the lowercase term bitcoin (abbreviated BTC), when referring
    to the unit of currency.

    The pull request under consideration applies this convention in a consistent way to the codebase.

    Resources: https://cseweb.ucsd.edu/~smeiklejohn/files/imc13.pdf

  17. fanquake commented at 1:46 PM on January 12, 2017: member

    Closing this. Doesn't seem to be consensus. Needs a rebase.

  18. fanquake closed this on Jan 12, 2017

  19. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-21 21:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me