block.coinbase should not be payable

[thegostep]

[axic]

Good catch, the coinbase does not need to be actually payable, since the "coinbase transaction" is not executing any code.

[thegostep]

there are use cases where you would want to send eth to the miner

see: https://ethresear.ch/t/flashbots-frontrunning-the-mev-crisis/8251

[leonardoalt]

@thegostep given your last comment, are you suggesting it should or should not be payable?

[thegostep]

I think it should default to non-payable for consistency but still work with payable(block.coinbase).transfer()

[cameel]

I guess this should have been changed in 0.8.0 along with the types of the other addresses. Do we consider it a bug or just a missing feature?

[chriseth]

It doesn't actually hurt anyone, so I would just schedule for 0.9