Changeset 12096 in josm for trunk/src/org/openstreetmap/josm/data/osm

Timestamp:

2017-05-10T22:30:20+02:00 (7 years ago)

Author:

michael2402

Message:

Document fire mode parameter for selection event manager.

File:

• trunk/src/org/openstreetmap/josm/data/osm/event/SelectionEventManager.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/event/SelectionEventManager.java

@@ -83 +83 @@
      * Registers a new {@code SelectionChangedListener}.
      * @param listener listener to add
-     * @param fireMode EDT firing mode
+     * @param fireMode Set this to IN_EDT_CONSOLIDATED if you want the event to be fired in the EDT thread.
+     *                 Set it to IMMEDIATELY if youw ant the event to fire in the thread that caused the selection update.
      */
     public void addSelectionListener(SelectionChangedListener listener, FireMode fireMode) {
-        if (fireMode == FireMode.IN_EDT)
+        if (fireMode == FireMode.IN_EDT)  {
             throw new UnsupportedOperationException("IN_EDT mode not supported, you probably want to use IN_EDT_CONSOLIDATED.");
-        if (fireMode == FireMode.IN_EDT || fireMode == FireMode.IN_EDT_CONSOLIDATED) {
+        } else if (fireMode == FireMode.IN_EDT_CONSOLIDATED) {
             inEDTListeners.addIfAbsent(new ListenerInfo(listener));
         } else {