Some minor cleanup and javadoc

1 files changed, 16 insertions, 6 deletions

diff --git a/bundles/org.eclipse.team.ui/src/org/eclipse/team/internal/ui/dialogs/MappingSelectionDialog.java b/bundles/org.eclipse.team.ui/src/org/eclipse/team/internal/ui/dialogs/MappingSelectionDialog.java
index 02c0a2f30..c68c710eb 100644
--- a/bundles/org.eclipse.team.ui/src/org/eclipse/team/internal/ui/dialogs/MappingSelectionDialog.java
+++ b/bundles/org.eclipse.team.ui/src/org/eclipse/team/internal/ui/dialogs/MappingSelectionDialog.java
@@ -30,7 +30,7 @@ public abstract class MappingSelectionDialog extends DetailsDialog implements IP
     private ResourceMappingResourceDisplayArea resourceArea;
     private final IResourceMappingResourceFilter filter;
 
-    public MappingSelectionDialog(Shell parentShell, String dialogTitle, ResourceMapping[] mappings, IResourceMappingResourceFilter filter) {
+    protected MappingSelectionDialog(Shell parentShell, String dialogTitle, ResourceMapping[] mappings, IResourceMappingResourceFilter filter) {
         super(parentShell, dialogTitle);
         this.mappings = mappings;
         this.filter = filter;
@@ -91,10 +91,6 @@ public abstract class MappingSelectionDialog extends DetailsDialog implements IP
         // Can always finish
         setPageComplete(true);
     }
-
-    protected ResourceMapping[] getCheckedMappings() {
-        return checkedMappings;
-    }
     
     /* (non-Javadoc)
      * @see org.eclipse.team.internal.ui.dialogs.DetailsDialog#includeErrorMessage()
@@ -172,8 +168,22 @@ public abstract class MappingSelectionDialog extends DetailsDialog implements IP
      */
     protected abstract String getResourceListMessage(ResourceMapping mapping);
 
-    public ResourceMapping[] getMappings() {
+    /**
+     * Return the <code>ResourceMappings</code> that are being displayed 
+     * by the dialog.
+     * @return the <code>ResourceMappings</code> that are being displayed 
+     * by the dialog
+     */
+    public final ResourceMapping[] getMappings() {
         return mappings;
     }
     
+    /**
+     * Return the <code>ResourceMappings</code> that were checked
+     * by the user.
+     * @return the <code>ResourceMappings</code> that were checked
+     */
+    protected final ResourceMapping[] getCheckedMappings() {
+        return checkedMappings;
+    }
 }