payout: say what actually failed instead of blaming transaction size

SendMoney's "oversized transaction that requires a transaction fee of
0.00" is not an oversized transaction and there is no fee. That branch is
taken when nValue + nFeeRequired > GetBalance() -- an empty spendable
balance -- and the fee reads 0.00 because CreateTransaction failed before
setting one. 3,267 of these in debug.log sent me looking for a
transaction-size problem that does not exist. Report what is short, by how
much, and against what balance.

Name COINBASE_SPENDABLE_DEPTH in main.h and derive GetBlocksToMaturity
from it, because the distinction it captures is genuinely surprising:
CMerkleTx::GetCredit values a mined coinbase at zero until
COINBASE_MATURITY+20 deep, not COINBASE_MATURITY.

I first changed the payout gate to that deeper value, on the theory that
each payout was firing 20 blocks before its own coinbase could be spent.
Measuring it refuted that: across 934 payments, 96% succeeded at depth
100, the moment they were queued, which is only possible because a payout
is funded from whatever coin is already mature rather than from its own
block. The gate stays at COINBASE_MATURITY, with a comment recording the
measurement so the next person does not re-make the same change.

The real shortfall is structural and not addressed here: the pool pays out
~99% of each block's subsidy while only matured coinbases are spendable,
so it runs with almost no buffer, and any orphaned block it queued a
payout for drains that buffer permanently. The 24h retry loop absorbs this
-- 0 payouts forfeited -- but the buffer is what needs fixing.

(cherry picked from commit e28df01f90d621b0cd818f28d70a20b8ab5d1a8d)

1 files changed, 6 insertions, 0 deletions

diff --git a/src/main.h b/src/main.h
index 22be3dc..6b1ff0a 100644
--- a/src/main.h
+++ b/src/main.h
@@ -41,6 +41,12 @@ static const int64 MAX_MONEY = 21000000 * COIN;
 inline bool MoneyRange(int64 nValue) { return (nValue >= 0 && nValue <= MAX_MONEY); }
 static const int64 CENT = 1000000;
 static const int COINBASE_MATURITY = 100;
+// Depth at which a coinbase we mined is actually usable. CMerkleTx::GetCredit
+// values a coinbase at zero until GetBlocksToMaturity() reaches 0, which is
+// this depth -- not COINBASE_MATURITY. Anything that plans to *spend* mined
+// coin must gate on this, or it will try to pay out of a balance the wallet
+// still reports as zero. Both rules live here so they cannot drift apart.
+static const int COINBASE_SPENDABLE_DEPTH = COINBASE_MATURITY + 20;
 
 // Bitflash difficulty floor. The PoW is RandomX (memory-hard), ~1000x slower
 // per hash than SHA-256d, so the target is much easier than Bitcoin's to let a