@jonasschnelli wrote in #14979:
> #14573 broke < Qt5.6 compatibility due to calling the lambda version of `addAction` that was added in Qt5.6.

We currently [support](https://github.com/bitcoin/bitcoin/blob/master/doc/dependencies.md) QT >=5.2, but hardly anyone tests with that. If we make one of the Travis machines use it, we should have a better change of catching this.

See also discussion about minimum QT version in #13478 
{
    AssertLockHeld(cs_main);
    AssertLockHeld(cs_wallet);
    vCoins.clear();
    CAmount nTotal = 0;
    for (const auto& entry : mapWallet)
    {
        const uint256& wtxid = entry.first;
        const CWalletTx* pcoin = &entry.second;
        if (!CheckFinalTx(*pcoin->tx))
            continue;
        if (pcoin->IsImmatureCoinBase(locked_chain))
            continue;
        int nDepth = pcoin->GetDepthInMainChain(locked_chain);
        if (nDepth < 0)
            continue;
        
        
        if (nDepth == 0 && !pcoin->InMempool())
            continue;
        bool safeTx = pcoin->IsTrusted(locked_chain);