Changeset 6413 in josm

Timestamp:

2013-11-25T02:19:43+01:00 (10 years ago)

Author:

Don-vip

Message:

undo/redo: param check + javadoc

File:

• trunk/src/org/openstreetmap/josm/data/UndoRedoHandler.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/data/UndoRedoHandler.java

@@ -12 +12 @@
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer.CommandQueueListener;
+import org.openstreetmap.josm.tools.CheckParameterUtil;
 
 public class UndoRedoHandler implements MapView.LayerChangeListener {
@@ -26 +27 @@
     private final LinkedList<CommandQueueListener> listenerCommands = new LinkedList<CommandQueueListener>();
 
+    /**
+     * Constructs a new {@code UndoRedoHandler}.
+     */
     public UndoRedoHandler() {
         MapView.addLayerChangeListener(this);
@@ -31 +35 @@
 
     /**
-     * Execute the command and add it to the intern command queue.
+     * Executes the command and add it to the intern command queue.
+     * @param c The command to execute. Must not be {@code null}.
      */
     public void addNoRedraw(final Command c) {
+        CheckParameterUtil.ensureParameterNotNull(c, "c");
         c.executeCommand();
         commands.add(c);
@@ -53 +59 @@
 
     /**
-     * Execute the command and add it to the intern command queue.
+     * Executes the command and add it to the intern command queue.
+     * @param c The command to execute. Must not be {@code null}.
      */
     synchronized public void add(final Command c) {
@@ -69 +76 @@
     /**
      * Undoes multiple commands.
+     * @param num The number of commands to undo
      */
     synchronized public void undo(int num) {
@@ -104 +112 @@
     /**
      * Redoes multiple commands.
+     * @param num The number of commands to redo
      */
     public void redo(int num) {
@@ -167 +176 @@
     public void activeLayerChange(Layer oldLayer, Layer newLayer) {}
 
+    /**
+     * Removes a command queue listener.
+     * @param l The command queue listener to remove
+     */
     public void removeCommandQueueListener(CommandQueueListener l) {
         listenerCommands.remove(l);
     }
 
+    /**
+     * Adds a command queue listener.
+     * @param l The commands queue listener to add
+     * @return {@code true} if the listener has been added, {@code false} otherwise
+     */
     public boolean addCommandQueueListener(CommandQueueListener l) {
         return listenerCommands.add(l);