This number was still tracked even though it's shown nowhere in the UI anymore. It was originally removed because it didn't match the actual number of records in the view (which contains outputs, not transactions) thus was confusing people.
gui: remove redundant numTransactions tracking #4718
pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2014_08_gui_remove_numtransactions changing 2 files +0 −25-
laanwj commented at 8:28 AM on August 18, 2014: member
-
88fe88cf36
gui: remove redundant numTransactions tracking
This number was still tracked even though it's shown nowhere in the UI anymore. It was originally removed because it didn't match the actual number of records in the view (which contains outputs, not transactions) thus was confusing people.
- laanwj added the label GUI on Aug 18, 2014
-
BitcoinPullTester commented at 3:23 PM on August 18, 2014: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4718_88fe88cf3640caede5222380383d0816a42a73b7/ 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 merged this on Aug 23, 2014
- laanwj closed this on Aug 23, 2014
- laanwj referenced this in commit 92b3d3630d on Aug 23, 2014
- wtogami referenced this in commit f50ccc1d06 on Sep 9, 2014
- wtogami referenced this in commit dbddbd7a59 on Sep 19, 2014
- wtogami referenced this in commit 9452027cf3 on Oct 2, 2014
- wtogami referenced this in commit 98229bd618 on Nov 14, 2014
- wtogami referenced this in commit 2d3d8e9f58 on Dec 23, 2014
- MarcoFalke locked this on Sep 8, 2021
Contributors
Labels