docs: Fix conflicting statements about initialization in developer notes #12896

pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:remove-conflicing-statements changing 1 files +12 −15
  1. practicalswift commented at 9:04 PM on April 5, 2018: contributor

    Fix conflicting statements about initialization in developer notes.

    Context: #12785 (comment)

  2. in doc/developer-notes.md:453 in 675452bcb8 outdated
     449 | @@ -450,12 +450,17 @@ C++ data structures
     450 |  
     451 |  - Vector bounds checking is only enabled in debug mode. Do not rely on it
     452 |  
     453 | -- Make sure that constructors initialize all fields. If this is skipped for a
    


    laanwj commented at 9:06 PM on April 5, 2018:

    Please don't remove this part about optimization.


    practicalswift commented at 9:18 PM on April 5, 2018:

    Fixed! :-)

  3. practicalswift force-pushed on Apr 5, 2018
  4. practicalswift force-pushed on Apr 5, 2018
  5. docs: Fix conflicting statements about initialization in developer notes b119e78733
  6. practicalswift force-pushed on Apr 5, 2018
  7. practicalswift commented at 9:26 PM on April 5, 2018: contributor

    @laanwj Updated. Please re-review :-)

  8. promag commented at 10:40 PM on April 5, 2018: member

    ACK b119e78, looks good.

  9. fanquake added the label Docs on Apr 5, 2018
  10. MarcoFalke commented at 1:15 PM on April 6, 2018: member

    ACK b119e78

  11. laanwj merged this on Apr 7, 2018
  12. laanwj closed this on Apr 7, 2018

  13. laanwj referenced this in commit b15485ecca on Apr 7, 2018
  14. MarcoFalke deleted a comment on Apr 7, 2018
  15. PastaPastaPasta referenced this in commit 9c357d0dad on Jul 17, 2020
  16. PastaPastaPasta referenced this in commit e88e7682a7 on Jul 17, 2020
  17. PastaPastaPasta referenced this in commit 1f173521f1 on Jul 19, 2020
  18. PastaPastaPasta referenced this in commit 5d007d8b3f on Jul 21, 2020
  19. deadalnix referenced this in commit f97ac61737 on Nov 17, 2020
  20. practicalswift deleted the branch on Apr 10, 2021
  21. DrahtBot locked this on Aug 16, 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: 2026-04-16 15:15 UTC

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