[PATCH] modularize BitcoinMiner a bit (code movement)

[jgarzik]

URL: http://yyz.us/bitcoin/patch.bitcoin-modularize_miner

This patch is similar to patches sometimes seen in the Linux kernel.  Analogous to an algebraic reduction, or a step in a mathematic proof, this patch intentionally makes zero functional code changes.  It moves two chunks of code -- hash meter and check-solution -- from the BitcoinMiner() function into separate functions.

This change makes it easier to integrate remote mining.

[Cdecker]

Seems to be ok: +1

[jgarzik]

Patch updated to move prepare-work and prepare-hash-buffer code into separate functions.

Again, no code changes, just code movement.

The patch now looks similar to m0mchil's getwork patch, and makes integrating remote mining much easier.

[jgarzik]

Patch updated to fix silly parameter passing bug in one of the new functions.  Code now generates correct blocks on testnet, as expected.

[jgarzik]

Updated to SVN r188 (accounts).

[jgarzik]

This patch is entirely superceded by SVN r189, satoshi's version of getwork.