add nModSize init to default constructor of CTxMemPoolEntry #4930

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:txmempool changing 1 files +1 −1

Diapolo commented at 8:34 AM on September 16, 2014: none

No description provided.
laanwj commented at 8:42 AM on September 16, 2014: member

ACK on the added initializer, I don't see the point of the whitespace changes however.
jgarzik commented at 12:45 PM on September 16, 2014: contributor

ACK added initializer, NAK inconsistent style changes
Diapolo commented at 1:01 PM on September 16, 2014: none

Wouldn't it look that way if we format our source with that clang tool anyway? That was the only intention I had in mind.
laanwj commented at 1:02 PM on September 16, 2014: member

@Diapolo let's just not do whitespace changes
add nModSize init to default constructor of CTxMemPoolEntry f4fe205034
Diapolo commented at 1:08 PM on September 16, 2014: none

@laanwj Alright, reduced to just the minor init change.
laanwj merged this on Sep 16, 2014
laanwj closed this on Sep 16, 2014
laanwj referenced this in commit cf04923618 on Sep 16, 2014
BitcoinPullTester commented at 1:22 PM on September 16, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4930_f4fe20503433032e08cfa2133ce799a4c3d27d61/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj commented at 1:22 PM on September 16, 2014: member

@diapolo Thanks!
Diapolo deleted the branch on Sep 16, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors