This is already done when database is open without caching or mmap. The current implementation
leads to different behaviour when different flags are used.

except: catches all exceptions, including exceptions like unbound variables etc.
Instead specify only exceptions which can be thrown if database is corrupted
(array overflow, decode() failure). read() and seek() shouldn't throw an exception
if the file is open and accessible so these two exceptions seem to be enough.

Detected by PyCharm inspections

The previous singleton pattern doesn't work when a class is inherited from the GeoIP class
and causes creation of a new GeoIP object every time GeoIP is accessed. In addition add
locks to protect from multiple instance creation from different threads.

This reverts commit 6ed94ce1.

Enable use in Google AppEngine