Comment 2 for bug 381356

A couple of comments, but +1:

[1]
+ if hash:
+ return hash

"If hash is not None" looks like a better condition

[2]
+ the desidered mapping is not found.

Typo: desired?

[3] It would be nice to have a test when multiple HashIdStore are involved.

Thanks!