Ignore:
Timestamp:
2015-06-06T01:48:41+02:00 (9 years ago)
Author:
Don-vip
Message:

fix some minor sonar/code style issues:

  • Avoid Protected Field In Final Class
  • Class defines fields that are used only as locals
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkList.java

    r8404 r8468  
    1414
    1515import javax.swing.DefaultListModel;
    16 import javax.swing.ImageIcon;
    1716import javax.swing.JLabel;
    1817import javax.swing.JList;
    … …  
    195194    static class BookmarkCellRenderer extends JLabel implements ListCellRenderer<BookmarkList.Bookmark> {
    196195
    197         private ImageIcon icon;
    198 
     196        /**
     197         * Constructs a new {@code BookmarkCellRenderer}.
     198         */
    199199        public BookmarkCellRenderer() {
    200200            setOpaque(true);
    201             icon = ImageProvider.get("dialogs", "bookmark");
    202             setIcon(icon);
     201            setIcon(ImageProvider.get("dialogs", "bookmark"));
    203202        }
    204203
    … …  
    217216            StringBuilder sb = new StringBuilder(128);
    218217            sb.append("<html>min[latitude,longitude]=<strong>[")
    219               .append(area.getMinLat()).append(',').append(area.getMinLon()).append("]</strong>")
    220               .append("<br>max[latitude,longitude]=<strong>[")
    221               .append(area.getMaxLat()).append(',').append(area.getMaxLon()).append("]</strong>")
    222               .append("</html>");
     218              .append(area.getMinLat()).append(',').append(area.getMinLon()).append("]</strong>"+
     219                      "<br>max[latitude,longitude]=<strong>[")
     220              .append(area.getMaxLat()).append(',').append(area.getMaxLon()).append("]</strong>"+
     221                      "</html>");
    223222            return sb.toString();
    224223        }
Note: See TracChangeset for help on using the changeset viewer.