Changeset 14193 in josm


Ignore:
Timestamp:
2018-08-29T01:25:32+02:00 (11 months ago)
Author:
Don-vip
Message:

spotbugs - IC_INIT_CIRCULARITY

Location:
trunk/src/org/openstreetmap/josm
Files:
1 added
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r13987 r14193  
    7272import org.openstreetmap.josm.gui.mappaint.styleelement.BoxTextElement.HorizontalTextAlignment;
    7373import org.openstreetmap.josm.gui.mappaint.styleelement.BoxTextElement.VerticalTextAlignment;
     74import org.openstreetmap.josm.gui.mappaint.styleelement.DefaultStyles;
    7475import org.openstreetmap.josm.gui.mappaint.styleelement.MapImage;
    75 import org.openstreetmap.josm.gui.mappaint.styleelement.NodeElement;
    7676import org.openstreetmap.josm.gui.mappaint.styleelement.RepeatImageElement.LineImageAlignment;
    7777import org.openstreetmap.josm.gui.mappaint.styleelement.StyleElement;
     
    140140            order <<= 1;
    141141            // simple node on top of icons and shapes
    142             if (NodeElement.SIMPLE_NODE_ELEMSTYLE.equals(this.style)) {
     142            if (DefaultStyles.SIMPLE_NODE_ELEMSTYLE.equals(this.style)) {
    143143                order |= 1;
    144144            }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r13917 r14193  
    2828import org.openstreetmap.josm.gui.mappaint.styleelement.AreaIconElement;
    2929import org.openstreetmap.josm.gui.mappaint.styleelement.BoxTextElement;
     30import org.openstreetmap.josm.gui.mappaint.styleelement.DefaultStyles;
    3031import org.openstreetmap.josm.gui.mappaint.styleelement.LineElement;
    3132import org.openstreetmap.josm.gui.mappaint.styleelement.NodeElement;
     
    161162            if (p.a.isEmpty()) {
    162163                if (TextLabel.AUTO_LABEL_COMPOSITION_STRATEGY.compose(osm) != null) {
    163                     p.a = NodeElement.DEFAULT_NODE_STYLELIST_TEXT;
     164                    p.a = DefaultStyles.DEFAULT_NODE_STYLELIST_TEXT;
    164165                } else {
    165                     p.a = NodeElement.DEFAULT_NODE_STYLELIST;
     166                    p.a = DefaultStyles.DEFAULT_NODE_STYLELIST;
    166167                }
    167168            } else {
     
    178179                }
    179180                if (!hasNonModifier) {
    180                     p.a = new StyleElementList(p.a, NodeElement.SIMPLE_NODE_ELEMSTYLE);
     181                    p.a = new StyleElementList(p.a, DefaultStyles.SIMPLE_NODE_ELEMSTYLE);
    181182                    if (!hasText && TextLabel.AUTO_LABEL_COMPOSITION_STRATEGY.compose(osm) != null) {
    182                         p.a = new StyleElementList(p.a, BoxTextElement.SIMPLE_NODE_TEXT_ELEMSTYLE);
     183                        p.a = new StyleElementList(p.a, DefaultStyles.SIMPLE_NODE_TEXT_ELEMSTYLE);
    183184                    }
    184185                }
     
    404405                    addIfNotNull(sl, BoxTextElement.create(env, nodeStyle.getBoxProvider()));
    405406                } else {
    406                     addIfNotNull(sl, BoxTextElement.create(env, NodeElement.SIMPLE_NODE_ELEMSTYLE_BOXPROVIDER));
     407                    addIfNotNull(sl, BoxTextElement.create(env, DefaultStyles.SIMPLE_NODE_ELEMSTYLE_BOXPROVIDER));
    407408                }
    408409            } else if (osm instanceof IRelation) {
  • trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/BoxTextElement.java

    r13919 r14193  
    99import org.openstreetmap.josm.data.osm.INode;
    1010import org.openstreetmap.josm.data.osm.IPrimitive;
    11 import org.openstreetmap.josm.data.osm.Node;
    1211import org.openstreetmap.josm.data.osm.visitor.paint.MapPaintSettings;
    1312import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
     
    1615import org.openstreetmap.josm.gui.mappaint.Environment;
    1716import org.openstreetmap.josm.gui.mappaint.Keyword;
    18 import org.openstreetmap.josm.gui.mappaint.MultiCascade;
    1917import org.openstreetmap.josm.tools.CheckParameterUtil;
    2018
     
    146144            return Objects.equals(box, that.box);
    147145        }
    148     }
    149 
    150     /**
    151      * The default style a simple node should use for it's text
    152      */
    153     public static final BoxTextElement SIMPLE_NODE_TEXT_ELEMSTYLE;
    154     static {
    155         MultiCascade mc = new MultiCascade();
    156         Cascade c = mc.getOrCreateCascade("default");
    157         c.put(TEXT, Keyword.AUTO);
    158         Node n = new Node();
    159         n.put("name", "dummy");
    160         SIMPLE_NODE_TEXT_ELEMSTYLE = create(new Environment(n, mc, "default", null), NodeElement.SIMPLE_NODE_ELEMSTYLE.getBoxProvider());
    161         if (SIMPLE_NODE_TEXT_ELEMSTYLE == null) throw new AssertionError();
    162146    }
    163147
  • trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java

    r13919 r14193  
    2020import org.openstreetmap.josm.gui.mappaint.Keyword;
    2121import org.openstreetmap.josm.gui.mappaint.MapPaintStyles.IconReference;
    22 import org.openstreetmap.josm.gui.mappaint.MultiCascade;
    23 import org.openstreetmap.josm.gui.mappaint.StyleElementList;
    2422import org.openstreetmap.josm.gui.mappaint.styleelement.BoxTextElement.BoxProvider;
    2523import org.openstreetmap.josm.gui.mappaint.styleelement.BoxTextElement.SimpleBoxProvider;
     
    4947    private static final String[] ICON_KEYS = {ICON_IMAGE, ICON_WIDTH, ICON_HEIGHT, ICON_OPACITY, ICON_OFFSET_X, ICON_OFFSET_Y};
    5048
    51     /**
    52      * The style used for simple nodes
    53      */
    54     public static final NodeElement SIMPLE_NODE_ELEMSTYLE;
    55     /**
    56      * A box provider that provides the size of a simple node
    57      */
    58     public static final BoxProvider SIMPLE_NODE_ELEMSTYLE_BOXPROVIDER;
    59     static {
    60         MultiCascade mc = new MultiCascade();
    61         mc.getOrCreateCascade("default");
    62         SIMPLE_NODE_ELEMSTYLE = create(new Environment(null, mc, "default", null), 4.1f, true);
    63         if (SIMPLE_NODE_ELEMSTYLE == null) throw new AssertionError();
    64         SIMPLE_NODE_ELEMSTYLE_BOXPROVIDER = SIMPLE_NODE_ELEMSTYLE.getBoxProvider();
    65     }
    66 
    67     /**
    68      * The default styles that are used for nodes.
    69      * @see #SIMPLE_NODE_ELEMSTYLE
    70      */
    71     public static final StyleElementList DEFAULT_NODE_STYLELIST = new StyleElementList(NodeElement.SIMPLE_NODE_ELEMSTYLE);
    72     /**
    73      * The default styles that are used for nodes with text.
    74      */
    75     public static final StyleElementList DEFAULT_NODE_STYLELIST_TEXT = new StyleElementList(NodeElement.SIMPLE_NODE_ELEMSTYLE,
    76             BoxTextElement.SIMPLE_NODE_TEXT_ELEMSTYLE);
    77 
    7849    protected NodeElement(Cascade c, MapImage mapImage, Symbol symbol, float defaultMajorZindex, RotationAngle rotationAngle) {
    7950        super(c, defaultMajorZindex);
     
    9263    }
    9364
    94     private static NodeElement create(Environment env, float defaultMajorZindex, boolean allowDefault) {
     65    static NodeElement create(Environment env, float defaultMajorZindex, boolean allowDefault) {
    9566        MapImage mapImage = createIcon(env);
    9667        Symbol symbol = null;
Note: See TracChangeset for help on using the changeset viewer.