Changeset 11383 in josm for trunk/src


Ignore:
Timestamp:
2016-12-12T17:29:27+01:00 (7 years ago)
Author:
Don-vip
Message:

findbugs - BC_UNCONFIRMED_CAST

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/AutoScaleAction.java

    r11288 r11383  
    235235        switch (mode) {
    236236        case "problem":
    237             return modeProblem(v);
     237            return modeProblem((ValidatorBoundingXYVisitor) v);
    238238        case "data":
    239239            return modeData(v);
    … …  
    250250    }
    251251
    252     private static BoundingXYVisitor modeProblem(BoundingXYVisitor v) {
     252    private static BoundingXYVisitor modeProblem(ValidatorBoundingXYVisitor v) {
    253253        TestError error = Main.map.validatorDialog.getSelectedError();
    254254        if (error == null)
    255255            return null;
    256         ((ValidatorBoundingXYVisitor) v).visit(error);
     256        v.visit(error);
    257257        if (v.getBounds() == null)
    258258            return null;

  • trunk/src/org/openstreetmap/josm/data/osm/Node.java

    r11292 r11383  
    238238    @Override
    239239    public void cloneFrom(OsmPrimitive osm) {
     240        if (!(osm instanceof Node))
     241            throw new IllegalArgumentException("Not a node: " + osm);
    240242        boolean locked = writeLock();
    241243        try {
    … …  
    260262    @Override
    261263    public void mergeFrom(OsmPrimitive other) {
     264        if (!(other instanceof Node))
     265            throw new IllegalArgumentException("Not a node: " + other);
    262266        boolean locked = writeLock();
    263267        try {
    … …  
    271275    }
    272276
    273     @Override public void load(PrimitiveData data) {
     277    @Override
     278    public void load(PrimitiveData data) {
     279        if (!(data instanceof NodeData))
     280            throw new IllegalArgumentException("Not a node data: " + data);
    274281        boolean locked = writeLock();
    275282        try {
    … …  
    281288    }
    282289
    283     @Override public NodeData save() {
     290    @Override
     291    public NodeData save() {
    284292        NodeData data = new NodeData();
    285293        saveCommonAttributes(data);

  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r11373 r11383  
    10881088     *
    10891089     * Both this and other must be new, or both must be assigned an OSM ID. If both this and <code>other</code>
    1090      * have an assigend OSM id, the IDs have to be the same.
     1090     * have an assigned OSM id, the IDs have to be the same.
    10911091     *
    10921092     * @param other the other primitive. Must not be null.

  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r11316 r11383  
    240240    @Override
    241241    public void cloneFrom(OsmPrimitive osm) {
     242        if (!(osm instanceof Relation))
     243            throw new IllegalArgumentException("Not a relation: " + osm);
    242244        boolean locked = writeLock();
    243245        try {
    … …  
    252254    @Override
    253255    public void load(PrimitiveData data) {
     256        if (!(data instanceof RelationData))
     257            throw new IllegalArgumentException("Not a relation data: " + data);
    254258        boolean locked = writeLock();
    255259        try {
    … …  
    271275    }
    272276
    273     @Override public RelationData save() {
     277    @Override
     278    public RelationData save() {
    274279        RelationData data = new RelationData();
    275280        saveCommonAttributes(data);

  • trunk/src/org/openstreetmap/josm/data/osm/Way.java

    r11292 r11383  
    277277    @Override
    278278    public void load(PrimitiveData data) {
     279        if (!(data instanceof WayData))
     280            throw new IllegalArgumentException("Not a way data: " + data);
    279281        boolean locked = writeLock();
    280282        try {
    … …  
    314316    @Override
    315317    public void cloneFrom(OsmPrimitive osm) {
     318        if (!(osm instanceof Way))
     319            throw new IllegalArgumentException("Not a way: " + osm);
    316320        boolean locked = writeLock();
    317321        try {
    … …  
    326330    @Override
    327331    public String toString() {
    328         String nodesDesc = isIncomplete() ? "(incomplete)" : "nodes=" + Arrays.toString(nodes);
     332        String nodesDesc = isIncomplete() ? "(incomplete)" : ("nodes=" + Arrays.toString(nodes));
    329333        return "{Way id=" + getUniqueId() + " version=" + getVersion()+ ' ' + getFlagsAsString() + ' ' + nodesDesc + '}';
    330334    }

  • trunk/src/org/openstreetmap/josm/data/validation/tests/UntaggedNode.java

    r11129 r11383  
    5454    }
    5555
     56    private static OsmPrimitive[] castPrim(AbstractPrimitive n) {
     57        return n instanceof OsmPrimitive ? (new OsmPrimitive[]{(OsmPrimitive) n}) : (new OsmPrimitive[0]);
     58    }
     59
    5660    @Override
    5761    public void visitKeyValue(AbstractPrimitive n, String key, String value) {
    … …  
    6064            errors.add(TestError.builder(this, Severity.WARNING, UNTAGGED_NODE_FIXME)
    6165                    .message(ERROR_MESSAGE, marktr("Has tag containing ''fixme'' or ''FIXME''"))
    62                     .primitives((OsmPrimitive) n)
     66                    .primitives(castPrim(n))
    6367                    .build());
    6468            return;
    … …  
    8791            errors.add(TestError.builder(this, Severity.WARNING, code)
    8892                    .message(ERROR_MESSAGE, msg)
    89                     .primitives((OsmPrimitive) n)
     93                    .primitives(castPrim(n))
    9094                    .build());
    9195            return;
    … …  
    9498        errors.add(TestError.builder(this, Severity.WARNING, UNTAGGED_NODE_OTHER)
    9599                .message(ERROR_MESSAGE, marktr("Other"))
    96                 .primitives((OsmPrimitive) n)
     100                .primitives(castPrim(n))
    97101                .build());
    98102    }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java

    r10627 r11383  
    943943    @Override
    944944    public void eventDispatched(AWTEvent event) {
    945         if (isShowing() && !isCollapsed && isDocked && buttonHiding == ButtonHidingType.DYNAMIC) {
    946             if (buttonsPanel != null) {
    947                 Rectangle b = this.getBounds();
    948                 b.setLocation(getLocationOnScreen());
    949                 if (b.contains(((MouseEvent) event).getLocationOnScreen())) {
    950                     if (!buttonsPanel.isVisible()) {
    951                         buttonsPanel.setVisible(true);
    952                     }
    953                 } else if (buttonsPanel.isVisible()) {
    954                     buttonsPanel.setVisible(false);
    955                 }
     945        if (event instanceof MouseEvent && isShowing() && !isCollapsed && isDocked && buttonHiding == ButtonHidingType.DYNAMIC
     946                && buttonsPanel != null) {
     947            Rectangle b = this.getBounds();
     948            b.setLocation(getLocationOnScreen());
     949            if (b.contains(((MouseEvent) event).getLocationOnScreen())) {
     950                if (!buttonsPanel.isVisible()) {
     951                    buttonsPanel.setVisible(true);
     952                }
     953            } else if (buttonsPanel.isVisible()) {
     954                buttonsPanel.setVisible(false);
    956955            }
    957956        }

  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r10959 r11383  
    280280    @Override
    281281    public void mergeFrom(Layer from) {
     282        if (!(from instanceof GpxLayer))
     283            throw new IllegalArgumentException("not a GpxLayer: " + from);
    282284        data.mergeFrom(((GpxLayer) from).data);
    283285        drawHelper.dataChanged();

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r10763 r11383  
    401401    @Override
    402402    public void mergeFrom(Layer from) {
     403        if (!(from instanceof GeoImageLayer))
     404            throw new IllegalArgumentException("not a GeoImageLayer: " + from);
    403405        GeoImageLayer l = (GeoImageLayer) from;
    404406

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java

    r11374 r11383  
    622622                matchingRuleIndex = wayRules;
    623623            }
    624         } else {
     624        } else if (osm instanceof Relation) {
    625625            if (((Relation) osm).isMultipolygon()) {
    626626                matchingRuleIndex = multipolygonRules;
    … …  
    630630                matchingRuleIndex = relationRules;
    631631            }
    632         }
    633 
    634         // the declaration indices are sorted, so it suffices to save the
    635         // last used index
     632        } else {
     633            throw new IllegalArgumentException("Unsupported type: " + osm);
     634        }
     635
     636        // the declaration indices are sorted, so it suffices to save the last used index
    636637        int lastDeclUsed = -1;
    637638
Note: See TracChangeset for help on using the changeset viewer.