Changeset 10300 in josm for trunk/src


Ignore:
Timestamp:
2016-05-30T01:17:28+02:00 (8 years ago)
Author:
Don-vip
Message:

sonar - Performance - Method passes constant String of length 1 to character overridden method + add unit tests/javadoc

Location:
trunk/src/org/openstreetmap/josm
Files:
39 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r10299 r10300  
    367367            return rhs;
    368368        }
     369
     370        protected static String parenthesis(Match m) {
     371            return '(' + m.toString() + ')';
     372        }
    369373    }
    370374
     
    413417        @Override
    414418        public String toString() {
    415             return "!" + match;
     419            return '!' + match.toString();
    416420        }
    417421
     
    473477        @Override
    474478        public String toString() {
    475             return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof And) ? ("(" + lhs + ')') : lhs) + " && "
    476                     + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof And) ? ("(" + rhs + ')') : rhs);
     479            return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof And) ? parenthesis(lhs) : lhs) + " && "
     480                 + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof And) ? parenthesis(rhs) : rhs);
    477481        }
    478482    }
     
    503507        @Override
    504508        public String toString() {
    505             return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof Or) ? ("(" + lhs + ')') : lhs) + " || "
    506                     + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof Or) ? ("(" + rhs + ')') : rhs);
     509            return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof Or) ? parenthesis(lhs) : lhs) + " || "
     510                 + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof Or) ? parenthesis(rhs) : rhs);
    507511        }
    508512    }
     
    533537        @Override
    534538        public String toString() {
    535             return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof Xor) ? ("(" + lhs + ')') : lhs) + " ^ "
    536                     + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof Xor) ? ("(" + rhs + ')') : rhs);
     539            return (lhs instanceof AbstractBinaryMatch && !(lhs instanceof Xor) ? parenthesis(lhs) : lhs) + " ^ "
     540                 + (rhs instanceof AbstractBinaryMatch && !(rhs instanceof Xor) ? parenthesis(rhs) : rhs);
    537541        }
    538542    }
  • trunk/src/org/openstreetmap/josm/data/AutosaveTask.java

    r10299 r10300  
    171171        while (true) {
    172172            String filename = String.format("%1$s_%2$tY%2$tm%2$td_%2$tH%2$tM%2$tS%2$tL%3$s",
    173                     layer.layerFileName, now, index == 0 ? "" : ("_" + index));
     173                    layer.layerFileName, now, index == 0 ? "" : ('_' + Integer.toString(index)));
    174174            File result = new File(autosaveDir, filename + '.' + Main.pref.get("autosave.extension", "osm"));
    175175            try {
  • trunk/src/org/openstreetmap/josm/data/ProjectionBounds.java

    r9419 r10300  
    106106    @Override
    107107    public String toString() {
    108         return "ProjectionBounds["+minEast+","+minNorth+","+maxEast+","+maxNorth+']';
     108        return "ProjectionBounds["+minEast+','+minNorth+','+maxEast+','+maxNorth+']';
    109109    }
    110110
  • trunk/src/org/openstreetmap/josm/data/Version.java

    r9407 r10300  
    191191            s += " SVN";
    192192        }
    193         String result = "JOSM/1.5 ("+ s+' '+LanguageInfo.getJOSMLocaleCode()+")";
     193        String result = "JOSM/1.5 ("+ s+' '+LanguageInfo.getJOSMLocaleCode()+')';
    194194        if (includeOsDetails && Main.platform != null) {
    195195            result += ' ' + Main.platform.getOSDescription();
  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r10212 r10300  
    863863    public String getExtendedUrl() {
    864864        return imageryType.getTypeString() + (defaultMaxZoom != 0
    865             ? "["+(defaultMinZoom != 0 ? Integer.toString(defaultMinZoom) + ',' : "")+defaultMaxZoom+"]" : "") + ':' + url;
     865            ? ('['+(defaultMinZoom != 0 ? (Integer.toString(defaultMinZoom) + ',') : "")+defaultMaxZoom+"]") : "") + ':' + url;
    866866    }
    867867
  • trunk/src/org/openstreetmap/josm/data/notes/Note.java

    r10134 r10300  
    148148    @Override
    149149    public String toString() {
    150         return tr("Note") + " " + id + ": " + getFirstComment();
     150        return tr("Note") + ' ' + id + ": " + getFirstComment();
    151151    }
    152152}
  • trunk/src/org/openstreetmap/josm/data/osm/SimplePrimitiveId.java

    r9371 r10300  
    5757    @Override
    5858    public String toString() {
    59         return type + " " + id;
     59        return type.toString() + ' ' + id;
    6060    }
    6161
  • trunk/src/org/openstreetmap/josm/data/validation/routines/DomainValidator.java

    r10298 r10300  
    18151815                case '\uFF0E': // fullwidth full stop
    18161816                case '\uFF61': // halfwidth ideographic full stop
    1817                     return ascii + "."; // restore the missing stop
     1817                    return ascii + '.'; // restore the missing stop
    18181818                default:
    18191819                    return ascii;
  • trunk/src/org/openstreetmap/josm/data/validation/tests/ConditionalKeys.java

    r8680 r10300  
    104104            // <restriction-value> @ <condition>[;<restriction-value> @ <condition>]
    105105            final List<ConditionalValue> r = new ArrayList<>();
    106             final Pattern part = Pattern.compile("([^@\\p{Space}][^@]*?)"
    107                     + "\\s*@\\s*" + "(\\([^)\\p{Space}][^)]+?\\)|[^();\\p{Space}][^();]*?)\\s*");
    108             final Matcher m = Pattern.compile("(" + part + ")(;\\s*" + part + ")*").matcher(value);
     106            final String part = Pattern.compile("([^@\\p{Space}][^@]*?)"
     107                    + "\\s*@\\s*" + "(\\([^)\\p{Space}][^)]+?\\)|[^();\\p{Space}][^();]*?)\\s*").toString();
     108            final Matcher m = Pattern.compile('(' + part + ")(;\\s*" + part + ")*").matcher(value);
    109109            if (!m.matches()) {
    110110                throw new ConditionalParsingException(tr("Does not match pattern ''restriction value @ condition''"));
  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r10285 r10300  
    258258        @Override
    259259        public String toString() {
    260             return "LayerChangeAdapter [wrapped=" + wrapped + "]";
    261         }
    262 
     260            return "LayerChangeAdapter [wrapped=" + wrapped + ']';
     261        }
    263262    }
    264263
     
    314313        @Override
    315314        public String toString() {
    316             return "EditLayerChangeAdapter [wrapped=" + wrapped + "]";
    317         }
    318 
     315            return "EditLayerChangeAdapter [wrapped=" + wrapped + ']';
     316        }
    319317    }
    320318
  • trunk/src/org/openstreetmap/josm/gui/dialogs/DeleteFromRelationConfirmationDialog.java

    r10179 r10300  
    114114                numParentRelations, numParentRelations);
    115115        @I18n.QuirkyPluralString
    116         final String msg = "<html>" + msg1 + " " + msg2 + "</html>";
     116        final String msg = "<html>" + msg1 + ' ' + msg2 + "</html>";
    117117        htmlPanel.getEditorPane().setText(msg);
    118118        invalidate();
  • trunk/src/org/openstreetmap/josm/gui/dialogs/DialogsPanel.java

    r10254 r10300  
    7272        p.setVisible(false);
    7373
    74         mSpltPane.add(p, "L"+i);
     74        mSpltPane.add(p, 'L'+Integer.toString(i));
    7575        panels.add(p);
    7676
     
    257257                ch.add(new Divider());
    258258            }
    259             Leaf l = new Leaf("L"+i);
     259            Leaf l = new Leaf('L'+Integer.toString(i));
    260260            l.setWeight(1.0 / numPanels);
    261261            ch.add(l);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/LatLonDialog.java

    r9214 r10300  
    200200        tfLatLon.setText(ll.latToString(CoordinateFormat.getDefaultFormat()) + ' ' + ll.lonToString(CoordinateFormat.getDefaultFormat()));
    201201        EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    202         tfEastNorth.setText(en.east()+" "+en.north());
     202        tfEastNorth.setText(Double.toString(en.east()) + ' ' + Double.toString(en.north()));
    203203        setOkEnabled(true);
    204204    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/MapPaintDialog.java

    r10179 r10300  
    688688            p.add(new JScrollPane(txtErrors), GBC.std().fill());
    689689            for (T t : items) {
    690                 txtErrors.append(t + "\n");
     690                txtErrors.append(t.toString() + '\n');
    691691            }
    692692        }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerVisibilityAction.java

    r10245 r10300  
    299299        @Override
    300300        public String toString() {
    301             return "OpacitySlider [getRealValue()=" + getRealValue() + "]";
     301            return "OpacitySlider [getRealValue()=" + getRealValue() + ']';
    302302        }
    303303    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TagEditHelper.java

    r10254 r10300  
    812812            Shortcut sc = Shortcut.findShortcut(KeyEvent.VK_1, commandDownMask);
    813813            if (sc != null) {
    814                 lines.add(sc.getKeyText() + " " + tr("to apply first suggestion"));
    815             }
    816             lines.add(KeyEvent.getKeyModifiersText(KeyEvent.SHIFT_MASK)+'+'+KeyEvent.getKeyText(KeyEvent.VK_ENTER) + " "
     814                lines.add(sc.getKeyText() + ' ' + tr("to apply first suggestion"));
     815            }
     816            lines.add(KeyEvent.getKeyModifiersText(KeyEvent.SHIFT_MASK)+'+'+KeyEvent.getKeyText(KeyEvent.VK_ENTER) + ' '
    817817                    +tr("to add without closing the dialog"));
    818818            sc = Shortcut.findShortcut(KeyEvent.VK_1, commandDownMask | KeyEvent.SHIFT_DOWN_MASK);
    819819            if (sc != null) {
    820                 lines.add(sc.getKeyText() + " " + tr("to add first suggestion without closing the dialog"));
     820                lines.add(sc.getKeyText() + ' ' + tr("to add first suggestion without closing the dialog"));
    821821            }
    822822            final JLabel helpLabel = new JLabel("<html>" + Utils.join("<br>", lines) + "</html>");
  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r10179 r10300  
    16251625            return null;
    16261626        if (Main.isTraceEnabled()) {
    1627             Main.trace("Clicked on tile: " + clickedTile.getXtile() + " " + clickedTile.getYtile() +
     1627            Main.trace("Clicked on tile: " + clickedTile.getXtile() + ' ' + clickedTile.getYtile() +
    16281628                " currentZoomLevel: " + currentZoomLevel);
    16291629        }
  • trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java

    r10179 r10300  
    304304            return process(to);
    305305        }
     306
     307        @Override
     308        public String toString() {
     309            return "GammaImageProcessor [gamma=" + gamma + ']';
     310        }
    306311    }
    307312
     
    387392        @Override
    388393        public String toString() {
    389             return "SharpenImageProcessor [sharpenLevel=" + sharpenLevel + "]";
     394            return "SharpenImageProcessor [sharpenLevel=" + sharpenLevel + ']';
    390395        }
    391396    }
     
    437442        @Override
    438443        public String toString() {
    439             return "ColorfulImageProcessor [colorfulness=" + colorfulness + "]";
     444            return "ColorfulImageProcessor [colorfulness=" + colorfulness + ']';
    440445        }
    441446    }
     
    470475            int type = src.getType();
    471476            if (type != dest.getType()) {
    472                 Main.trace("Cannot apply color filter: Src / Dest differ in type (" + type + "/" + dest.getType() + ")");
     477                Main.trace("Cannot apply color filter: Src / Dest differ in type (" + type + '/' + dest.getType() + ')');
    473478                return src;
    474479            }
  • trunk/src/org/openstreetmap/josm/gui/layer/MapViewPaintable.java

    r10031 r10300  
    3131         */
    3232        public PaintableInvalidationEvent(MapViewPaintable paintable) {
    33             super();
    3433            this.paintable = paintable;
    3534        }
     
    4544        @Override
    4645        public String toString() {
    47             return "LayerInvalidationEvent [layer=" + paintable + "]";
     46            return "LayerInvalidationEvent [layer=" + paintable + ']';
    4847        }
    4948    }
  • trunk/src/org/openstreetmap/josm/gui/layer/NativeScaleLayer.java

    r9914 r10300  
    240240            StringBuilder stringBuilder = new StringBuilder();
    241241            for (Scale s: this.scales) {
    242                 stringBuilder.append(s + "\n");
     242                stringBuilder.append(s.toString() + '\n');
    243243            }
    244244            return stringBuilder.toString();
  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r10280 r10300  
    804804            final GpxLayer gpxLayer = new GpxLayer(gpxData, tr("Converted from: {0}", getName()));
    805805            if (getAssociatedFile() != null) {
    806                 final String filename = getAssociatedFile().getName().replaceAll(Pattern.quote(".gpx.osm") + "$", "") + ".gpx";
     806                String filename = getAssociatedFile().getName().replaceAll(Pattern.quote(".gpx.osm") + '$', "") + ".gpx";
    807807                gpxLayer.setAssociatedFile(new File(getAssociatedFile().getParentFile(), filename));
    808808            }
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ThumbsLoader.java

    r9277 r10300  
    9595
    9696    private BufferedImage loadThumb(ImageEntry entry) {
    97         final String cacheIdent = entry.getFile()+":"+maxSize;
     97        final String cacheIdent = entry.getFile().toString()+':'+maxSize;
    9898
    9999        if (!cacheOff && cache != null) {
  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

    r10050 r10300  
    222222    @Override
    223223    public String getToolTipText() {
    224         return data.size()+" "+trn("marker", "markers", data.size());
     224        return Integer.toString(data.size())+' '+trn("marker", "markers", data.size());
    225225    }
    226226
    227227    @Override
    228228    public void mergeFrom(Layer from) {
    229         MarkerLayer layer = (MarkerLayer) from;
    230         data.addAll(layer.data);
    231         Collections.sort(data, new Comparator<Marker>() {
    232             @Override
    233             public int compare(Marker o1, Marker o2) {
    234                 return Double.compare(o1.time, o2.time);
    235             }
    236         });
     229        if (from instanceof MarkerLayer) {
     230            data.addAll(((MarkerLayer) from).data);
     231            Collections.sort(data, new Comparator<Marker>() {
     232                @Override
     233                public int compare(Marker o1, Marker o2) {
     234                    return Double.compare(o1.time, o2.time);
     235                }
     236            });
     237        }
    237238    }
    238239
  • trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java

    r10200 r10300  
    130130            } catch (NumberFormatException e) {
    131131                if (Main.isDebugEnabled()) {
    132                     Main.debug("'"+o+"' cannot be converted to float");
     132                    Main.debug('\'' + (String) o + "' cannot be converted to float");
    133133                }
    134134            }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Condition.java

    r10245 r10300  
    743743        private final Expression e;
    744744
     745        /**
     746         * Constructs a new {@code ExpressionFactory}
     747         * @param e expression
     748         */
    745749        public ExpressionCondition(Expression e) {
    746750            this.e = e;
     
    755759        @Override
    756760        public String toString() {
    757             return "[" + e + ']';
     761            return '[' + e.toString() + ']';
    758762        }
    759763    }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/LiteralExpression.java

    r8846 r10300  
    99/**
    1010 * Simple literal value, that does not depend on other expressions.
     11 * @since 5705
    1112 */
    1213public class LiteralExpression implements Expression {
    1314    private final Object literal;
    1415
     16    /**
     17     * Constructs a new {@code LiteralExpression}.
     18     * @param literal literal
     19     */
    1520    public LiteralExpression(Object literal) {
    1621        CheckParameterUtil.ensureParameterNotNull(literal);
     
    2833            return Arrays.toString((float[]) literal);
    2934        }
    30         return "<" + literal + '>';
     35        return '<' + literal.toString() + '>';
    3136    }
    3237}
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Selector.java

    r10033 r10300  
    435435        @Override
    436436        public String toString() {
    437             return left + " " + (ChildOrParentSelectorType.PARENT.equals(type) ? '<' : '>') + link + ' ' + right;
     437            return left.toString() + ' ' + (ChildOrParentSelectorType.PARENT.equals(type) ? '<' : '>') + link + ' ' + right;
    438438        }
    439439    }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/LabelCompositionStrategy.java

    r9371 r10300  
    269269        @Override
    270270        public String toString() {
    271             return "{" + getClass().getSimpleName() +'}';
     271            return '{' + getClass().getSimpleName() + '}';
    272272        }
    273273
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPreset.java

    r10254 r10300  
    530530    @Override
    531531    public String toString() {
    532         return (types == null ? "" : types) + " " + name;
     532        return (types == null ? "" : types.toString()) + ' ' + name;
    533533    }
    534534
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetSelector.java

    r9422 r10300  
    168168        @Override
    169169        public String toString() {
    170             return classification + " " + preset;
     170            return Integer.toString(classification) + ' ' + preset;
    171171        }
    172172    }
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r10216 r10300  
    241241        CheckParameterUtil.ensureParameterNotNull(changesetIds, "changesetIds");
    242242        if (changesetIds.size() > MAX_CHANGESETS_NUMBER) {
    243             Main.warn("Changeset query built with more than " + MAX_CHANGESETS_NUMBER + " changeset ids (" + changesetIds.size() + ")");
     243            Main.warn("Changeset query built with more than " + MAX_CHANGESETS_NUMBER + " changeset ids (" + changesetIds.size() + ')');
    244244        }
    245245        this.changesetIds = changesetIds;
  • trunk/src/org/openstreetmap/josm/io/MultiFetchOverpassObjectReader.java

    r9669 r10300  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.io;
     3
     4import java.util.Set;
    35
    46import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    57import org.openstreetmap.josm.gui.preferences.server.OverpassServerPreference;
    68import org.openstreetmap.josm.tools.Utils;
    7 
    8 import java.util.Set;
    99
    1010/**
     
    2020            @Override
    2121            public Object apply(Long x) {
    22                 return type.getAPIName() + "(" + x + ");>;";
     22                return type.getAPIName() + '(' + x + ");>;";
    2323            }
    2424        };
    25         final String query = "(" + Utils.join("", Utils.transform(idPackage, toOverpassExpression)) + ");out meta;";
     25        final String query = '(' + Utils.join("", Utils.transform(idPackage, toOverpassExpression)) + ");out meta;";
    2626        return "interpreter?data=" + Utils.encodeUrl(query);
    2727    }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/VersionHandler.java

    r8846 r10300  
    2323        contentType = "application/json";
    2424        if (args.containsKey("jsonp")) {
    25             content = args.get("jsonp") + " && " + args.get("jsonp") + "(" + content + ')';
     25            content = args.get("jsonp") + " && " + args.get("jsonp") + '(' + content + ')';
    2626        }
    2727    }
  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r10245 r10300  
    12291229            return true;
    12301230        } else if (jar != null) {
    1231             Main.warn("Invalid jar file ''"+jar+"'' (exists: "+jar.exists()+", canRead: "+jar.canRead()+")");
     1231            Main.warn("Invalid jar file ''"+jar+"'' (exists: "+jar.exists()+", canRead: "+jar.canRead()+')');
    12321232        }
    12331233        return false;
  • trunk/src/org/openstreetmap/josm/tools/HttpClient.java

    r10160 r10300  
    122122                final boolean hasReason = reasonForRequest != null && !reasonForRequest.isEmpty();
    123123                Main.info("{0} {1}{2} -> {3}{4}",
    124                         requestMethod, url, hasReason ? " (" + reasonForRequest + ")" : "",
     124                        requestMethod, url, hasReason ? (" (" + reasonForRequest + ')') : "",
    125125                        connection.getResponseCode(),
    126126                        connection.getContentLengthLong() > 0
    127                                 ? " (" + Utils.getSizeString(connection.getContentLengthLong(), Locale.getDefault()) + ")"
     127                                ? (" (" + Utils.getSizeString(connection.getContentLengthLong(), Locale.getDefault()) + ')')
    128128                                : ""
    129129                );
  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r10212 r10300  
    928928            } else {
    929929                final String fnMD5 = Utils.md5Hex(fn);
    930                 url = b + fnMD5.substring(0, 1) + '/' + fnMD5.substring(0, 2) + "/" + fn;
     930                url = b + fnMD5.substring(0, 1) + '/' + fnMD5.substring(0, 2) + '/' + fn;
    931931            }
    932932            result = getIfAvailableHttp(url, type);
  • trunk/src/org/openstreetmap/josm/tools/Pair.java

    r9983 r10300  
    4242        Pair<?, ?> pair = (Pair<?, ?>) other;
    4343        return Objects.equals(a, pair.a) &&
    44                 Objects.equals(b, pair.b);
     44               Objects.equals(b, pair.b);
    4545    }
    4646
     
    6262    @Override
    6363    public String toString() {
    64         return "<"+a+','+b+'>';
     64        return '<'+a.toString()+','+b.toString()+'>';
    6565    }
    6666
  • trunk/src/org/openstreetmap/josm/tools/bugreport/BugReport.java

    r10285 r10300  
    7070            if (className.equals(element.getClassName()) && "getCallingMethod".equals(element.getMethodName())) {
    7171                StackTraceElement toReturn = stackTrace[i + offset];
    72                 return toReturn.getClassName().replaceFirst(".*\\.", "") + "#" + toReturn.getMethodName();
     72                return toReturn.getClassName().replaceFirst(".*\\.", "") + '#' + toReturn.getMethodName();
    7373            }
    7474        }
  • trunk/src/org/openstreetmap/josm/tools/bugreport/ReportedException.java

    r10285 r10300  
    124124
    125125    private static String niceThreadName(Thread thread) {
    126         String name = "Thread: " + thread.getName() + " (" + thread.getId() + ")";
     126        String name = "Thread: " + thread.getName() + " (" + thread.getId() + ')';
    127127        ThreadGroup threadGroup = thread.getThreadGroup();
    128128        if (threadGroup != null) {
     
    221221    @Override
    222222    public String toString() {
    223         StringBuilder builder = new StringBuilder();
    224         builder.append("CrashReportedException [on thread ");
    225         builder.append(caughtOnThread);
    226         builder.append("]");
    227         return builder.toString();
     223        return new StringBuilder(48)
     224            .append("CrashReportedException [on thread ")
     225            .append(caughtOnThread)
     226            .append(']')
     227            .toString();
    228228    }
    229229
     
    235235            this.key = key;
    236236            this.value = value;
    237 
    238237        }
    239238
     
    273272         */
    274273        public void printSection(PrintWriter out) {
    275             out.println(sectionName + ":");
     274            out.println(sectionName + ':');
    276275            if (entries.isEmpty()) {
    277276                out.println("No data collected.");
Note: See TracChangeset for help on using the changeset viewer.