Remove unused variable #4467

pull rebroad wants to merge 1 commits into bitcoin:master from rebroad:nFeeUnused changing 1 files +0 −3

rebroad commented at 3:44 PM on July 5, 2014: contributor

nFee not used (introduced in #4314)
Remove unused variable 1f740ddc4d
BitcoinPullTester commented at 4:02 PM on July 5, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4467_1f740ddc4d8d9312dc62bca3bf0b6632b08ee6a9/ 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.
cozz commented at 4:07 PM on July 5, 2014: contributor

Would also be fixed with #4465
laanwj cross-referenced this on Jul 7, 2014 from issue Remove unused variable nFee from coincontroldialog by fanquake
laanwj commented at 7:36 AM on July 7, 2014: member

ACK, although @cozz's pull may be preferable, I need to have a better look at it.
jtimon commented at 7:50 AM on July 7, 2014: contributor

Ack, this removes a warning while compiling.
laanwj merged this on Jul 7, 2014
laanwj closed this on Jul 7, 2014
laanwj referenced this in commit 4ea8f632fe on Jul 7, 2014
laanwj commented at 8:15 AM on July 7, 2014: member

@jtimon you're right, this fixes an annoying warning, so I'd like to merge it. #4465 can be rebased on top of it.
bitcoin locked this on Sep 8, 2021

Contributors