I\'m looking at some legacy code which has the following idiom:
Map myMap = someGlobalInstance.getMap(); synchronized (myMap) { item =
I think it would be better to use synchronized wrapper for your map