Changes between Initial Version and Version 1 of Ticket #15709, comment 53


Ignore:
Timestamp:
2018-01-02T05:20:09+01:00 (8 years ago)
Author:
cmuelle8

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #15709, comment 53

    initial v1  
    1818
    1919According to [https://www.javamex.com/tutorials/synchronization_volatile.shtml] volatiles should be self synchronizing after java 5, so r13264 should work, considering that getInstance() may be called from different threads, shouldn't it?
     20
     21EDIT: The LayerListDialog creates its instance in a separate static method as well, there must be a reason to do so.  Prior to r13264 ImageViewerDialog used that same creation pattern.  I'm rather curious, why r13264 truely makes a difference and exhibits unexpected behavior when combined with an enabled [https://trac.openstreetmap.org/browser/subversion/applications/editors/josm/plugins/photoadjust/src/org/openstreetmap/josm/plugins/photoadjust photoadjust plugin].