Index: /trunk/src/org/openstreetmap/josm/actions/AbstractInfoAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractInfoAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractInfoAction.java	(revision 16553)
@@ -30,5 +30,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public AbstractInfoAction(boolean installAdapters) {
+    protected AbstractInfoAction(boolean installAdapters) {
         super(installAdapters);
     }
@@ -48,5 +48,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public AbstractInfoAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register,
+    protected AbstractInfoAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register,
             String toolbarId, boolean installAdapters) {
         super(name, iconName, tooltip, shortcut, register, toolbarId, installAdapters);
Index: /trunk/src/org/openstreetmap/josm/actions/AbstractMergeAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractMergeAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractMergeAction.java	(revision 16553)
@@ -97,5 +97,5 @@
      * @param register register this action for the toolbar preferences?
      */
-    public AbstractMergeAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register) {
+    protected AbstractMergeAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register) {
         super(name, iconName, tooltip, shortcut, register);
     }
@@ -115,5 +115,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public AbstractMergeAction(String name, String iconName, String tooltip, Shortcut shortcut,
+    protected AbstractMergeAction(String name, String iconName, String tooltip, Shortcut shortcut,
     boolean register, String toolbar, boolean installAdapters) {
         super(name, iconName, tooltip, shortcut, register, toolbar, installAdapters);
Index: /trunk/src/org/openstreetmap/josm/actions/AbstractPasteAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractPasteAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractPasteAction.java	(revision 16553)
@@ -39,5 +39,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public AbstractPasteAction(String name, String iconName, String tooltip, Shortcut shortcut,
+    protected AbstractPasteAction(String name, String iconName, String tooltip, Shortcut shortcut,
             boolean registerInToolbar) {
         this(name, iconName, tooltip, shortcut, registerInToolbar, null);
@@ -57,5 +57,5 @@
      * @param toolbarId identifier for the toolbar preferences. The iconName is used, if this parameter is null
      */
-    public AbstractPasteAction(String name, String iconName, String tooltip, Shortcut shortcut,
+    protected AbstractPasteAction(String name, String iconName, String tooltip, Shortcut shortcut,
             boolean registerInToolbar, String toolbarId) {
         super(name, iconName, tooltip, shortcut, registerInToolbar, toolbarId, true);
Index: /trunk/src/org/openstreetmap/josm/actions/AbstractSelectAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractSelectAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractSelectAction.java	(revision 16553)
@@ -17,5 +17,5 @@
      * Constructs a new {@code AbstractSelectAction}.
      */
-    public AbstractSelectAction() {
+    protected AbstractSelectAction() {
         putValue(NAME, tr("Select"));
         putValue(SHORT_DESCRIPTION, tr("Selects those elements on the map which are chosen on the list above."));
Index: /trunk/src/org/openstreetmap/josm/actions/AbstractShowHistoryAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractShowHistoryAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractShowHistoryAction.java	(revision 16553)
@@ -13,8 +13,9 @@
  */
 public abstract class AbstractShowHistoryAction extends AbstractAction {
+
     /**
      * Constructs a new {@code AbstractShowHistoryAction}.
      */
-    public AbstractShowHistoryAction() {
+    protected AbstractShowHistoryAction() {
         putValue(NAME, tr("History"));
         putValue(SHORT_DESCRIPTION, tr("Download and show the history of the selected objects"));
Index: /trunk/src/org/openstreetmap/josm/actions/AbstractUploadAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AbstractUploadAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/AbstractUploadAction.java	(revision 16553)
@@ -29,5 +29,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public AbstractUploadAction(String name, String iconName, String tooltip, Shortcut shortcut,
+    protected AbstractUploadAction(String name, String iconName, String tooltip, Shortcut shortcut,
             boolean registerInToolbar) {
         super(name, iconName, tooltip, shortcut, registerInToolbar);
Index: /trunk/src/org/openstreetmap/josm/actions/ActionParameter.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/ActionParameter.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/ActionParameter.java	(revision 16553)
@@ -17,5 +17,5 @@
      * @param name parameter name (the key)
      */
-    public ActionParameter(String name) {
+    protected ActionParameter(String name) {
         this.name = name;
     }
Index: /trunk/src/org/openstreetmap/josm/actions/DiskAccessAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/DiskAccessAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/DiskAccessAction.java	(revision 16553)
@@ -26,5 +26,5 @@
      * @since 1084
      */
-    public DiskAccessAction(String name, String iconName, String tooltip, Shortcut shortcut) {
+    protected DiskAccessAction(String name, String iconName, String tooltip, Shortcut shortcut) {
         super(name, iconName, tooltip, shortcut, true);
     }
@@ -42,5 +42,5 @@
      * @since 5438
      */
-    public DiskAccessAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register,
+    protected DiskAccessAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean register,
             String toolbarId, boolean installAdapters) {
         super(name, iconName, tooltip, shortcut, register, toolbarId, installAdapters);
Index: /trunk/src/org/openstreetmap/josm/actions/DownloadAlongAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/DownloadAlongAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/DownloadAlongAction.java	(revision 16553)
@@ -60,5 +60,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public DownloadAlongAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
+    protected DownloadAlongAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
         super(name, iconName, tooltip, shortcut, registerInToolbar);
     }
Index: /trunk/src/org/openstreetmap/josm/actions/JosmAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/JosmAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/JosmAction.java	(revision 16553)
@@ -79,5 +79,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public JosmAction(String name, ImageProvider icon, String tooltip, Shortcut shortcut, boolean registerInToolbar,
+    protected JosmAction(String name, ImageProvider icon, String tooltip, Shortcut shortcut, boolean registerInToolbar,
             String toolbarId, boolean installAdapters) {
         super(name);
@@ -127,5 +127,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar,
+    protected JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar,
             String toolbarId, boolean installAdapters) {
         this(name, iconName == null ? null : new ImageProvider(iconName).setOptional(true), tooltip, shortcut, registerInToolbar,
@@ -149,5 +149,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar, boolean installAdapters) {
+    protected JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar, boolean installAdapters) {
         this(name, iconName, tooltip, shortcut, registerInToolbar, null, installAdapters);
     }
@@ -168,5 +168,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
+    protected JosmAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
         this(name, iconName, tooltip, shortcut, registerInToolbar, null, true);
     }
@@ -175,5 +175,5 @@
      * Constructs a new {@code JosmAction}.
      */
-    public JosmAction() {
+    protected JosmAction() {
         this(true);
     }
@@ -184,5 +184,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public JosmAction(boolean installAdapters) {
+    protected JosmAction(boolean installAdapters) {
         setHelpId();
         if (installAdapters) {
@@ -203,5 +203,5 @@
      * @since 14012
      */
-    public JosmAction(String name, String iconName, String tooltip, List<Shortcut> shortcuts) {
+    protected JosmAction(String name, String iconName, String tooltip, List<Shortcut> shortcuts) {
         this(name, iconName, tooltip, shortcuts.get(0), true, null, true);
         for (int i = 1; i < shortcuts.size(); i++) {
Index: /trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java	(revision 16553)
@@ -44,5 +44,5 @@
      * @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) {
+    protected SaveActionBase(String name, String iconName, String tooltip, Shortcut shortcut) {
         super(name, iconName, tooltip, shortcut);
     }
@@ -57,5 +57,5 @@
      * @since 15496
      */
-    public SaveActionBase(String name, String iconName, String tooltip, Shortcut shortcut, boolean quiet) {
+    protected SaveActionBase(String name, String iconName, String tooltip, Shortcut shortcut, boolean quiet) {
         super(name, iconName, tooltip, shortcut);
         this.quiet = quiet;
Index: /trunk/src/org/openstreetmap/josm/actions/ToggleAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/ToggleAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/ToggleAction.java	(revision 16553)
@@ -40,5 +40,5 @@
      * @param installAdapters false, if you don't want to install layer changed and selection changed adapters
      */
-    public ToggleAction(String name, ImageProvider icon, String tooltip, Shortcut shortcut, boolean registerInToolbar,
+    protected ToggleAction(String name, ImageProvider icon, String tooltip, Shortcut shortcut, boolean registerInToolbar,
             String toolbarId, boolean installAdapters) {
         super(name, icon, tooltip, shortcut, registerInToolbar, toolbarId, installAdapters);
@@ -60,5 +60,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public ToggleAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
+    protected ToggleAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
         super(name, iconName, tooltip, shortcut, registerInToolbar);
         // It is required to set the SELECTED_KEY to a non-null value in order to let Swing components update it
Index: /trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java	(revision 16553)
@@ -451,6 +451,5 @@
     /**
      * dupe all nodes that are selected, and put the copies on the selected way
-     * @throws UserCancelException
-     *
+     * @throws UserCancelException if user cancels choice
      */
     private void unglueOneWayAnyNodes() throws UserCancelException {
Index: /trunk/src/org/openstreetmap/josm/actions/audio/AbstractAudioAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/audio/AbstractAudioAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/audio/AbstractAudioAction.java	(revision 16553)
@@ -22,5 +22,5 @@
      * @param registerInToolbar register this action for the toolbar preferences?
      */
-    public AbstractAudioAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
+    protected AbstractAudioAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean registerInToolbar) {
         super(name, iconName, tooltip, shortcut, registerInToolbar);
         updateEnabledState();
Index: /trunk/src/org/openstreetmap/josm/actions/audio/AudioFastSlowAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/audio/AudioFastSlowAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/audio/AudioFastSlowAction.java	(revision 16553)
@@ -27,5 +27,5 @@
      * @param fast {@code true} to increase speed (faster audio), {@code false} to decrease it (slower audio).
      */
-    public AudioFastSlowAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean fast) {
+    protected AudioFastSlowAction(String name, String iconName, String tooltip, Shortcut shortcut, boolean fast) {
         super(name, iconName, tooltip, shortcut, true);
         multiplier = Config.getPref().getDouble("audio.fastfwdmultiplier", 1.3);
Index: /trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java	(revision 16553)
@@ -25,5 +25,5 @@
      * Constructs a new {@code AbstractDownloadTask}.
      */
-    public AbstractDownloadTask() {
+    protected AbstractDownloadTask() {
         errorMessages = new ArrayList<>();
     }
Index: /trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java	(revision 16553)
@@ -233,5 +233,5 @@
          * @param zoomAfterDownload If true, the map view will zoom to download area after download
          */
-        public AbstractInternalTask(DownloadParams settings, String title, boolean ignoreException, boolean zoomAfterDownload) {
+        protected AbstractInternalTask(DownloadParams settings, String title, boolean ignoreException, boolean zoomAfterDownload) {
             super(title, ignoreException);
             this.settings = Objects.requireNonNull(settings);
@@ -249,5 +249,5 @@
          * @param zoomAfterDownload If true, the map view will zoom to download area after download
          */
-        public AbstractInternalTask(DownloadParams settings, String title, ProgressMonitor progressMonitor, boolean ignoreException,
+        protected AbstractInternalTask(DownloadParams settings, String title, ProgressMonitor progressMonitor, boolean ignoreException,
                 boolean zoomAfterDownload) {
             super(title, progressMonitor, ignoreException);
Index: /trunk/src/org/openstreetmap/josm/actions/mapmode/MapMode.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/mapmode/MapMode.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/actions/mapmode/MapMode.java	(revision 16553)
@@ -44,5 +44,5 @@
      * @since 11713
      */
-    public MapMode(String name, String iconName, String tooltip, Shortcut shortcut, Cursor cursor) {
+    protected MapMode(String name, String iconName, String tooltip, Shortcut shortcut, Cursor cursor) {
         super(name, "mapmode/"+iconName, tooltip, shortcut, false);
         this.cursor = cursor;
@@ -58,5 +58,5 @@
      * @since 11713
      */
-    public MapMode(String name, String iconName, String tooltip, Cursor cursor) {
+    protected MapMode(String name, String iconName, String tooltip, Cursor cursor) {
         putValue(NAME, name);
         new ImageProvider("mapmode", iconName).getResource().attachImageIcon(this);
Index: /trunk/src/org/openstreetmap/josm/command/Command.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/command/Command.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/command/Command.java	(revision 16553)
@@ -135,5 +135,5 @@
      * @since 11240
      */
-    public Command(DataSet data) {
+    protected Command(DataSet data) {
         CheckParameterUtil.ensureParameterNotNull(data, "data");
         this.data = data;
Index: /trunk/src/org/openstreetmap/josm/command/TransformNodesCommand.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/command/TransformNodesCommand.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/command/TransformNodesCommand.java	(revision 16553)
@@ -51,5 +51,5 @@
      * @throws NoSuchElementException if objects is empty
      */
-    public TransformNodesCommand(Collection<? extends OsmPrimitive> objects) {
+    protected TransformNodesCommand(Collection<? extends OsmPrimitive> objects) {
         super(objects.iterator().next().getDataSet());
         this.nodes = AllNodesVisitor.getAllNodes(objects);
Index: /trunk/src/org/openstreetmap/josm/command/conflict/ConflictResolveCommand.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/command/conflict/ConflictResolveCommand.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/command/conflict/ConflictResolveCommand.java	(revision 16553)
@@ -29,5 +29,5 @@
      * @param ds the data set. Must not be null.
      */
-    public ConflictResolveCommand(DataSet ds) {
+    protected ConflictResolveCommand(DataSet ds) {
         super(ds);
     }
Index: /trunk/src/org/openstreetmap/josm/data/Bounds.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/Bounds.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/Bounds.java	(revision 16553)
@@ -361,4 +361,5 @@
 
     /**
+     * Returns center of the bounding box.
      * @return Center of the bounding box.
      */
Index: /trunk/src/org/openstreetmap/josm/data/cache/CacheEntry.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/cache/CacheEntry.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/cache/CacheEntry.java	(revision 16553)
@@ -22,4 +22,5 @@
 
     /**
+     * Returns cache entry content.
      * @return cache entry content
      */
Index: /trunk/src/org/openstreetmap/josm/data/cache/CacheEntryAttributes.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/cache/CacheEntryAttributes.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/cache/CacheEntryAttributes.java	(revision 16553)
@@ -54,4 +54,5 @@
 
     /**
+     * Determines if the entry is marked as "no tile at this zoom level".
      * @return if the entry is marked as "no tile at this zoom level"
      */
@@ -69,4 +70,5 @@
 
     /**
+     * Returns ETag header value, that was returned for this entry.
      * @return ETag header value, that was returned for this entry.
      */
@@ -101,4 +103,5 @@
 
     /**
+     * Returns last modification of the object in cache in milliseconds from Epoch.
      * @return last modification of the object in cache in milliseconds from Epoch
      */
@@ -117,4 +120,5 @@
 
     /**
+     * Returns when the object expires in milliseconds from Epoch.
      * @return when the object expires in milliseconds from Epoch
      */
@@ -143,4 +147,5 @@
 
     /**
+     * Returns HTTP response code.
      * @return http status code
      * @since 8389
@@ -178,4 +183,5 @@
 
     /**
+     * Returns error message returned while retrieving this object.
      * @return error message returned while retrieving this object
      */
@@ -185,4 +191,5 @@
 
     /**
+     * Sets error related to this object.
      * @param error error related to this object
      * @since 10469
@@ -193,4 +200,5 @@
 
     /**
+     * Sets error message related to this object.
      * @param message error message related to this object
      */
@@ -200,6 +208,6 @@
 
     /**
+     * Sets exception that caused error.
      * @param e exception that caused error
-     *
      */
     public void setException(Exception e) {
@@ -208,6 +216,6 @@
 
     /**
+     * Returns optional exception that was thrown when fetching resource.
      * @return Optional exception that was thrown when fetching resource
-     *
      */
     public Optional<Class<? extends Exception>> getException() {
Index: /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java	(revision 16553)
@@ -100,5 +100,5 @@
      * @param downloadJobExecutor that will be executing the jobs
      */
-    public JCSCachedTileLoaderJob(ICacheAccess<K, V> cache,
+    protected JCSCachedTileLoaderJob(ICacheAccess<K, V> cache,
             TileJobOptions options,
             ThreadPoolExecutor downloadJobExecutor) {
@@ -117,5 +117,5 @@
      * @param options of the request
      */
-    public JCSCachedTileLoaderJob(ICacheAccess<K, V> cache,
+    protected JCSCachedTileLoaderJob(ICacheAccess<K, V> cache,
             TileJobOptions options) {
         this(cache, options, DEFAULT_DOWNLOAD_JOB_DISPATCHER);
@@ -174,6 +174,6 @@
 
     /**
-     *
-     * @return checks if object from cache has sufficient data to be returned
+     * Checks if object from cache has sufficient data to be returned.
+     * @return {@code true} if object from cache has sufficient data to be returned
      */
     protected boolean isObjectLoadable() {
@@ -195,4 +195,5 @@
 
     /**
+     * Returns key under which discovered server settings will be kept.
      * @return key under which discovered server settings will be kept
      */
Index: /trunk/src/org/openstreetmap/josm/data/conflict/Conflict.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/conflict/Conflict.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/conflict/Conflict.java	(revision 16553)
@@ -98,5 +98,5 @@
 
     /**
-     *
+     * Determines if my primitive was deleted but it has set non deleted status.
      * @return True if my primitive was deleted but it has set non deleted status because it's referred by another
      * primitive and references to deleted primitives are not allowed.
Index: /trunk/src/org/openstreetmap/josm/data/coor/LatLon.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/coor/LatLon.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/coor/LatLon.java	(revision 16553)
@@ -182,4 +182,5 @@
 
     /**
+     * Determines if the other point has almost the same lat/lon values.
      * @param other other lat/lon
      * @return <code>true</code> if the other point has almost the same lat/lon
Index: /trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java	(revision 16553)
@@ -1166,4 +1166,5 @@
 
     /**
+     * Determines whether anything has been modified.
      * @return whether anything has been modified (e.g. colors)
      * @since 15496
@@ -1214,5 +1215,6 @@
 
         /**
-         * @return the URI of the namesapce
+         * Returns the URI of the namespace.
+         * @return the URI of the namespace
          */
         public String getURI() {
@@ -1221,4 +1223,5 @@
 
         /**
+         * Returns the prefix of the namespace.
          * @return the prefix of the namespace, determined from URI if possible
          */
@@ -1228,4 +1231,5 @@
 
         /**
+         * Returns the location of the schema.
          * @return the location of the schema
          */
Index: /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtension.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtension.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtension.java	(revision 16553)
@@ -94,4 +94,5 @@
 
     /**
+     * Returns the qualified name of the XML element.
      * @return the qualified name of the XML element
      */
@@ -101,4 +102,5 @@
 
     /**
+     * Returns the prefix of the XML namespace.
      * @return the prefix of the XML namespace
      */
@@ -108,4 +110,5 @@
 
     /**
+     * Returns the key (local element name) of the extension.
      * @return the key (local element name) of the extension
      */
@@ -115,9 +118,10 @@
 
     /**
+     * Returns the flattened extension key of this extension.
      * @return the flattened extension key of this extension, used for conversion to OSM layers
      */
     public String getFlatKey() {
         String ret = "";
-        if (parent != null && parent instanceof GpxExtension) {
+        if (parent instanceof GpxExtension) {
             GpxExtension ext = (GpxExtension) parent;
             ret = ext.getFlatKey() + ":";
@@ -144,4 +148,5 @@
 
     /**
+     * Returns the value of the extension.
      * @return the value of the extension
      */
@@ -151,4 +156,5 @@
 
     /**
+     * Sets the value.
      * @param value the value to set
      */
@@ -204,4 +210,5 @@
 
     /**
+     * Determines if this extension should be written.
      * @return if this extension should be written, used for hiding colors during export without removing them
      */
@@ -211,4 +218,5 @@
 
     /**
+     * Returns the parent element of this extension.
      * @return the parent element of this extension, can be another extension or gpx elements (data, track, segment, ...)
      */
Index: /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtensionCollection.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtensionCollection.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/gpx/GpxExtensionCollection.java	(revision 16553)
@@ -250,4 +250,5 @@
 
     /**
+     * Determines if this collection contains writable extensions.
      * @return <code>true</code> if this collection contains writable extensions
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/AbstractWMSTileSource.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/AbstractWMSTileSource.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/AbstractWMSTileSource.java	(revision 16553)
@@ -44,5 +44,5 @@
      * @param tileProjection the tile projection
      */
-    public AbstractWMSTileSource(TileSourceInfo info, Projection tileProjection) {
+    protected AbstractWMSTileSource(TileSourceInfo info, Projection tileProjection) {
         super(info);
         this.tileProjection = tileProjection;
Index: /trunk/src/org/openstreetmap/josm/data/imagery/DefaultLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/DefaultLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/DefaultLayer.java	(revision 16553)
@@ -41,4 +41,5 @@
 
     /**
+     * Returns layer name of the default layer.
      * @return layer name of the default layer
      */
@@ -48,4 +49,5 @@
 
     /**
+     * Returns default tileMatrixSet. Only usable for WMTS
      * @return default tileMatrixSet. Only usable for WMTS
      */
@@ -55,4 +57,5 @@
 
     /**
+     * Returns style for this WMS / WMTS layer to use.
      * @return style for this WMS / WMTS layer to use
      */
@@ -62,4 +65,5 @@
 
     /**
+     * Returns JSON representation of the default layer object.
      * @return JSON representation of the default layer object
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/LayerDetails.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/LayerDetails.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/LayerDetails.java	(revision 16553)
@@ -45,4 +45,5 @@
 
     /**
+     * Returns projections that are supported by this layer.
      * @return projections that are supported by this layer
      */
@@ -57,5 +58,5 @@
 
     /**
-     *
+     * Returns styles defined for this layer.
      * @return styles defined for this layer
      */
@@ -70,5 +71,6 @@
 
     /**
-     * @return title "Human readable" title of this layer
+     * Returns "Human readable" title of this layer
+     * @return "Human readable" title of this layer
      * @see LayerDetails#getName()
      */
@@ -78,5 +80,6 @@
 
     /**
-     * @param title set title of this layer
+     * Sets title of this layer
+     * @param title title of this layer
      * @see LayerDetails#getName()
      */
@@ -104,5 +107,6 @@
 
     /**
-     * @param name sets the name of this Layer
+     * Sets the name of this Layer.
+     * @param name the name of this Layer
      * @see LayerDetails#getName()
      */
@@ -129,5 +133,5 @@
 
     /**
-     *
+     * Returns bounds within layer might be queried.
      * @return bounds within layer might be queried
      */
@@ -137,5 +141,5 @@
 
     /**
-     * sets bounds of this layer
+     * Sets bounds of this layer
      * @param bounds of this layer
      */
@@ -151,6 +155,6 @@
 
     /**
-     *
-     * @return parent layer for his layer
+     * Returns parent layer for this layer.
+     * @return parent layer for this layer
      */
     public LayerDetails getParent() {
@@ -168,5 +172,5 @@
 
     /**
-     *
+     * Returns children layers of this layer.
      * @return children layers of this layer
      */
@@ -184,4 +188,5 @@
 
     /**
+     * Returns abstract of this layer.
      * @return "Narrative description of the layer"
      */
@@ -199,4 +204,5 @@
 
     /**
+     * Returns flattened stream of this layer and its children.
      * @return flattened stream of this layer and its children (as well as recursively children of its children)
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoader.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoader.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoader.java	(revision 16553)
@@ -63,4 +63,5 @@
 
     /**
+     * Returns a new {@link ThreadPoolExecutor}.
      * @param nameFormat see {@link Utils#newThreadFactory(String, int)}
      * @param workers number of worker thread to keep
@@ -72,4 +73,5 @@
 
     /**
+     * Returns a new {@link ThreadPoolExecutor}.
      * @param nameFormat see {@link Utils#newThreadFactory(String, int)}
      * @param workers number of worker thread to keep
@@ -89,6 +91,7 @@
 
     /**
+     * Returns a new {@link ThreadPoolExecutor}.
      * @param name name of threads
-     * @return new ThreadPoolExecutor that will use a @see HostLimitQueue based queue, with default number of threads
+     * @return new ThreadPoolExecutor that will use a {@link HostLimitQueue} based queue, with default number of threads
      */
     public static ThreadPoolExecutor getNewThreadPoolExecutor(String name) {
@@ -112,4 +115,5 @@
 
     /**
+     * Returns cache statistics as string.
      * @return cache statistics as string
      */
@@ -148,4 +152,5 @@
 
     /**
+     * Returns download executor that is used by this factory.
      * @return download executor that is used by this factory
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/TileJobOptions.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/TileJobOptions.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/TileJobOptions.java	(revision 16553)
@@ -9,5 +9,5 @@
  *
  * @author Wiktor Niesiobedzki
- *
+ * @since 13733
  */
 public class TileJobOptions {
@@ -34,5 +34,5 @@
 
     /**
-     *
+     * Returns socket connection timeout in milliseconds.
      * @return socket connection timeout in milliseconds
      */
@@ -42,5 +42,5 @@
 
     /**
-     *
+     * Returns socket read timeout in milliseconds.
      * @return socket read timeout in milliseconds
      */
@@ -50,5 +50,5 @@
 
     /**
-     *
+     * Returns unmodifiable map with headers to be sent to tile server.
      * @return unmodifiable map with headers to be sent to tile server
      */
@@ -58,5 +58,5 @@
 
     /**
-     *
+     * Returns minimum cache expire time in seconds for downloaded tiles.
      * @return minimum cache expire time in seconds for downloaded tiles
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/WMSEndpointTileSource.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/WMSEndpointTileSource.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/WMSEndpointTileSource.java	(revision 16553)
@@ -86,5 +86,5 @@
 
     /**
-     *
+     * Returns list of EPSG codes that current layer selection supports.
      * @return list of EPSG codes that current layer selection supports (this may differ from layer to layer)
      */
Index: /trunk/src/org/openstreetmap/josm/data/imagery/WMTSTileSource.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/imagery/WMTSTileSource.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/imagery/WMTSTileSource.java	(revision 16553)
@@ -175,5 +175,5 @@
 
         /**
-         *
+         * Returns identifier of this TileMatrixSet.
          * @return identifier of this TileMatrixSet
          */
@@ -183,5 +183,5 @@
 
         /**
-         *
+         * Returns projection of this tileMatrix.
          * @return projection of this tileMatrix
          */
@@ -254,5 +254,5 @@
 
         /**
-         *
+         * Returns identifier of this layer.
          * @return identifier of this layer
          */
@@ -262,5 +262,5 @@
 
         /**
-         *
+         * Returns style of this layer.
          * @return style of this layer
          */
@@ -270,5 +270,5 @@
 
         /**
-         *
+         * Returns tileMatrixSet of this layer.
          * @return tileMatrixSet of this layer
          */
@@ -1069,4 +1069,5 @@
 
     /**
+     * Group layers by name and tile matrix set.
      * @param layers to be grouped
      * @return list with entries - grouping identifier + list of layers
@@ -1079,4 +1080,5 @@
 
     /**
+     * Returns set of projection codes that this TileSource supports.
      * @return set of projection codes that this TileSource supports
      */
Index: /trunk/src/org/openstreetmap/josm/data/osm/AbstractDataSourceChangeEvent.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/AbstractDataSourceChangeEvent.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/AbstractDataSourceChangeEvent.java	(revision 16553)
@@ -24,5 +24,5 @@
      * @param old    The previous set of DataSources
      */
-    public AbstractDataSourceChangeEvent(DataSet source, Set<DataSource> old) {
+    protected AbstractDataSourceChangeEvent(DataSet source, Set<DataSource> old) {
         CheckParameterUtil.ensureParameterNotNull(source, "source");
         CheckParameterUtil.ensureParameterNotNull(old, "old");
Index: /trunk/src/org/openstreetmap/josm/data/osm/BBox.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/BBox.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/BBox.java	(revision 16553)
@@ -378,5 +378,6 @@
 
     /**
-     * @return true if the bbox covers a part of the planets surface
+     * Determines if the bbox covers a part of the planet surface.
+     * @return true if the bbox covers a part of the planet surface.
      * Height and width must be non-negative, but may (both) be 0.
      * @since 11269
@@ -387,5 +388,6 @@
 
     /**
-     * @return true if the bbox  is avalid and covers a part of the planets surface
+     * Determines if the bbox is valid and covers a part of the planet surface.
+     * @return true if the bbox is valid and covers a part of the planet surface
      * @since 11269
      */
Index: /trunk/src/org/openstreetmap/josm/data/osm/ChangesetDataSet.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/ChangesetDataSet.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/ChangesetDataSet.java	(revision 16553)
@@ -177,4 +177,5 @@
 
     /**
+     * Returns an unmodifiable set of all primitives in this dataset.
      * @return an unmodifiable set of all primitives in this dataset.
      * @since 14946
Index: /trunk/src/org/openstreetmap/josm/data/osm/DataSelectionListener.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/DataSelectionListener.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/DataSelectionListener.java	(revision 16553)
@@ -96,5 +96,5 @@
         private final Set<OsmPrimitive> old;
 
-        public AbstractSelectionEvent(DataSet source, Set<OsmPrimitive> old) {
+        protected AbstractSelectionEvent(DataSet source, Set<OsmPrimitive> old) {
             CheckParameterUtil.ensureParameterNotNull(source, "source");
             CheckParameterUtil.ensureParameterNotNull(old, "old");
Index: /trunk/src/org/openstreetmap/josm/data/osm/DataSet.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/DataSet.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/DataSet.java	(revision 16553)
@@ -1278,4 +1278,5 @@
 
     /**
+     * Determines if this Dataset contains no primitives.
      * @return true if this Dataset contains no primitives
      * @since 14835
Index: /trunk/src/org/openstreetmap/josm/data/osm/Hash.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/Hash.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/Hash.java	(revision 16553)
@@ -6,5 +6,5 @@
  * based on some inner state of an object for a set.
  * It supports two type parameters to implement effective foreign key implementation
- * inside (@link Storage}, but for basic use, both type parameters are the same.
+ * inside {@link Storage}, but for basic use, both type parameters are the same.
  *
  * For use cases, see {@link Storage}.
Index: /trunk/src/org/openstreetmap/josm/data/osm/Node.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/Node.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/Node.java	(revision 16553)
@@ -129,5 +129,5 @@
     }
 
-    protected Node(long id, boolean allowNegative) {
+    Node(long id, boolean allowNegative) {
         super(id, allowNegative);
     }
Index: /trunk/src/org/openstreetmap/josm/data/osm/PrimitiveData.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/PrimitiveData.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/PrimitiveData.java	(revision 16553)
@@ -32,5 +32,5 @@
      * @since 12017
      */
-    public PrimitiveData(long id) {
+    protected PrimitiveData(long id) {
         this.id = id;
     }
@@ -40,5 +40,5 @@
      * @param data the data to copy
      */
-    public PrimitiveData(PrimitiveData data) {
+    protected PrimitiveData(PrimitiveData data) {
         cloneFrom(data);
     }
Index: /trunk/src/org/openstreetmap/josm/data/osm/Relation.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/Relation.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/Relation.java	(revision 16553)
@@ -182,5 +182,5 @@
     }
 
-    protected Relation(long id, boolean allowNegative) {
+    Relation(long id, boolean allowNegative) {
         super(id, allowNegative);
     }
Index: /trunk/src/org/openstreetmap/josm/data/osm/Way.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/Way.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/Way.java	(revision 16553)
@@ -182,5 +182,5 @@
     }
 
-    protected Way(long id, boolean allowNegative) {
+    Way(long id, boolean allowNegative) {
         super(id, allowNegative);
     }
Index: /trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java	(revision 16553)
@@ -54,5 +54,5 @@
      * @throws IllegalArgumentException if preconditions are violated
      */
-    public HistoryOsmPrimitive(long id, long version, boolean visible, User user, long changesetId, Date timestamp) {
+    protected HistoryOsmPrimitive(long id, long version, boolean visible, User user, long changesetId, Date timestamp) {
         this(id, version, visible, user, changesetId, timestamp, true);
     }
@@ -73,5 +73,5 @@
      * @since 5440
      */
-    public HistoryOsmPrimitive(long id, long version, boolean visible, User user, long changesetId, Date timestamp,
+    protected HistoryOsmPrimitive(long id, long version, boolean visible, User user, long changesetId, Date timestamp,
             boolean checkHistoricParams) {
         ensurePositiveLong(id, "id");
@@ -95,5 +95,5 @@
      * @param p the primitive
      */
-    public HistoryOsmPrimitive(OsmPrimitive p) {
+    protected HistoryOsmPrimitive(OsmPrimitive p) {
         this(p.getId(), p.getVersion(), p.isVisible(), p.getUser(), p.getChangesetId(), p.getTimestamp());
     }
Index: /trunk/src/org/openstreetmap/josm/data/osm/search/PushbackTokenizer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/search/PushbackTokenizer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/search/PushbackTokenizer.java	(revision 16553)
@@ -38,4 +38,5 @@
 
         /**
+         * Returns the range start.
          * @return The start
          */
@@ -45,4 +46,5 @@
 
         /**
+         * Returns the range end.
          * @return The end (inclusive)
          */
Index: /trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java	(revision 16553)
@@ -335,5 +335,5 @@
         protected final Match match;
 
-        public UnaryMatch(Match match) {
+        protected UnaryMatch(Match match) {
             if (match == null) {
                 // "operator" (null) should mean the same as "operator()"
@@ -383,5 +383,5 @@
          * @param rhs Right hand side
          */
-        public AbstractBinaryMatch(Match lhs, Match rhs) {
+        protected AbstractBinaryMatch(Match lhs, Match rhs) {
             this.lhs = lhs;
             this.rhs = rhs;
Index: /trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java	(revision 16553)
@@ -135,4 +135,5 @@
 
     /**
+     * Returns the bounding box.
      * @return The bounding box or <code>null</code> if no coordinates have passed
      */
Index: /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/AbstractMapRenderer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/AbstractMapRenderer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/AbstractMapRenderer.java	(revision 16553)
@@ -75,5 +75,5 @@
      * @throws IllegalArgumentException if {@code nc} is null
      */
-    public AbstractMapRenderer(Graphics2D g, NavigatableComponent nc, boolean isInactiveMode) {
+    protected AbstractMapRenderer(Graphics2D g, NavigatableComponent nc, boolean isInactiveMode) {
         CheckParameterUtil.ensureParameterNotNull(g);
         CheckParameterUtil.ensureParameterNotNull(nc);
Index: /trunk/src/org/openstreetmap/josm/data/preferences/AbstractProperty.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/preferences/AbstractProperty.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/preferences/AbstractProperty.java	(revision 16553)
@@ -1,4 +1,6 @@
 // License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.data.preferences;
+
+import java.util.Objects;
 
 import org.openstreetmap.josm.spi.preferences.Config;
@@ -9,6 +11,4 @@
 import org.openstreetmap.josm.tools.bugreport.BugReport;
 
-import java.util.Objects;
-
 /**
  * Captures the common functionality of preference properties
@@ -160,5 +160,5 @@
      * @since 5464
      */
-    public AbstractProperty(String key, T defaultValue) {
+    protected AbstractProperty(String key, T defaultValue) {
         // Config.getPref() should not change in production but may change during tests.
         preferences = Config.getPref();
Index: /trunk/src/org/openstreetmap/josm/data/preferences/AbstractToStringProperty.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/preferences/AbstractToStringProperty.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/preferences/AbstractToStringProperty.java	(revision 16553)
@@ -77,5 +77,5 @@
      * @see AbstractProperty#AbstractProperty(String, Object)
      */
-    public AbstractToStringProperty(String key, T defaultValue) {
+    protected AbstractToStringProperty(String key, T defaultValue) {
         super(key, defaultValue);
         storeDefaultValue();
Index: /trunk/src/org/openstreetmap/josm/data/preferences/ParametrizedEnumProperty.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/preferences/ParametrizedEnumProperty.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/preferences/ParametrizedEnumProperty.java	(revision 16553)
@@ -15,5 +15,5 @@
     protected final Class<T> enumClass;
 
-    public ParametrizedEnumProperty(Class<T> enumClass, T defaultValue) {
+    protected ParametrizedEnumProperty(Class<T> enumClass, T defaultValue) {
         this.defaultValue = defaultValue;
         this.enumClass = enumClass;
Index: /trunk/src/org/openstreetmap/josm/data/preferences/sources/ExtendedSourceEntry.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/preferences/sources/ExtendedSourceEntry.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/preferences/sources/ExtendedSourceEntry.java	(revision 16553)
@@ -42,4 +42,5 @@
 
     /**
+     * Returns string representation for GUI list or menu entry.
      * @return string representation for GUI list or menu entry
      */
Index: /trunk/src/org/openstreetmap/josm/data/preferences/sources/SourcePrefHelper.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/preferences/sources/SourcePrefHelper.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/preferences/sources/SourcePrefHelper.java	(revision 16553)
@@ -30,5 +30,5 @@
      * @since 12825
      */
-    public SourcePrefHelper(String pref, SourceType type) {
+    protected SourcePrefHelper(String pref, SourceType type) {
         this.pref = pref;
         this.type = type;
Index: /trunk/src/org/openstreetmap/josm/data/projection/AbstractProjection.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/projection/AbstractProjection.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/projection/AbstractProjection.java	(revision 16553)
@@ -181,4 +181,5 @@
 
     /**
+     * Returns The EPSG Code of this CRS.
      * @return The EPSG Code of this CRS, null if it doesn't have one.
      */
Index: /trunk/src/org/openstreetmap/josm/data/projection/datum/AbstractDatum.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/projection/datum/AbstractDatum.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/projection/datum/AbstractDatum.java	(revision 16553)
@@ -22,5 +22,5 @@
      * @param ellps The ellipsoid
      */
-    public AbstractDatum(String name, String proj4Id, Ellipsoid ellps) {
+    protected AbstractDatum(String name, String proj4Id, Ellipsoid ellps) {
         this.name = name;
         this.proj4Id = proj4Id;
Index: /trunk/src/org/openstreetmap/josm/data/projection/datum/Datum.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/projection/datum/Datum.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/projection/datum/Datum.java	(revision 16553)
@@ -14,4 +14,5 @@
 
     /**
+     * Returns a human readable name of this projection.
      * @return a human readable name of this projection
      */
@@ -26,4 +27,5 @@
 
     /**
+     * Returns the ellipsoid associated with this datum.
      * @return the ellipsoid associated with this datum
      */
Index: /trunk/src/org/openstreetmap/josm/data/validation/Test.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/Test.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/validation/Test.java	(revision 16553)
@@ -107,5 +107,5 @@
          * @param description The test description
          */
-        public TagTest(String name, String description) {
+        protected TagTest(String name, String description) {
             super(name, description);
         }
@@ -115,5 +115,5 @@
          * @param name The test name
          */
-        public TagTest(String name) {
+        protected TagTest(String name) {
             super(name);
         }
@@ -377,6 +377,7 @@
 
     /**
+     * Returns the name of this class.
      * @return the name of this class (for ToolTip)
-     * since 15972
+     * @since 15972
      */
     public Object getSource() {
Index: /trunk/src/org/openstreetmap/josm/data/validation/TestError.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/TestError.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/validation/TestError.java	(revision 16553)
@@ -531,4 +531,5 @@
 
     /**
+     * Returns a new {@link MultipleNameVisitor} for the list of primitives affected by this error.
      * @return Name visitor (used in cell renderer and for sorting)
      */
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java	(revision 16553)
@@ -304,5 +304,5 @@
      * @since 12958
      */
-    public CrossingWays(String title, int code) {
+    protected CrossingWays(String title, int code) {
         super(title, tr("This test checks if two roads, railways, waterways or buildings crosses in the same layer, " +
                 "but are not connected by a node."));
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java	(revision 16553)
@@ -211,7 +211,6 @@
      * @since 6691
      */
-    public UnconnectedWays(String title) {
+    protected UnconnectedWays(String title) {
         this(title, UNCONNECTED_WAYS, false);
-
     }
 
@@ -223,5 +222,5 @@
      * @since 14468
      */
-    public UnconnectedWays(String title, int code, boolean isHighwayTest) {
+    protected UnconnectedWays(String title, int code, boolean isHighwayTest) {
         super(title, tr("This test checks if a way has an endpoint very near to another way."));
         this.code = code;
Index: /trunk/src/org/openstreetmap/josm/gui/MainMenu.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/MainMenu.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/MainMenu.java	(revision 16553)
@@ -471,4 +471,5 @@
 
     /**
+     * Returns the default position of new top-level menus.
      * @return the default position of new top-level menus
      * @since 6088
Index: /trunk/src/org/openstreetmap/josm/gui/MapViewState.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/MapViewState.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/MapViewState.java	(revision 16553)
@@ -334,4 +334,5 @@
 
     /**
+     * Creates a new state that is moved to an east/north coordinate.
      * @param mapViewPoint The reference point.
      * @param newEastNorthThere The east/north coordinate that should be there.
Index: /trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java	(revision 16553)
@@ -431,4 +431,5 @@
 
     /**
+     * Returns geographic coordinates from a specific pixel coordination on the screen.
      * @param x X-Pixelposition to get coordinate from
      * @param y Y-Pixelposition to get coordinate from
@@ -492,4 +493,5 @@
 
     /**
+     * Returns minimum bounds that will cover a given rectangle.
      * @param r rectangle
      * @return Minimum bounds that will cover rectangle
@@ -1236,5 +1238,5 @@
     /**
      * The result *order* depends on the current map selection state.
-     * Segments within 10px of p are searched and sorted by their distance to @param p,
+     * Segments within 10px of p are searched and sorted by their distance to {@code p},
      * then, within groups of equally distant segments, prefer those that are selected.
      *
@@ -1637,4 +1639,5 @@
 
     /**
+     * Returns the projection to be used in calculating stuff.
      * @return The projection to be used in calculating stuff.
      */
Index: /trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java	(revision 16553)
@@ -35,5 +35,5 @@
      * @param title message for the user
      */
-    public PleaseWaitRunnable(String title) {
+    protected PleaseWaitRunnable(String title) {
         this(title, false);
     }
@@ -47,5 +47,5 @@
      * then use false unless you read result of task (because exception will get lost if you don't)
      */
-    public PleaseWaitRunnable(String title, boolean ignoreException) {
+    protected PleaseWaitRunnable(String title, boolean ignoreException) {
         this(title, new PleaseWaitProgressMonitor(title), ignoreException);
     }
@@ -61,5 +61,5 @@
      * @throws IllegalArgumentException if parent is null
      */
-    public PleaseWaitRunnable(Component parent, String title, boolean ignoreException) {
+    protected PleaseWaitRunnable(Component parent, String title, boolean ignoreException) {
         CheckParameterUtil.ensureParameterNotNull(parent, "parent");
         this.title = title;
@@ -77,5 +77,5 @@
      * then use false unless you read result of task (because exception will get lost if you don't)
      */
-    public PleaseWaitRunnable(String title, ProgressMonitor progressMonitor, boolean ignoreException) {
+    protected PleaseWaitRunnable(String title, ProgressMonitor progressMonitor, boolean ignoreException) {
         this.title = title;
         this.progressMonitor = progressMonitor == null ? new PleaseWaitProgressMonitor(title) : progressMonitor;
Index: /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMergeModel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMergeModel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMergeModel.java	(revision 16553)
@@ -199,5 +199,5 @@
      * Constructs a new {@code ListMergeModel}.
      */
-    public AbstractListMergeModel() {
+    protected AbstractListMergeModel() {
         entries = new EnumMap<>(ListRole.class);
         for (ListRole role : ListRole.values()) {
Index: /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMerger.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMerger.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMerger.java	(revision 16553)
@@ -408,5 +408,5 @@
      * @param model list merger model
      */
-    public AbstractListMerger(AbstractListMergeModel<T, C> model) {
+    protected AbstractListMerger(AbstractListMergeModel<T, C> model) {
         this.model = model;
         model.addChangeListener(this);
Index: /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractMergePanel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractMergePanel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractMergePanel.java	(revision 16553)
@@ -185,5 +185,5 @@
      * Create a new merge panel.
      */
-    public AbstractMergePanel() {
+    protected AbstractMergePanel() {
         super(new GridBagLayout());
     }
Index: /trunk/src/org/openstreetmap/josm/gui/conflict/pair/PairTable.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/conflict/pair/PairTable.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/conflict/pair/PairTable.java	(revision 16553)
@@ -28,5 +28,5 @@
      * @param sm selection model
      */
-    public PairTable(String name, AbstractListMergeModel<? extends PrimitiveId, ? extends ConflictResolveCommand> model,
+    protected PairTable(String name, AbstractListMergeModel<? extends PrimitiveId, ? extends ConflictResolveCommand> model,
             OsmPrimitivesTableModel dm, TableColumnModel cm, ListSelectionModel sm) {
         super(dm, cm, sm);
Index: /trunk/src/org/openstreetmap/josm/gui/correction/CorrectionTableModel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/correction/CorrectionTableModel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/correction/CorrectionTableModel.java	(revision 16553)
@@ -25,5 +25,5 @@
      * @param corrections list of corrections
      */
-    public CorrectionTableModel(List<C> corrections) {
+    protected CorrectionTableModel(List<C> corrections) {
         this.corrections = corrections;
         apply = new boolean[this.corrections.size()];
Index: /trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java	(revision 16553)
@@ -1033,4 +1033,5 @@
 
     /**
+     * Returns the last used height stored in preferences or preferredHeight.
      * @return the last used height stored in preferences or preferredHeight
      * @since 14425
Index: /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/AbstractCopyAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/AbstractCopyAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/AbstractCopyAction.java	(revision 16553)
@@ -33,5 +33,5 @@
      * @param objectSupplier a supplier which returns the selected tagged object(s)
      */
-    public AbstractCopyAction(JTable tagTable, IntFunction<String> keySupplier, Supplier<Collection<? extends Tagged>> objectSupplier) {
+    protected AbstractCopyAction(JTable tagTable, IntFunction<String> keySupplier, Supplier<Collection<? extends Tagged>> objectSupplier) {
         this.tagTable = Objects.requireNonNull(tagTable);
         this.keySupplier = Objects.requireNonNull(keySupplier);
Index: /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/HelpAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/HelpAction.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/dialogs/properties/HelpAction.java	(revision 16553)
@@ -35,5 +35,5 @@
      * Constructs a new {@code HelpAction}.
      */
-    public HelpAction() {
+    protected HelpAction() {
         putValue(SHORT_DESCRIPTION, tr("Launch browser with wiki help for selected object"));
         new ImageProvider("dialogs", "search").getResource().attachImageIcon(this, true);
Index: /trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableCellRenderer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableCellRenderer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableCellRenderer.java	(revision 16553)
@@ -15,5 +15,5 @@
  * This is the {@link TableCellRenderer} used in the tables of
  * {@link org.openstreetmap.josm.gui.conflict.pair.relation.RelationMemberMerger}.
- *
+ * @since 1790
  */
 public abstract class MemberTableCellRenderer extends JLabel implements TableCellRenderer {
@@ -25,5 +25,5 @@
      * constructor
      */
-    public MemberTableCellRenderer() {
+    protected MemberTableCellRenderer() {
         setIcon(null);
         setOpaque(true);
Index: /trunk/src/org/openstreetmap/josm/gui/download/AbstractDownloadSourcePanel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/download/AbstractDownloadSourcePanel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/download/AbstractDownloadSourcePanel.java	(revision 16553)
@@ -27,5 +27,5 @@
      * @param downloadSource The download source this panel is for
      */
-    public AbstractDownloadSourcePanel(final DownloadSource<T> downloadSource) {
+    protected AbstractDownloadSourcePanel(final DownloadSource<T> downloadSource) {
         Objects.requireNonNull(downloadSource);
         this.downloadSource = downloadSource;
Index: /trunk/src/org/openstreetmap/josm/gui/download/IDownloadSourceType.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/download/IDownloadSourceType.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/download/IDownloadSourceType.java	(revision 16553)
@@ -17,5 +17,7 @@
  */
 public interface IDownloadSourceType {
+
     /**
+     * Returns the checkbox to be added to the UI.
      * @return The checkbox to be added to the UI
      */
@@ -25,4 +27,5 @@
 
     /**
+     * Returns the checkbox to be added to the UI.
      * @param checkboxChangeListener The listener for checkboxes (may be {@code null})
      * @return The checkbox to be added to the UI
@@ -31,4 +34,5 @@
 
     /**
+     * Returns the download task class which will be getting the data.
      * @return The {@link DownloadTask} class which will be getting the data
      */
@@ -36,4 +40,5 @@
 
     /**
+     * Determines the last state of the download type.
      * @return The boolean indicating the last state of the download type
      */
@@ -43,4 +48,5 @@
 
     /**
+     * Returns the boolean property for this particular download type.
      * @return The boolean property for this particular download type
      */
Index: /trunk/src/org/openstreetmap/josm/gui/download/OSMDownloadSource.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/download/OSMDownloadSource.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/download/OSMDownloadSource.java	(revision 16553)
@@ -132,4 +132,5 @@
 
     /**
+     * Returns the possible downloads that JOSM can make in the default Download screen.
      * @return The possible downloads that JOSM can make in the default Download screen
      * @since 16503
@@ -152,4 +153,5 @@
 
     /**
+     * Removes a download source type.
      * @param type The IDownloadSourceType object to remove
      * @return {@code true} if this download types contained the specified object
@@ -360,5 +362,4 @@
             }
         }
-
     }
 
@@ -371,4 +372,5 @@
 
         /**
+         * Constructs a new {@code OSMDownloadData}.
          * @param downloadPossibilities A list of DataDownloadTypes (instantiated, with
          *                              options set)
@@ -379,4 +381,5 @@
 
         /**
+         * Returns the download possibilities.
          * @return A list of DataDownloadTypes (instantiated, with options set)
          */
Index: /trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserDialogManager.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserDialogManager.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserDialogManager.java	(revision 16553)
@@ -69,5 +69,5 @@
     private static final List<HistoryHook> hooks = new ArrayList<>();
 
-    protected HistoryBrowserDialogManager() {
+    private HistoryBrowserDialogManager() {
         MainApplication.getLayerManager().addLayerChangeListener(this);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/io/AbstractIOTask.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/AbstractIOTask.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/AbstractIOTask.java	(revision 16553)
@@ -18,5 +18,5 @@
      * Constructs a new {@code AbstractIOTask}.
      */
-    public AbstractIOTask() {
+    protected AbstractIOTask() {
         canceled = false;
         failed = false;
Index: /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadDialog.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadDialog.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadDialog.java	(revision 16553)
@@ -47,5 +47,5 @@
      * @see JComponent#getDefaultLocale
      */
-    public AbstractUploadDialog(Window owner, ModalityType modalityType) {
+    protected AbstractUploadDialog(Window owner, ModalityType modalityType) {
         super(owner, modalityType);
     }
@@ -89,5 +89,5 @@
      * @see JComponent#getDefaultLocale
      */
-    public AbstractUploadDialog(Window owner, String title, ModalityType modalityType, GraphicsConfiguration gc) {
+    protected AbstractUploadDialog(Window owner, String title, ModalityType modalityType, GraphicsConfiguration gc) {
         super(owner, title, modalityType, gc);
     }
@@ -125,5 +125,5 @@
      * @see JComponent#getDefaultLocale
      */
-    public AbstractUploadDialog(Window owner, String title, ModalityType modalityType) {
+    protected AbstractUploadDialog(Window owner, String title, ModalityType modalityType) {
         super(owner, title, modalityType);
     }
@@ -152,5 +152,5 @@
      * @see JComponent#getDefaultLocale
      */
-    public AbstractUploadDialog(Window owner, String title) {
+    protected AbstractUploadDialog(Window owner, String title) {
         super(owner, title);
     }
@@ -177,5 +177,5 @@
      * @see JComponent#getDefaultLocale
      */
-    public AbstractUploadDialog(Window owner) {
+    protected AbstractUploadDialog(Window owner) {
         super(owner);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadTask.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadTask.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadTask.java	(revision 16553)
@@ -53,5 +53,5 @@
      * then use false unless you read result of task (because exception will get lost if you don't)
      */
-    public AbstractUploadTask(String title, boolean ignoreException) {
+    protected AbstractUploadTask(String title, boolean ignoreException) {
         super(title, ignoreException);
     }
@@ -65,5 +65,5 @@
      * then use false unless you read result of task (because exception will get lost if you don't)
      */
-    public AbstractUploadTask(String title, ProgressMonitor progressMonitor, boolean ignoreException) {
+    protected AbstractUploadTask(String title, ProgressMonitor progressMonitor, boolean ignoreException) {
         super(title, progressMonitor, ignoreException);
     }
@@ -73,5 +73,5 @@
      * @param title message for the user
      */
-    public AbstractUploadTask(String title) {
+    protected AbstractUploadTask(String title) {
         super(title);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java	(revision 16553)
@@ -66,5 +66,5 @@
      *     i.e., parent relations, and for nodes, additionally, parent ways
      * @param full if the members of a relation should be downloaded as well
-     * @param newLayerName the name to use for the new layer, can be {@code null}.
+     * @param newLayerName the name to use for the new layer, can be null.
      * @param monitor ProgressMonitor to use, or null to create a new one
      */
Index: /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileExporter.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileExporter.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileExporter.java	(revision 16553)
@@ -27,5 +27,5 @@
      * @param filter The extension file filter
      */
-    public FileExporter(ExtensionFileFilter filter) {
+    protected FileExporter(ExtensionFileFilter filter) {
         this.filter = filter;
         this.enabled = true;
Index: /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileImporter.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileImporter.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/io/importexport/FileImporter.java	(revision 16553)
@@ -41,5 +41,5 @@
      * @param filter The extension file filter
      */
-    public FileImporter(ExtensionFileFilter filter) {
+    protected FileImporter(ExtensionFileFilter filter) {
         this.filter = filter;
         this.enabled = true;
Index: /trunk/src/org/openstreetmap/josm/gui/layer/AbstractCachedTileSourceLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/AbstractCachedTileSourceLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/AbstractCachedTileSourceLayer.java	(revision 16553)
@@ -45,5 +45,5 @@
      * @param info ImageryInfo describing the layer
      */
-    public AbstractCachedTileSourceLayer(ImageryInfo info) {
+    protected AbstractCachedTileSourceLayer(ImageryInfo info) {
         super(info);
 
@@ -112,4 +112,5 @@
 
     /**
+     * Returns cache configured object for specified cache region.
      * @param name of cache region
      * @return cache configured object for specified cache region
Index: /trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java	(revision 16553)
@@ -21,5 +21,5 @@
      * @param name Layer name
      */
-    public AbstractModifiableLayer(String name) {
+    protected AbstractModifiableLayer(String name) {
         super(name);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java	(revision 16553)
@@ -215,5 +215,5 @@
      * @param info imagery info
      */
-    public AbstractTileSourceLayer(ImageryInfo info) {
+    protected AbstractTileSourceLayer(ImageryInfo info) {
         super(info);
         setBackgroundLayer(true);
@@ -709,4 +709,5 @@
 
     /**
+     * Returns maximum max zoom level, that will be shown on layer.
      * @param ts TileSource for which we want to know maximum zoom level
      * @return maximum max zoom level, that will be shown on layer
@@ -717,4 +718,5 @@
 
     /**
+     * Returns minimum zoom level, that will be shown on layer.
      * @param ts TileSource for which we want to know minimum zoom level
      * @return minimum zoom level, that will be shown on layer
@@ -774,6 +776,6 @@
 
     /**
-     *
-     * @return if its allowed to zoom in
+     * Determines if it is allowed to zoom in.
+     * @return if it is allowed to zoom in
      */
     public boolean zoomIncreaseAllowed() {
@@ -1822,4 +1824,5 @@
 
         /**
+         * Determines if the task is finished.
          * @return true, if all is done
          */
@@ -1829,4 +1832,5 @@
 
         /**
+         * Returns total number of tiles to download.
          * @return total number of tiles to download
          */
@@ -1860,4 +1864,5 @@
 
         /**
+         * Returns tile loader that is used to load the tiles.
          * @return tile loader that is used to load the tiles
          */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/AlignImageryPanel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/AlignImageryPanel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/AlignImageryPanel.java	(revision 16553)
@@ -84,4 +84,5 @@
 
     /**
+     * Adds a nag panel for a given imagery info.
      * @param infoToAdd ImageryInfo for which the nag panel should be created
      */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java	(revision 16553)
@@ -373,4 +373,5 @@
 
     /**
+     * Returns the linked MarkerLayer.
      * @return the linked MarkerLayer (imported from the same file)
      * @since 15496
@@ -381,4 +382,5 @@
 
     /**
+     * Sets the linked MarkerLayer.
      * @param linkedMarkerLayer the linked MarkerLayer
      * @since 15496
Index: /trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java	(revision 16553)
@@ -69,5 +69,5 @@
      * @param info imagery info
      */
-    public ImageryLayer(ImageryInfo info) {
+    protected ImageryLayer(ImageryInfo info) {
         super(info.getName());
         this.info = info;
Index: /trunk/src/org/openstreetmap/josm/gui/layer/Layer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/Layer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/Layer.java	(revision 16553)
@@ -168,5 +168,5 @@
      * @param name Layer name
      */
-    public Layer(String name) {
+    protected Layer(String name) {
         this.propertyChangeSupport = new PropertyChangeSupport(this);
         setName(name);
@@ -195,4 +195,5 @@
 
     /**
+     * Determines whether the layer has / can handle colors.
      * @return whether the layer has / can handle colors.
      * @since 15496
@@ -220,4 +221,5 @@
 
     /**
+     * Returns a small tooltip hint about some statistics for this layer.
      * @return A small tooltip hint about some statistics for this layer.
      */
@@ -234,4 +236,5 @@
 
     /**
+     * Determines if the other layer can be merged into this layer.
      * @param other The other layer that is tested to be mergable with this.
      * @return Whether the other layer can be merged into this layer.
Index: /trunk/src/org/openstreetmap/josm/gui/layer/MapViewPaintable.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/MapViewPaintable.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/MapViewPaintable.java	(revision 16553)
@@ -123,5 +123,6 @@
 
         /**
-         * @return true if this {@link MapViewPaintable} is used as a temporary layer.
+         * Determines if this {@link MapViewPaintable} is used as a temporary layer
+         * @return true if this {@code MapViewPaintable} is used as a temporary layer.
          */
         public boolean isTemporaryLayer() {
Index: /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java	(revision 16553)
@@ -846,4 +846,5 @@
 
     /**
+     * Converts a node to a waypoint.
      * @param n the {@code Node} to convert
      * @return {@code WayPoint} object
@@ -855,4 +856,5 @@
 
     /**
+     * Converts a node to a waypoint.
      * @param n the {@code Node} to convert
      * @param time a timestamp value in milliseconds from the epoch.
Index: /trunk/src/org/openstreetmap/josm/gui/layer/TMSLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/TMSLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/TMSLayer.java	(revision 16553)
@@ -140,4 +140,5 @@
 
     /**
+     * Returns cache for TMS region.
      * @return cache for TMS region
      */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/WMSLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/WMSLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/WMSLayer.java	(revision 16553)
@@ -196,4 +196,5 @@
 
     /**
+     * Returns cache region for WMS layer.
      * @return cache region for WMS layer
      */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/WMTSLayer.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/WMTSLayer.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/WMTSLayer.java	(revision 16553)
@@ -115,4 +115,5 @@
 
     /**
+     * Returns cache region for WMTS layer.
      * @return cache region for WMTS layer
      */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/gpx/DateFilterPanel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/gpx/DateFilterPanel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/gpx/DateFilterPanel.java	(revision 16553)
@@ -37,7 +37,7 @@
 
     /**
-     * Create the panel to filter tracks on GPX layer @param layer by date
-     * Preferences will be stored in @param preferencePrefix
-     * If @param enabled = true, then the panel is created as active and filtering occurs immediately.
+     * Create the panel to filter tracks on GPX layer {@code layer} by date
+     * Preferences will be stored in {@code preferencePrefix}
+     * If {@code enabled = true}, then the panel is created as active and filtering occurs immediately.
      * @param layer GPX layer
      * @param preferencePrefix preference prefix
Index: /trunk/src/org/openstreetmap/josm/gui/layer/imagery/TilePosition.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/imagery/TilePosition.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/imagery/TilePosition.java	(revision 16553)
@@ -34,4 +34,5 @@
 
     /**
+     * Returns the x position.
      * @return the x position
      */
@@ -41,4 +42,5 @@
 
     /**
+     * Returns the y position.
      * @return the y position
      */
@@ -48,4 +50,5 @@
 
     /**
+     * Returns the zoom.
      * @return the zoom
      */
Index: /trunk/src/org/openstreetmap/josm/gui/layer/imagery/WMTSLayerSelection.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/layer/imagery/WMTSLayerSelection.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/layer/imagery/WMTSLayerSelection.java	(revision 16553)
@@ -133,4 +133,5 @@
 
     /**
+     * Returns selected layer.
      * @return selected layer
      */
Index: /trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java	(revision 16553)
@@ -147,5 +147,5 @@
 
     /**
-     * Image provider for icon. Note that this is a provider only. A @link{ImageProvider#get()} call may still fail!
+     * Image provider for icon. Note that this is a provider only. A {@link ImageProvider#get()} call may still fail!
      *
      * @param ref reference to the requested icon
Index: /trunk/src/org/openstreetmap/josm/gui/mappaint/StyleSource.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/mappaint/StyleSource.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/mappaint/StyleSource.java	(revision 16553)
@@ -78,5 +78,5 @@
      * @param title The title that can be used as menu entry
      */
-    public StyleSource(String url, String name, String title) {
+    protected StyleSource(String url, String name, String title) {
         super(SourceType.MAP_PAINT_STYLE, url, name, title, true);
     }
@@ -86,5 +86,5 @@
      * @param entry The entry to copy the data (url, name, ...) from.
      */
-    public StyleSource(SourceEntry entry) {
+    protected StyleSource(SourceEntry entry) {
         super(entry);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Functions.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Functions.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Functions.java	(revision 16553)
@@ -659,5 +659,5 @@
      * @param a first value
      * @param b second value
-     * @return {@code true} if {@code a &gt;= b}
+     * @return {@code true} if {@code a >= b}
      */
     public static boolean greater_equal(float a, float b) { // NO_UCD (unused code)
@@ -669,5 +669,5 @@
      * @param a first value
      * @param b second value
-     * @return {@code true} if {@code a &lt;= b}
+     * @return {@code true} if {@code a <= b}
      */
     public static boolean less_equal(float a, float b) { // NO_UCD (unused code)
@@ -679,5 +679,5 @@
      * @param a first value
      * @param b second value
-     * @return {@code true} if {@code a &gt; b}
+     * @return {@code true} if {@code a > b}
      */
     public static boolean greater(float a, float b) { // NO_UCD (unused code)
@@ -689,5 +689,5 @@
      * @param a first value
      * @param b second value
-     * @return {@code true} if {@code a &lt; b}
+     * @return {@code true} if {@code a < b}
      */
     public static boolean less(float a, float b) { // NO_UCD (unused code)
Index: /trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/StyleElement.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/StyleElement.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/StyleElement.java	(revision 16553)
@@ -65,5 +65,5 @@
      * is enabled, false if a style for selected state is given explicitly
      */
-    public StyleElement(float majorZIndex, float zIndex, float objectZIndex, boolean isModifier, boolean defaultSelectedHandling) {
+    protected StyleElement(float majorZIndex, float zIndex, float objectZIndex, boolean isModifier, boolean defaultSelectedHandling) {
         this.majorZIndex = majorZIndex;
         this.zIndex = zIndex;
Index: /trunk/src/org/openstreetmap/josm/gui/oauth/AbstractAuthorizationUI.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/oauth/AbstractAuthorizationUI.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/oauth/AbstractAuthorizationUI.java	(revision 16553)
@@ -28,5 +28,5 @@
      * @since 10189
      */
-    public AbstractAuthorizationUI() {
+    protected AbstractAuthorizationUI() {
     }
 
@@ -36,5 +36,5 @@
      * @since 5422
      */
-    public AbstractAuthorizationUI(String apiUrl) {
+    protected AbstractAuthorizationUI(String apiUrl) {
         setApiUrl(apiUrl);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultPreferenceSetting.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultPreferenceSetting.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultPreferenceSetting.java	(revision 16553)
@@ -24,5 +24,5 @@
      * (Not an expert option by default.)
      */
-    public DefaultPreferenceSetting() {
+    protected DefaultPreferenceSetting() {
         this(false);
     }
@@ -33,5 +33,5 @@
      * @param isExpert true, if it is an expert option
      */
-    public DefaultPreferenceSetting(boolean isExpert) {
+    protected DefaultPreferenceSetting(boolean isExpert) {
         this.isExpert = isExpert;
     }
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultTabPreferenceSetting.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultTabPreferenceSetting.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/DefaultTabPreferenceSetting.java	(revision 16553)
@@ -30,17 +30,17 @@
      * Constructs a new {@code DefaultTabPreferenceSetting}.
      */
-    public DefaultTabPreferenceSetting() {
+    protected DefaultTabPreferenceSetting() {
         this(null, null, null);
     }
 
-    public DefaultTabPreferenceSetting(String iconName, String title, String description) {
+    protected DefaultTabPreferenceSetting(String iconName, String title, String description) {
         this(iconName, title, description, false);
     }
 
-    public DefaultTabPreferenceSetting(String iconName, String title, String description, boolean isExpert) {
+    protected DefaultTabPreferenceSetting(String iconName, String title, String description, boolean isExpert) {
         this(iconName, title, description, isExpert, null);
     }
 
-    public DefaultTabPreferenceSetting(String iconName, String title, String description, boolean isExpert, JTabbedPane tabpane) {
+    protected DefaultTabPreferenceSetting(String iconName, String title, String description, boolean isExpert, JTabbedPane tabpane) {
         super(isExpert);
         this.iconName = iconName;
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java	(revision 16553)
@@ -194,5 +194,5 @@
     public interface ValidationListener {
         /**
-         *
+         * Determines if preferences can be saved.
          * @return True if preferences can be saved
          */
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java	(revision 16553)
@@ -146,5 +146,5 @@
      * @param handleIcons {@code true} if icons may be managed, {@code false} otherwise
      */
-    public SourceEditor(SourceType sourceType, String availableSourcesUrl, List<SourceProvider> sourceProviders, boolean handleIcons) {
+    protected SourceEditor(SourceType sourceType, String availableSourcesUrl, List<SourceProvider> sourceProviders, boolean handleIcons) {
 
         this.sourceType = sourceType;
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractListEditor.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractListEditor.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractListEditor.java	(revision 16553)
@@ -28,5 +28,5 @@
      * @param entry        Preference entry
      */
-    public AbstractListEditor(Component parent, String title, PrefEntry entry) {
+    protected AbstractListEditor(Component parent, String title, PrefEntry entry) {
         super(parent, title, tr("OK"), tr("Cancel"));
         this.entry = entry;
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractTableListEditor.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractTableListEditor.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AbstractTableListEditor.java	(revision 16553)
@@ -48,5 +48,5 @@
      * @param entry        Preference entry
      */
-    public AbstractTableListEditor(Component parent, String title, PrefEntry entry) {
+    protected AbstractTableListEditor(Component parent, String title, PrefEntry entry) {
         super(parent, title, entry);
         entryModel = newEntryListModel();
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/imagery/HeadersTable.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/imagery/HeadersTable.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/imagery/HeadersTable.java	(revision 16553)
@@ -111,4 +111,5 @@
 
     /**
+     * Returns headers provided by user.
      * @return headers provided by user
      */
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/projection/AbstractProjectionChoice.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/projection/AbstractProjectionChoice.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/projection/AbstractProjectionChoice.java	(revision 16553)
@@ -24,5 +24,5 @@
      * @param id unique identifier for the projection choice
      */
-    public AbstractProjectionChoice(String name, String id) {
+    protected AbstractProjectionChoice(String name, String id) {
         this.name = name;
         this.id = id;
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/projection/ListProjectionChoice.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/projection/ListProjectionChoice.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/projection/ListProjectionChoice.java	(revision 16553)
@@ -33,5 +33,5 @@
      * @param defaultIndex the default index for the combo-box
      */
-    public ListProjectionChoice(String name, String id, String[] entries, String label, int defaultIndex) {
+    protected ListProjectionChoice(String name, String id, String[] entries, String label, int defaultIndex) {
         super(name, id);
         this.entries = Utils.copyArray(entries);
@@ -47,5 +47,5 @@
      * @param label a label shown left to the combo-box
      */
-    public ListProjectionChoice(String name, String id, String[] entries, String label) {
+    protected ListProjectionChoice(String name, String id, String[] entries, String label) {
         this(name, id, entries, label, 0);
     }
Index: /trunk/src/org/openstreetmap/josm/gui/preferences/projection/SubPrefsOptions.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/preferences/projection/SubPrefsOptions.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/preferences/projection/SubPrefsOptions.java	(revision 16553)
@@ -3,9 +3,11 @@
 
 /**
- * ProjectionChoice can implement this interface to set some additional options
+ * ProjectionChoice can implement this interface to set some additional options.
+ * @since 5226
  */
 public interface SubPrefsOptions {
 
     /**
+     * Determines if the projection code should be displayed in the top panel.
      * @return true, if the projection code should be displayed in the top panel
      */
@@ -13,4 +15,5 @@
 
     /**
+     * Determines if the projection name should be displayed in the top panel.
      * @return true, if the projection name should be displayed in the top panel
      */
Index: /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java	(revision 16553)
@@ -72,4 +72,5 @@
 
     /**
+     * Returns the current filter expression.
      * @return the current filter expression; null, if no filter expression is set
      */
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractIdTextField.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractIdTextField.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractIdTextField.java	(revision 16553)
@@ -21,5 +21,5 @@
      * @param klass The validator class
      */
-    public AbstractIdTextField(Class<T> klass) {
+    protected AbstractIdTextField(Class<T> klass) {
         this(klass, 0);
     }
@@ -31,5 +31,5 @@
      * @see JosmTextField#JosmTextField(int)
      */
-    public AbstractIdTextField(Class<T> klass, int columns) {
+    protected AbstractIdTextField(Class<T> klass, int columns) {
         super(columns);
         T validator = null;
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractTextComponentValidator.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractTextComponentValidator.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/AbstractTextComponentValidator.java	(revision 16553)
@@ -85,5 +85,5 @@
      * @throws IllegalArgumentException if tc is null
      */
-    public AbstractTextComponentValidator(JTextComponent tc) {
+    protected AbstractTextComponentValidator(JTextComponent tc) {
         this(tc, true);
     }
@@ -95,5 +95,5 @@
      * @param addActionListener {@code true} to add the action listener
      */
-    public AbstractTextComponentValidator(JTextComponent tc, boolean addActionListener) {
+    protected AbstractTextComponentValidator(JTextComponent tc, boolean addActionListener) {
         this(tc, true, true, addActionListener);
     }
@@ -106,5 +106,6 @@
      * @param addActionListener {@code true} to add the action listener
      */
-    public AbstractTextComponentValidator(JTextComponent tc, boolean addFocusListener, boolean addDocumentListener, boolean addActionListener) {
+    protected AbstractTextComponentValidator(
+            JTextComponent tc, boolean addFocusListener, boolean addDocumentListener, boolean addActionListener) {
         CheckParameterUtil.ensureParameterNotNull(tc, "tc");
         this.tc = tc;
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java	(revision 16553)
@@ -190,4 +190,5 @@
 
     /**
+     * Returns the value of the floatingDividers property.
      * @return the value of the floatingDividers property
      * @see #setFloatingDividers
@@ -715,4 +716,5 @@
 
         /**
+         * Returns the invalid Node.
          * @return the invalid Node.
          */
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/OsmPrimitivesTable.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/OsmPrimitivesTable.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/OsmPrimitivesTable.java	(revision 16553)
@@ -36,5 +36,5 @@
      * @param sm selection model
      */
-    public OsmPrimitivesTable(OsmPrimitivesTableModel dm, TableColumnModel cm, ListSelectionModel sm) {
+    protected OsmPrimitivesTable(OsmPrimitivesTableModel dm, TableColumnModel cm, ListSelectionModel sm) {
         super(dm, cm, sm);
         addMouseListener(new PopupMenuLauncher(getPopUpMenu()));
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/PopupMenuLauncher.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/PopupMenuLauncher.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/PopupMenuLauncher.java	(revision 16553)
@@ -149,4 +149,5 @@
 
     /**
+     * Returns the popup menu.
      * @return the popup menu if defined, {@code null} otherwise.
      * @since 5884
Index: /trunk/src/org/openstreetmap/josm/gui/widgets/SearchTextResultListPanel.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/gui/widgets/SearchTextResultListPanel.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/gui/widgets/SearchTextResultListPanel.java	(revision 16553)
@@ -40,5 +40,5 @@
      * Constructs a new {@code SearchTextResultListPanel}.
      */
-    public SearchTextResultListPanel() {
+    protected SearchTextResultListPanel() {
         super(new BorderLayout());
 
Index: /trunk/src/org/openstreetmap/josm/io/BoundingBoxDownloader.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/BoundingBoxDownloader.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/BoundingBoxDownloader.java	(revision 16553)
@@ -256,4 +256,5 @@
 
     /**
+     * Determines if download is complete for the given bounding box.
      * @return true if download is complete for the given bounding box (not filtered)
      */
Index: /trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java	(revision 16553)
@@ -73,5 +73,5 @@
      * @param updateInterval update interval in seconds. -1 means always
      */
-    public CacheCustomContent(String ident, int updateInterval) {
+    protected CacheCustomContent(String ident, int updateInterval) {
         this.ident = ident;
         this.updateInterval = updateInterval;
Index: /trunk/src/org/openstreetmap/josm/io/OsmServerLocationReader.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/OsmServerLocationReader.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/OsmServerLocationReader.java	(revision 16553)
@@ -47,5 +47,5 @@
         protected InputStream in;
 
-        public Parser(ProgressMonitor progressMonitor, Compression compression) {
+        protected Parser(ProgressMonitor progressMonitor, Compression compression) {
             this.progressMonitor = progressMonitor;
             this.compression = compression;
Index: /trunk/src/org/openstreetmap/josm/io/OsmServerReader.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/OsmServerReader.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/OsmServerReader.java	(revision 16553)
@@ -47,5 +47,5 @@
      * Constructs a new {@code OsmServerReader}.
      */
-    public OsmServerReader() {
+    protected OsmServerReader() {
         try {
             doAuthenticate = OsmApi.isUsingOAuth()
Index: /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java	(revision 16553)
@@ -245,4 +245,5 @@
 
     /**
+     * Returns root URL of services in this GetCapabilities.
      * @return root URL of services in this GetCapabilities
      */
@@ -272,4 +273,5 @@
 
     /**
+     * Returns root URL of services without the GetCapabilities call.
      * @return root URL of services without the GetCapabilities call
      * @since 15209
@@ -304,4 +306,12 @@
 
     /**
+     * Returns URL for accessing GetMap service. String will contain following parameters:
+     * * {proj} - that needs to be replaced with projection (one of {@link #getServerProjections(List)})
+     * * {width} - that needs to be replaced with width of the tile
+     * * {height} - that needs to be replaces with height of the tile
+     * * {bbox} - that needs to be replaced with area that should be fetched (in {proj} coordinates)
+     *
+     * Format of the response will be calculated using {@link #getPreferredFormat()}
+     *
      * @param selectedLayers selected layers as subset of the tree returned by {@link #getLayers()}
      * @param selectedStyles selected styles for all selectedLayers
@@ -315,4 +325,10 @@
 
     /**
+     * Returns URL for accessing GetMap service. String will contain following parameters:
+     * * {proj} - that needs to be replaced with projection (one of {@link #getServerProjections(List)})
+     * * {width} - that needs to be replaced with width of the tile
+     * * {height} - that needs to be replaces with height of the tile
+     * * {bbox} - that needs to be replaced with area that should be fetched (in {proj} coordinates)
+     *
      * @param selectedLayers selected layers as subset of the tree returned by {@link #getLayers()}
      * @param selectedStyles selected styles for all selectedLayers
@@ -332,4 +348,10 @@
 
     /**
+     * Returns URL for accessing GetMap service. String will contain following parameters:
+     * * {proj} - that needs to be replaced with projection (one of {@link #getServerProjections(List)})
+     * * {width} - that needs to be replaced with width of the tile
+     * * {height} - that needs to be replaces with height of the tile
+     * * {bbox} - that needs to be replaced with area that should be fetched (in {proj} coordinates)
+     *
      * @param selectedLayers selected layers as list of strings
      * @param selectedStyles selected styles of layers as list of strings
@@ -524,4 +546,5 @@
 
     /**
+     * Determines if this service operates at protocol level below WMS 1.3.0
      * @return if this service operates at protocol level below 1.3.0
      */
@@ -709,6 +732,7 @@
 
     /**
+     * Returns collection of LayerDetails specified by defaultLayers.
      * @param defaultLayers default layers that should select layer object
-     * @return collection of LayerDetails specified by DefaultLayers
+     * @return collection of LayerDetails specified by defaultLayers
      */
     public List<LayerDetails> getLayers(List<DefaultLayer> defaultLayers) {
@@ -721,4 +745,5 @@
 
     /**
+     * Returns title of this service.
      * @return title of this service
      */
Index: /trunk/src/org/openstreetmap/josm/io/session/SessionReader.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/session/SessionReader.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/io/session/SessionReader.java	(revision 16553)
@@ -200,4 +200,5 @@
 
     /**
+     * Returns list of layers that are later added to the mapview.
      * @return list of layers that are later added to the mapview
      */
@@ -207,4 +208,5 @@
 
     /**
+     * Returns active layer.
      * @return active layer, or {@code null} if not set
      * @since 6271
@@ -216,4 +218,5 @@
 
     /**
+     * Returns actions executed in EDT after layers have been added.
      * @return actions executed in EDT after layers have been added (message dialog, etc.)
      */
@@ -223,5 +226,5 @@
 
     /**
-     * Return the viewport (map position and scale).
+     * Returns the viewport (map position and scale).
      * @return the viewport; can be null when no viewport info is found in the file
      */
@@ -231,5 +234,5 @@
 
     /**
-     * Return the projection choice data.
+     * Returns the projection choice data.
      * @return the projection; can be null when no projection info is found in the file
      */
Index: /trunk/src/org/openstreetmap/josm/plugins/Plugin.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/plugins/Plugin.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/plugins/Plugin.java	(revision 16553)
@@ -102,5 +102,5 @@
      * @param info the plugin information describing the plugin.
      */
-    public Plugin(PluginInformation info) {
+    protected Plugin(PluginInformation info) {
         this.info = info;
     }
Index: /trunk/src/org/openstreetmap/josm/spi/preferences/AbstractSetting.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/spi/preferences/AbstractSetting.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/spi/preferences/AbstractSetting.java	(revision 16553)
@@ -14,9 +14,10 @@
     protected Long time;
     protected boolean isNew;
+
     /**
      * Constructs a new {@code AbstractSetting} with the given value
      * @param value The setting value
      */
-    public AbstractSetting(T value) {
+    protected AbstractSetting(T value) {
         this.value = value;
         this.time = null;
Index: /trunk/src/org/openstreetmap/josm/tools/HttpClient.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/tools/HttpClient.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/tools/HttpClient.java	(revision 16553)
@@ -513,4 +513,5 @@
 
         /**
+         * Indicates that other requests to the server are unlikely in the near future.
          * @see HttpURLConnection#disconnect()
          */
Index: /trunk/src/org/openstreetmap/josm/tools/WindowsShortcut.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/tools/WindowsShortcut.java	(revision 16552)
+++ /trunk/src/org/openstreetmap/josm/tools/WindowsShortcut.java	(revision 16553)
@@ -68,4 +68,5 @@
 
     /**
+     * Returns the name of the filesystem object pointed to by this shortcut.
      * @return the name of the filesystem object pointed to by this shortcut
      */