Update tracing.md #25854

pull itsliamdowd wants to merge 1 commits into bitcoin:master from itsliamdowd:patch-1 changing 1 files +5 −5

itsliamdowd commented at 9:12 PM on August 15, 2022: none

Update tracing.md
Update tracing.md 4a7609ebda
fanquake added the label Docs on Aug 15, 2022

in doc/tracing.md:120 in 4a7609ebda

 116 | @@ -117,7 +117,7 @@ added to and removed (spent) from the cache when we connect a new block.
 117 |  `getblocktemplate` call `TestBlockValidity()`, which applies the UTXO set
 118 |  changes to a temporary cache. Similarly, mempool consistency checks, which are
 119 |  frequent on regtest, also apply the the UTXO set changes to a temporary cache.
 120 | -Changes to the _main_ UTXO cache and to temporary caches trigger the tracepoints.
 121 | +Changes to the _main_ UTXO cache and temporary caches trigger the tracepoints.

luke-jr commented at 1:03 AM on August 19, 2022:

Should this be "or"?

0xB10C commented at 8:32 AM on August 26, 2022:

yes, changes to all UTXO caches affect this. I'm not sure if main or temporary is clearer than main and temporary.

in doc/tracing.md:79 in 4a7609ebda
```
  75 | @@ -76,8 +76,8 @@ the passed message.
  76 |  
  77 |  #### Tracepoint `net:outbound_message`
  78 |  
  79 | -Is called when a message is send to a peer over the P2P network. Passes
```
0xB10C commented at 8:28 AM on August 26, 2022:

nit: technically, this happens before a message is send to a peer, but to a user it happens nearly simultaneously. Don't think it's worth changing this.
fanquake commented at 8:07 PM on August 31, 2022: member

@SkiingIsFun123 are you planning on updating this? Note that you'll need to squash all commits, and you can write a proper commit message, i.e 'doc: commit message explaining change'
fanquake commented at 5:56 PM on September 4, 2022: member

Closing up for grabs.
fanquake closed this on Sep 4, 2022
fanquake added the label Up for grabs on Sep 4, 2022
bitcoin locked this on Sep 4, 2023
fanquake removed the label Up for grabs on Jul 23, 2025

Contributors

Labels