Avoid double calls to expensive/allocating functions.

There's more that can be done here:
- store the MD5 result in uint64_t[2] rather than a QByteArray, even
  without considering malloc overhead that is not needing more space
  and it avoids one allocation.
- run md5 on the raw data of the subject QString to avoid the UTF-8
  temporary.