Index: trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java
===================================================================
--- trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java	(revision 9675)
+++ trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java	(revision 9676)
@@ -24,7 +24,18 @@
 import org.openstreetmap.josm.tools.Shortcut;
 
+/**
+ * Abstract superclass of save actions.
+ * @since 290
+ */
 public abstract class SaveActionBase extends DiskAccessAction {
     private File file;
 
+    /**
+     * Constructs a new {@code SaveActionBase}.
+     * @param name The action's text as displayed on the menu (if it is added to a menu)
+     * @param iconName The filename of the icon to use
+     * @param tooltip A longer description of the action that will be displayed in the tooltip
+     * @param shortcut A ready-created shortcut object or {@code null} if you don't want a shortcut
+     */
     public SaveActionBase(String name, String iconName, String tooltip, Shortcut shortcut) {
         super(name, iconName, tooltip, shortcut);
@@ -38,4 +49,8 @@
     }
 
+    /**
+     * Saves the active layer.
+     * @return {@code true} if the save operation succeeds
+     */
     public boolean doSave() {
         if (Main.isDisplayingMapView()) {
@@ -48,4 +63,9 @@
     }
 
+    /**
+     * Saves the given layer.
+     * @param layer layer to save
+     * @return {@code true} if the save operation succeeds
+     */
     public boolean doSave(Layer layer) {
         if (!layer.checkSaveConditions())
@@ -160,8 +180,10 @@
      *
      * @param fc FileChooser where file was already selected
+     * @param extension file extension
      * @return the {@code File} or {@code null} if the user cancelled the dialog.
      */
     public static File checkFileAndConfirmOverWrite(AbstractFileChooser fc, String extension) {
-        if (fc == null) return null;
+        if (fc == null)
+            return null;
         File file = fc.getSelectedFile();
 
@@ -190,6 +212,11 @@
     }
 
+    /**
+     * Asks user to confirm overwiting a file.
+     * @param file file to overwrite
+     * @return {@code true} if the file can be written
+     */
     public static boolean confirmOverwrite(File file) {
-        if (file == null || (file.exists())) {
+        if (file == null || file.exists()) {
             ExtendedDialog dialog = new ExtendedDialog(
                     Main.parent,
@@ -210,4 +237,5 @@
             filepath = file.getCanonicalPath();
         } catch (IOException ign) {
+            Main.warn(ign);
             return;
         }