I saw in several places in the source, that there are functions variables and all that stuff, which were used for mining coins with the client. As this has been disabled a few releases ago, I suggest to remove it to cleanup and shrink the source. This would take it's time, but as it's unneeded and untouched, it could potentially become "harmful" in the further development to leave it in.
-
Diapolo commented at 12:06 PM on March 30, 2012: none
-
sipa commented at 12:11 PM on March 30, 2012: member
It's quite useful still for testnet, It's not entirely disabled by the way, just inaccessible from the GUI.
Maybe releases could be compiled without it, though.
-
Diapolo commented at 12:16 PM on March 30, 2012: none
It's only a suggestion, if we could seperate the code and compile without it, that would be nice, but I guess very low priority :). Leave it on this list as a reminder, if other devs say NACK simply close this one.
-
laanwj commented at 12:56 PM on March 30, 2012: member
There was a discussion about this some time ago and the outcome was to leave it in as example. It can still be enabled through RPC or the command line.
-
jgarzik commented at 1:22 PM on March 30, 2012: contributor
NAK - it is still useful on test networks and other dev scenarios, including as working reference code
-
Diapolo commented at 10:40 PM on March 30, 2012: none
Closed, as the answers were pretty clear :).
- Diapolo closed this on Mar 30, 2012
- suprnurd referenced this in commit c81f0a3202 on Dec 5, 2017
- lateminer referenced this in commit 75b8ad2ae5 on Oct 30, 2019
- MarcoFalke locked this on Sep 8, 2021
