payout: drop a disjunct in the forfeit path that decides nothing

ForfeitPayoutRecipient retired an entry on
`it->amounts.empty() || PayoutFullyPaid(*it)`. PayoutFullyPaid iterates
amounts looking for a recipient with no txid, so it returns true for an
empty map: the first test can only be true where the second already is.

Left in, it reads as though the two cases differ, which invites someone
to make them differ. The condition is unchanged; only the claim about it
is.

Property: unchanged behaviour, and the code now says only what it does.

0 files changed, 0 insertions, 0 deletions