utACK f416446

Is there any specific test I can do to confirm the extension of PoW to the end of supply?
(I have run this, and it worked fine in the current era.)

The only thing I can think of would be to change this value for regtest to a much lower height and verify that PoW stops at that height, thus validating the behavior of the parameter.

Aside from my adding the extension to all chains, I'm not really seeing where this value is compared in the code for the lower number to have had any effect whatsoever. Maybe it was never going to happen?

Aside from my adding the extension to all chains, I'm not really seeing where this value is compared in the code for the lower number to have had any effect whatsoever. Maybe it was never going to happen?

Ah, found it in validation.cpp!

After chainparams.h passes the value of nLastPOWBlock to LAST_POW_BLOCK it then uses that below.

    //Do not accept PoW blocks after the last pow height
    if (pindex->nHeight > chainparams.LAST_POW_BLOCK() && !block.IsProofOfStake())
        return state.DoS(100, error("%s: Proof of Work block added after the last allowed PoW height", __func__), REJECT_INVALID, "pow-after-cutoff");