Unless debugging, show a more useful format for the askfors #1344
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
ACK. |
src/net.h
Outdated
There was a problem hiding this comment.
If you indent, can you use 4 spaces :)?
Contributor
Author
There was a problem hiding this comment.
4 spaces would be misleading as it would look like it was within the logic of the preceeding line.
Member
There was a problem hiding this comment.
Either keep it on the same line, or use 4 or 8 spaces as indent.
Contributor
Author
There was a problem hiding this comment.
ok, changed (although I'm fairly certain it was better before this change).
Member
|
Contributor
Author
|
@luke-jr whoops. fixed. thanks for helping. |
jgarzik
pushed a commit
that referenced
this pull request
Aug 1, 2012
Unless debugging, show a more useful format for the askfors
Contributor
Author
|
thanks! |
suprnurd
pushed a commit
to chaincoin-legacy/chaincoin
that referenced
this pull request
Dec 5, 2017
lateminer
pushed a commit
to lateminer/bitcoin
that referenced
this pull request
May 6, 2020
64465b7 [Refactor] Consensus: Stake Modifier V2 (random-zebra) 8083725 [Refactor] Consensus: TimeProtocolV2 parameters (random-zebra) 4d1c696 [Refactor] Consensus: nStakeMinAge, nStakeMinDepth, nFutureTimeDrift (random-zebra) e69cee2 [Refactor] Consensus: restore nMaxMoneyOut as consensus param (random-zebra) 08a581d [Refactor] Consensus: fAllowMinDifficultyBlocks, fPowNoRetargeting (random-zebra) ca6ea3c [Refactor] Consensus: nTargetTimespan, nTargetSpacing, nTimeSlotLength (random-zebra) a281dee [Refactor] Consensus: remove unneeded getters. Rename BIP65Height (random-zebra) 284271c [Refactor] Consensus: bnProofOfWorkLimit / bnProofOfStakeLimit (v1-v2) (random-zebra) Pull request description: This PR builds on - [x] bitcoin#1328 Populates the `consensus` struct with all relevant consensus parameters. Resets `nMaxMoneyOut` as chain param (with different value for TestNet). Cleans up several unneeded fields in ChainParams. EDIT: Current PR has been split in two. Second part is bitcoin#1344 ACKs for top commit: furszy: good, utACK 64465b7. Fuzzbawls: utACK 64465b7 Tree-SHA512: d3e46bc8bb2ad9ee48abcbc4d4b1df2f59914360aaca72e93e19a5578248ecb95fd6a422477238e4736bc767d4e226f5b7f867174d96dcace73621946bf84df8
lateminer
pushed a commit
to lateminer/bitcoin
that referenced
this pull request
May 6, 2020
92221b1 [Cleanup] Remove ChainParams::DefaultMinerThreads (random-zebra) 98494ab [Refactor] Consensus: move other consensus params to the struct (random-zebra) 7c673bf [Refactor] Consensus: masternode/budget-related fields (random-zebra) 7736d6c [Refactor] Consensus: zerocoin-related fields (random-zebra) 0b63af4 [Refactor] ChainParams: move genesis creation at the beginning (random-zebra) 1e5a33d [Refactor] Consensus: wrapped serials params (nSupplyBeforeFakeSerial) (random-zebra) 70c4ae5 [Refactor] Consensus: height-based activations (random-zebra) Pull request description: Second part of - [x] bitcoin#1341 Split as individual PR for ease of review. ACKs for top commit: Fuzzbawls: ACK 92221b1 furszy: utACK 92221b1. Tree-SHA512: a3f1c10fc568e264039d2fc5a4dedc01d947b380e55c5170679598cb6859274c73157fb9ade6f5345a9781b9028943629b304aac8c3b0d7849946bdaed85fb52
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Will show a human readable time that the getdata will be triggered. Easier to follow. Might even be better to show this when debugging also...?