Fixed typos #23776

pull DavidRomanovizc wants to merge 3 commits into bitcoin:master from DavidRomanovizc:master changing 3 files +10 −9
  1. DavidRomanovizc commented at 7:23 pm on December 14, 2021: none
    Fixed typos in some files And also added tabs in the context.h file to make the code read better
  2. ✏️ Fix typos 0605fce549
  3. ✏️ Fix typos
    and added tabs in namespace
    d58159d440
  4. ✏️ Fix typos 46292b75cc
  5. ghost commented at 7:48 pm on December 14, 2021: none

    Not sure about changing block chain to blockchain

    See #5242

    All PRs that tried changing it since 2014 got NACKs

  6. DrahtBot added the label Block storage on Dec 14, 2021
  7. mzumsande commented at 8:11 pm on December 14, 2021: member
    NACK, none of these are actual typos. Namespaces aren’t indented (https://github.com/bitcoin/bitcoin/blob/master/doc/developer-notes.md#coding-style-c), -Wbitwise-instead-of-logical begins with a W because that’s compiler warning syntax, and the other “typos” change the meaning to the worse, or are a matter of taste at best.
  8. Zero-1729 commented at 9:36 pm on December 14, 2021: contributor

    NACK for the reasons expressed above.

    These changes are not fixing typos.

  9. fanquake closed this on Dec 14, 2021

  10. DrahtBot locked this on Dec 14, 2022

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: 2024-11-23 09:12 UTC

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