Check hash in viewer constructor.

[avandecreme]

Fix #229
Fix #176

[iangilman]

Awesome, good find!

[iangilman]

Any thoughts on how to make a test for this case?

[avandecreme]

I guess we can check that an error is thrown if neither id and hash is specified.

[iangilman]

Sounds reasonable. I was also wondering about the higher-level test of "images in collections resolve properly"... seems trickier.

Would you be up for writing a test for this?

[wynnme]

@avandecreme , @iangilman
I am using 0.9.131 but need a fix for #180 (multiple image takes a lot of memory). According to the comments, changelog #255 (Check hash in viewer constructor) is related, but I am skeptical that it alone fixes the memory leak. Do either of you (or anyone) know which specific changes would fix issue #180?
Thanks!

[avandecreme]

This commit fixes the zooming problem referenced in #180
Here is what I wrote about the memory:

Not sure about the memory problem but it might be fixed too (maybe the improper filling of the hash table was causing a leak).

[iangilman]

@wynnme I do not know enough about the memory leak to say for sure. I was taking @shipon-eye's report that it had been fixed:

#180 (comment)

Can you test with it and see if it's fixed for you as well?