Avoid using values() to iterate over QMap/QHash containers

It's more performant to iterate over the values directly than to first
stick them into a new container.

Reviewed-by: Thomas Hartmann