MOVEONLY: Move logging code from util.{h,cpp} to new files. #13021

pull jimpo wants to merge 1 commits into bitcoin:master from jimpo:logging-files changing 5 files +412 −377
  1. jimpo commented at 5:19 PM on April 18, 2018: contributor

    Split out first commit from #12954 to reduce amount of rebasing necessary.

    This introduces a cyclic dependency between logging and util that should be cleaned up in a future PR.

  2. MOVEONLY: Move logging code from util.{h,cpp} to new files. b77b6e2345
  3. MarcoFalke added the label Refactoring on Apr 18, 2018
  4. MarcoFalke commented at 5:39 PM on April 18, 2018: member

    utACK b77b6e23457dcbf160a3d586eca9df7a8d0bb351 (Lines are kept in the same order, which makes it easier to verify moveonly-ness)

  5. Empact commented at 9:14 PM on April 18, 2018: member

    utACK b77b6e2 also confirmed that the lines are repeated faithfully

  6. ajtowns commented at 2:25 AM on April 20, 2018: contributor

    utACK b77b6e23457dcbf160a3d586eca9df7a8d0bb351

    Comparison with git diff HEAD^ --color-moved=zebra looks good. The cyclic dependency is logging.cpp to util.h to logging.h and there's a plan for breaking that cycle in #12954. Agree that this would be good to merge soon, since it's essentially a blocker for anything touching logging code.

  7. Empact cross-referenced this on Apr 20, 2018 from issue util: Refactor logging code into a global object by jimpo
  8. sipa commented at 4:58 AM on April 20, 2018: member

    utACK b77b6e23457dcbf160a3d586eca9df7a8d0bb351. Verified move-only.

  9. sipa merged this on Apr 20, 2018
  10. sipa closed this on Apr 20, 2018

  11. sipa referenced this in commit 8b262eb2d8 on Apr 20, 2018
  12. jimpo deleted the branch on Apr 20, 2018
  13. Bushstar cross-referenced this on Apr 20, 2018 from issue commits from bitcoin/master by Bushstar
  14. sipsorcery referenced this in commit 6fd771e946 on Apr 20, 2018
  15. Empact cross-referenced this on May 18, 2018 from issue [moveonly] Fix CConnman template methods to be fully-defined in net.h by Empact
  16. random-zebra cross-referenced this on Mar 25, 2020 from issue [Util] Refactor logging code into a global object by random-zebra
  17. Fuzzbawls referenced this in commit c94f2412af on Apr 14, 2020
  18. PastaPastaPasta referenced this in commit 12309092de on Jul 18, 2020
  19. PastaPastaPasta referenced this in commit fb21ce6d7e on Jul 19, 2020
  20. str4d referenced this in commit b42582395b on Oct 9, 2020
  21. str4d cross-referenced this on Oct 9, 2020 from issue MOVEONLY: Move logging code from util.{h,cpp} to new files by str4d
  22. zkbot referenced this in commit ebc8de390c on Oct 9, 2020
  23. gades referenced this in commit dcf9b6220e on Jun 30, 2021
  24. bitcoin 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-05-20 06:55 UTC