hash_password prints hash in wrong order

[Alexey Torkhov]

hash_password() swaps adjacent hex digits in hash.
For example, md5("abc") =
900150983CD24FB0D6963F7D28E17F72

while hash_password("abc") =
09100589C32DF40B6D69F3D7821EF727

It happens because this code is done in wrong order:
        hash.push_back(hex_table[buf[i] & 0xf]);
        hash.push_back(hex_table[(buf[i] & 0xf0) >> 4]);

[Al Riddoch (alriddoch)]

Tragically fixing this would break compatibility with earlier versions, and has no obvious advantage as with the existing code the function serves its purpose as a password hash even if it does not transform the data as one might expect from the stated algorithm.

I am going to hold off fixing until something else breaks the schema in such a way as I don't care about breaking compatibility.