mirror of https://github.com/bitcoin/bitcoin.git synced 2026-09-13 06:04:42 +02:00

Files

Ryan Ofsky 735b1cf431 Merge bitcoin/bitcoin#34806 : refactor: logging: Various API improvements

02b2c41103 logging: use util/log.h where possible (Anthony Towns)
57d7495fe5 IWYU fixes (Anthony Towns)
611878b46f scripted-diff: logging: Drop LogAcceptCategory (Anthony Towns)
34332dba2f util/log, logging: Provide ShouldDebugLog and ShouldTraceLog instead of a generic ShouldLog (Anthony Towns)
abea304dd6 logging: Move GetLogCategory into Logger class (Anthony Towns)
58113e5833 util/log: Rename LogPrintLevel_ into detail_ namespace (Anthony Towns)
f69d1ae56d util/log: Provide util::log::NO_RATE_LIMIT to avoid rate limits (Anthony Towns)
72e92d67df logging: Protect ShrinkDebugFile by m_cs (Anthony Towns)
904c0d07bb util/stdmutex: Drop StdLockGuard (Anthony Towns)

Pull request description:

  `ShrinkDebugFile` now takes the logging mutex for its entire run; though it's only called in init so shouldn't have any races in the first place.

  Adds a `NO_RATE_LIMIT` tag that can be used with info/warning/error logs to avoid rate-limiting. This allows `LogPrintLevel_` to be restricted to being an internal API.

  The `GetLogCategory` function is moved out of the global namespace.

  `ShouldLog` is split into separate `ShouldDebugLog` and `ShouldTraceLog` so that filtering checks are somewhat more enforced via function signature checks.

  Redundant `LogAcceptCategory` function is removed.

  More files are pointed at util/log.h instead of logging.h.

ACKs for top commit:
  maflcko:
    review ACK 02b2c41103 📅
  sedited:
    Re-ACK 02b2c41103
  l0rinc:
    untested ACK 02b2c41103
  ryanofsky:
    Code review ACK 02b2c41103435d8dbaa77a526e484066471b2b8c! Overall a lot of nice improvements here.

Tree-SHA512: 3bffdca91afbe5c45a522815fe82e6f4cfa96529a4a243b29aad21234650502d6cac780126b584ee3e7ec129d8fdd50670d8a05036cc5c36e586b8c4c3563970

2026-05-21 23:01:09 -04:00

data

…

abort.cpp

…

abort.h

…

blockmanager_args.cpp

…

blockmanager_args.h

…

blockstorage.cpp

…

blockstorage.h

doc: clarify IWYU workflow

2026-04-29 18:10:39 +02:00

caches.cpp

Merge bitcoin/bitcoin#34806 : refactor: logging: Various API improvements

2026-05-21 23:01:09 -04:00

caches.h

…

chainstate.cpp

…

chainstate.h

…

chainstatemanager_args.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

chainstatemanager_args.h

…

coin.cpp

…

coin.h

…

coins_view_args.cpp

…

coins_view_args.h

…

connection_types.cpp

…

connection_types.h

…

context.cpp

…

context.h

…

database_args.cpp

…

database_args.h

…

eviction.cpp

…

eviction.h

…

interface_ui.cpp

…

interface_ui.h

…

interfaces.cpp

…

kernel_notifications.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

kernel_notifications.h

…

mempool_args.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

mempool_args.h

…

mempool_persist_args.cpp

…

mempool_persist_args.h

…

mempool_persist.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

mempool_persist.h

…

miner.cpp

Merge bitcoin/bitcoin#34806 : refactor: logging: Various API improvements

2026-05-21 23:01:09 -04:00

miner.h

…

mini_miner.cpp

…

mini_miner.h

…

minisketchwrapper.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

minisketchwrapper.h

…

peerman_args.cpp

…

peerman_args.h

…

protocol_version.h

…

psbt.cpp

Replace PSBT.tx with PSBT::GetUnsignedTx and PSBT::GetUniqueID

2026-04-29 15:04:22 -07:00

psbt.h

…

README.md

…

timeoffsets.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

timeoffsets.h

…

transaction.cpp

…

transaction.h

…

txdownloadman_impl.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

txdownloadman_impl.h

…

txdownloadman.h

…

txorphanage.cpp

Merge bitcoin/bitcoin#34806 : refactor: logging: Various API improvements

2026-05-21 23:01:09 -04:00

txorphanage.h

…

txreconciliation.cpp

logging: use util/log.h where possible

2026-05-16 03:36:51 +10:00

txreconciliation.h

…

types.h

mining: drop unused include_dummy_extranonce option

2026-05-13 18:27:48 +02:00

utxo_snapshot.cpp

…

utxo_snapshot.h

…

warnings.cpp

…

warnings.h

…

README.md

src/node/

The src/node/ directory contains code that needs to access node state (state in CChain, CBlockIndex, CCoinsView, CTxMemPool, and similar classes).

Code in src/node/ is meant to be segregated from code in src/wallet/ and src/qt/, to ensure wallet and GUI code changes don't interfere with node operation, to allow wallet and GUI code to run in separate processes, and to perhaps eventually allow wallet and GUI code to be maintained in separate source repositories.

As a rule of thumb, code in one of the src/node/, src/wallet/, or src/qt/ directories should avoid calling code in the other directories directly, and only invoke it indirectly through the more limited src/interfaces/ classes.

This directory is at the moment sparsely populated. Eventually more substantial files like src/validation.cpp and src/txmempool.cpp might be moved there.