Update tracing.md #25854

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

SkiingIsFun123 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

Contributors
SkiingIsFun123 luke-jr 0xB10C fanquake

Labels
Docs Up for grabs