Changeset 10378 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2016-06-15T10:30:37+02:00 (10 years ago)

Author:

Don-vip

Message:

Checkstyle 6.19: enable SingleSpaceSeparator and fix violations

Location:

trunk/src/org/openstreetmap/josm/io

Files:

• Capabilities.java (modified) (1 diff)
• ChangesetQuery.java (modified) (3 diffs)
• DefaultProxySelector.java (modified) (1 diff)
• DiffResultProcessor.java (modified) (3 diffs)
• GpxReader.java (modified) (1 diff)
• MultiFetchServerObjectReader.java (modified) (2 diffs)
• NoteReader.java (modified) (1 diff)
• OsmApi.java (modified) (1 diff)
• OsmChangeImporter.java (modified) (1 diff)
• OsmServerBackreferenceReader.java (modified) (2 diffs)
• OsmServerReader.java (modified) (2 diffs)
• auth/AbstractCredentialsAgent.java (modified) (1 diff)
• remotecontrol/AddTagsDialog.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/io/Capabilities.java

@@ -131 +131 @@
             }
         } else {
-            if (!capabilities.containsKey(element))  {
+            if (!capabilities.containsKey(element)) {
                 Map<String, String> h = new HashMap<>();
                 capabilities.put(element, h);

trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

@@ -157 +157 @@
         CheckParameterUtil.ensureParameterNotNull(min, "min");
         CheckParameterUtil.ensureParameterNotNull(max, "max");
-        this.bounds  = new Bounds(min, max);
+        this.bounds = new Bounds(min, max);
         return this;
     }
@@ -215 +215 @@
      */
     public ChangesetQuery beingOpen(boolean isOpen) {
-        this.open =  isOpen;
+        this.open = isOpen;
         return this;
     }
@@ -474 +474 @@
 
         protected Map<String, String> createMapFromQueryString(String query) {
-            Map<String, String> queryParams  = new HashMap<>();
+            Map<String, String> queryParams = new HashMap<>();
             String[] keyValuePairs = query.split("&");
             for (String keyValuePair: keyValuePairs) {

trunk/src/org/openstreetmap/josm/io/DefaultProxySelector.java

@@ -97 +97 @@
             return 0;
         }
-        if (port <= 0 || port >  65535) {
+        if (port <= 0 || port > 65535) {
             Main.error(tr("Illegal port number in preference ''{0}''. Got {1}.", property, port));
             Main.error(tr("The proxy will not be used."));

trunk/src/org/openstreetmap/josm/io/DiffResultProcessor.java

@@ -32 +32 @@
 import org.xml.sax.helpers.DefaultHandler;
 
-public class DiffResultProcessor  {
+public class DiffResultProcessor {
 
     private static class DiffResultEntry {
@@ -76 +76 @@
      *
      */
-    public  void parse(String diffUploadResponse, ProgressMonitor progressMonitor) throws XmlParsingException {
+    public void parse(String diffUploadResponse, ProgressMonitor progressMonitor) throws XmlParsingException {
         if (progressMonitor == null) {
             progressMonitor = NullProgressMonitor.INSTANCE;
@@ -168 +168 @@
                 case "way":
                 case "relation":
-                    PrimitiveId id  = new SimplePrimitiveId(
+                    PrimitiveId id = new SimplePrimitiveId(
                             Long.parseLong(atts.getValue("old_id")),
                             OsmPrimitiveType.fromApiTypeName(qName)

trunk/src/org/openstreetmap/josm/io/GpxReader.java

@@ -504 +504 @@
 
         @Override
-        public void endDocument() throws SAXException  {
+        public void endDocument() throws SAXException {
             if (!states.empty())
                 throw new SAXException(tr("Parse error: invalid document structure for GPX document."));

trunk/src/org/openstreetmap/josm/io/MultiFetchServerObjectReader.java

@@ -314 +314 @@
         final String baseUrl = getBaseUrl();
         switch (type) {
+            // CHECKSTYLE.OFF: SingleSpaceSeparator
             case NODE:     msg = tr("Fetching a package of nodes from ''{0}''",     baseUrl); break;
             case WAY:      msg = tr("Fetching a package of ways from ''{0}''",      baseUrl); break;
             case RELATION: msg = tr("Fetching a package of relations from ''{0}''", baseUrl); break;
+            // CHECKSTYLE.ON: SingleSpaceSeparator
             default: throw new AssertionError();
         }
@@ -584 +586 @@
                     String msg;
                     switch (type) {
+                        // CHECKSTYLE.OFF: SingleSpaceSeparator
                         case NODE:     msg = tr("Fetching node with id {0} from ''{1}''",     id, baseUrl); break;
                         case WAY:      msg = tr("Fetching way with id {0} from ''{1}''",      id, baseUrl); break;
                         case RELATION: msg = tr("Fetching relation with id {0} from ''{1}''", id, baseUrl); break;
+                        // CHECKSTYLE.ON: SingleSpaceSeparator
                         default: throw new AssertionError();
                     }

trunk/src/org/openstreetmap/josm/io/NoteReader.java

@@ -194 +194 @@
 
         @Override
-        public void endDocument() throws SAXException  {
+        public void endDocument() throws SAXException {
             parsedNotes = notes;
         }

trunk/src/org/openstreetmap/josm/io/OsmApi.java

@@ -133 +133 @@
      * @throws IllegalArgumentException if serverUrl is null
      */
-    protected OsmApi(String serverUrl)  {
+    protected OsmApi(String serverUrl) {
         CheckParameterUtil.ensureParameterNotNull(serverUrl, "serverUrl");
         this.serverUrl = serverUrl;

trunk/src/org/openstreetmap/josm/io/OsmChangeImporter.java

@@ -44 +44 @@
     }
 
-    protected void importData(InputStream in, final File associatedFile, ProgressMonitor  progressMonitor) throws IllegalDataException {
+    protected void importData(InputStream in, final File associatedFile, ProgressMonitor progressMonitor) throws IllegalDataException {
         final DataSet dataSet = OsmChangeReader.parseDataSet(in, progressMonitor);
         final OsmDataLayer layer = new OsmDataLayer(dataSet, associatedFile.getName(), associatedFile);

trunk/src/org/openstreetmap/josm/io/OsmServerBackreferenceReader.java

@@ -98 +98 @@
      * @throws IllegalArgumentException if type is null
      */
-    public OsmServerBackreferenceReader(long id, OsmPrimitiveType type, boolean readFull)  {
+    public OsmServerBackreferenceReader(long id, OsmPrimitiveType type, boolean readFull) {
         this(id, type);
         this.readFull = readFull;
@@ -202 +202 @@
             }
             if (isReadFull()) {
-                Collection<Relation> relationsToCheck  = new ArrayList<>(ds.getRelations());
+                Collection<Relation> relationsToCheck = new ArrayList<>(ds.getRelations());
                 for (Relation relation: relationsToCheck) {
                     if (!relation.isNew() && relation.hasIncompleteMembers()) {

trunk/src/org/openstreetmap/josm/io/OsmServerReader.java

@@ -41 +41 @@
      * @throws OsmTransferException if data transfer errors occur
      */
-    protected InputStream getInputStream(String urlStr, ProgressMonitor progressMonitor) throws OsmTransferException  {
+    protected InputStream getInputStream(String urlStr, ProgressMonitor progressMonitor) throws OsmTransferException {
         return getInputStream(urlStr, progressMonitor, null);
     }
@@ -55 +55 @@
      * @throws OsmTransferException if data transfer errors occur
      */
-    protected InputStream getInputStream(String urlStr, ProgressMonitor progressMonitor, String reason) throws OsmTransferException  {
+    protected InputStream getInputStream(String urlStr, ProgressMonitor progressMonitor, String reason) throws OsmTransferException {
         try {
             api.initialize(progressMonitor);

trunk/src/org/openstreetmap/josm/io/auth/AbstractCredentialsAgent.java

@@ -20 +20 @@
         if (requestorType == null)
             return null;
-        PasswordAuthentication credentials =  lookup(requestorType, host);
+        PasswordAuthentication credentials = lookup(requestorType, host);
         final String username = (credentials == null || credentials.getUserName() == null) ? "" : credentials.getUserName();
         final String password = (credentials == null || credentials.getPassword() == null) ? "" : String.valueOf(credentials.getPassword());

trunk/src/org/openstreetmap/josm/io/remotecontrol/AddTagsDialog.java

@@ -150 +150 @@
             ExistingValues old = new ExistingValues(key);
             for (OsmPrimitive osm : sel) {
-                oldValue  = osm.get(key);
+                oldValue = osm.get(key);
                 if (oldValue != null) {
                     old.addValue(oldValue);
@@ -209 +209 @@
         propertyTable.getInputMap().put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, KeyEvent.SHIFT_MASK), "shiftenter");
         propertyTable.getActionMap().put("shiftenter", new AbstractAction() {
-            @Override  public void actionPerformed(ActionEvent e) {
+            @Override public void actionPerformed(ActionEvent e) {
                 buttonAction(1, e); // add all tags on Shift-Enter
             }
@@ -242 +242 @@
     protected void buttonAction(int buttonIndex, ActionEvent evt) {
         // if layer all layers were closed, ignore all actions
-        if (Main.main.getCurrentDataSet() != null  && buttonIndex != 2) {
+        if (Main.main.getCurrentDataSet() != null && buttonIndex != 2) {
             TableModel tm = propertyTable.getModel();
             for (int i = 0; i < tm.getRowCount(); i++) {