I locally changed all QList<SendCoinsEntry> usages with processPaymentRequest() to a single SendCoinsEntry, as we don't have multiple SendCoinsEntries with payment requests (as that is one of the things which this pull is changing), so yes I can be sure (as long as there are no subtle bugs left then).

Your nit should be fixed by the next push to this branch :).