Changeset 7801 in josm for trunk/src/org


Ignore:
Timestamp:
2014-12-13T01:07:04+01:00 (10 years ago)
Author:
Don-vip
Message:

fix some Sonar issues

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r7678 r7801  
    983983            dialog.setVisible(true);
    984984            switch(dialog.getUserAction()) {
    985             case CANCEL: return false;
    986985            case PROCEED: return true;
     986            case CANCEL:
    987987            default: return false;
    988988            }

  • trunk/src/org/openstreetmap/josm/corrector/ReverseWayNoTagCorrector.java

    r6524 r7801  
    2929        // Hide default constructor for utils classes
    3030    }
    31    
     31
    3232    /**
    3333     * Tags that imply a semantic meaning from the way direction and cannot be changed.
    … …  
    4545            new Tag("waterway", "canal")
    4646    }));
    47    
     47
    4848    /**
    4949     * Replies the tags that imply a semantic meaning from <code>way</code> direction and cannot be changed.
    … …  
    5454        return directionalTags.intersect(TagCollection.from(way));
    5555    }
    56    
     56
    5757    /**
    5858     * Tests whether way can be reversed without semantic change.
    … …  
    6464        return getDirectionalTags(way).isEmpty();
    6565    }
    66    
     66
    6767    protected static String getHTML(TagCollection tags) {
    6868        if (tags.size() == 1) {
    … …  
    7474        }
    7575    }
    76    
     76
    7777    protected static boolean confirmReverseWay(Way way, TagCollection tags) {
    7878        String msg = trn(
    … …  
    100100        );
    101101        switch(ret) {
    102             case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION : return true;
    103             case JOptionPane.YES_OPTION: return true;
    104             default: return false;
     102            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION:
     103            case JOptionPane.YES_OPTION:
     104                return true;
     105            default:
     106                return false;
    105107        }
    106108    }
    107    
     109
    108110    /**
    109111     * Checks the given way can be safely reversed and asks user to confirm the operation if it not the case.

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

    r7752 r7801  
    14581458        private final int to;
    14591459        private final List<StyleRecord> output;
    1460         private final DataSet data;
    14611460
    14621461        private final ElemStyles styles = MapPaintStyles.getStyles();
    … …  
    14721471         * @param to last index + 1
    14731472         * @param output the list of styles to which styles will be added
    1474          * @param data the data set
    14751473         */
    1476         public ComputeStyleListWorker(final List<? extends OsmPrimitive> input, int from, int to, List<StyleRecord> output, DataSet data) {
     1474        public ComputeStyleListWorker(final List<? extends OsmPrimitive> input, int from, int to, List<StyleRecord> output) {
    14771475            this.input = input;
    14781476            this.from = from;
    14791477            this.to = to;
    14801478            this.output = output;
    1481             this.data = data;
    14821479            this.styles.setDrawMultipolygon(drawMultipolygon);
    14831480        }
    … …  
    15951592                int to = Math.min((i+1)*bucketsize, prims.size());
    15961593                List<StyleRecord> target = singleThread ? allStyleElems : new ArrayList<StyleRecord>(to - from);
    1597                 tasks.add(new ComputeStyleListWorker(prims, from, to, target, data));
     1594                tasks.add(new ComputeStyleListWorker(prims, from, to, target));
    15981595            }
    15991596            if (singleThread) {

  • trunk/src/org/openstreetmap/josm/data/validation/routines/DomainValidator.java

    r7779 r7801  
    126126     * @return true if the parameter is a valid domain name
    127127     */
     128    @Override
    128129    public boolean isValid(String domain) {
    129130        String[] groups = domainRegex.match(domain);

  • trunk/src/org/openstreetmap/josm/data/validation/routines/InetAddressValidator.java

    r7489 r7801  
    5555     * @return true if the string validates as an IP address
    5656     */
     57    @Override
    5758    public boolean isValid(String inetAddress) {
    5859        return isValidInet4Address(inetAddress);

  • trunk/src/org/openstreetmap/josm/data/validation/routines/RegexValidator.java

    r7489 r7801  
    123123     * otherwise <code>false</code>.
    124124     */
     125    @Override
    125126    public boolean isValid(String value) {
    126127        if (value == null) {

  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/MultiValueResolutionDecision.java

    r7751 r7801  
    3535
    3636    /**
    37      * constuctor
     37     * constructor
    3838     */
    3939    public MultiValueResolutionDecision() {
    … …  
    148148        case KEEP_ONE: return value;
    149149        case SUM_ALL_NUMERIC: return tags.getSummedValues(getKey());
    150         case KEEP_NONE: return null;
    151150        case KEEP_ALL: return tags.getJoinedValues(getKey());
     151        case KEEP_NONE:
    152152        default: return null;
    153153        }
    … …  
    308308        case KEEP_ONE: return new Tag(getKey(),value);
    309309        case KEEP_NONE: return new Tag(getKey(), "");
    310         case UNDECIDED: return null;
     310        case UNDECIDED:
    311311        default: return null;
    312312        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r7742 r7801  
    664664            );
    665665            switch(ret) {
    666             case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return;
    667             case JOptionPane.CLOSED_OPTION: return;
    668             case JOptionPane.NO_OPTION: return;
     666            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION:
     667            case JOptionPane.CLOSED_OPTION:
     668            case JOptionPane.NO_OPTION:
     669                return;
    669670            case JOptionPane.YES_OPTION:
    670671                memberTableModel.removeMembersReferringTo(toCheck);
    … …  
    714715        );
    715716        switch(ret) {
    716         case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION : return true;
    717         case JOptionPane.YES_OPTION: return true;
    718         case JOptionPane.NO_OPTION: return false;
    719         case JOptionPane.CLOSED_OPTION: return false;
    720         case JOptionPane.CANCEL_OPTION: throw new AddAbortException();
     717        case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION:
     718        case JOptionPane.YES_OPTION:
     719            return true;
     720        case JOptionPane.NO_OPTION:
     721        case JOptionPane.CLOSED_OPTION:
     722            return false;
     723        case JOptionPane.CANCEL_OPTION:
     724            throw new AddAbortException();
    721725        }
    722726        // should not happen
    … …  
    15811585            );
    15821586            switch(ret) {
    1583             case JOptionPane.YES_OPTION: return true;
    1584             case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return true;
     1587            case JOptionPane.YES_OPTION:
     1588            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION:
     1589                return true;
    15851590            default:
    15861591                return false;

  • trunk/src/org/openstreetmap/josm/gui/history/TagInfoViewer.java

    r6207 r7801  
    1717public class TagInfoViewer extends HistoryViewerPanel {
    1818
     19    @Override
    1920    protected JTable buildReferenceTable() {
    2021        JTable table = new JTable(
    … …  
    2829    }
    2930
     31    @Override
    3032    protected JTable buildCurrentTable() {
    3133        JTable table = new JTable(

  • trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java

    r7712 r7801  
    252252        switch(strategy) {
    253253        case INDIVIDUAL_OBJECTS_STRATEGY:
    254             spec.setStrategy(strategy);
    255             break;
    256254        case SINGLE_REQUEST_STRATEGY:
    257255            spec.setStrategy(strategy);

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

    r7509 r7801  
    414414         * @return {@code true} if all conditions apply, false otherwise.
    415415         */
     416        @Override
    416417        public boolean matches(Environment env) {
    417418            if (conds == null) return true;

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlCondition.java

    r6890 r7801  
    44import org.openstreetmap.josm.data.osm.OsmUtils;
    55
    6 public class XmlCondition
    7 {
     6public class XmlCondition {
     7
    88    public String key;
    99    public String value;
    … …  
    3333    }
    3434
     35    @Override
    3536    public String toString() {
    3637      return "Rule["+key+","+(boolValue != null ? "b="+boolValue:"v="+value)+"]";

  • trunk/src/org/openstreetmap/josm/gui/widgets/EditableList.java

    r7005 r7801  
    22package org.openstreetmap.josm.gui.widgets;
    33
    4 import org.openstreetmap.josm.Main;
    5 import org.openstreetmap.josm.tools.GBC;
     4import static org.openstreetmap.josm.tools.I18n.tr;
    65
    7 import javax.swing.DefaultListModel;
    8 import javax.swing.JButton;
    9 import javax.swing.JList;
    10 import javax.swing.JOptionPane;
    11 import javax.swing.JPanel;
    12 import javax.swing.JScrollPane;
    136import java.awt.BorderLayout;
    147import java.awt.Dimension;
    … …  
    1912import java.util.List;
    2013
    21 import static org.openstreetmap.josm.tools.I18n.tr;
     14import javax.swing.DefaultListModel;
     15import javax.swing.JButton;
     16import javax.swing.JList;
     17import javax.swing.JOptionPane;
     18import javax.swing.JPanel;
     19import javax.swing.JScrollPane;
     20
     21import org.openstreetmap.josm.Main;
     22import org.openstreetmap.josm.tools.GBC;
    2223
    2324/**
    … …  
    138139    }
    139140
     141    @Override
    140142    public void setEnabled(boolean enabled) {
    141143        sourcesList.setEnabled(enabled);

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

    r7575 r7801  
    405405                    break;
    406406                case "time":
    407                     currentWayPoint.put(localName, accumulator.toString());
    408                     currentWayPoint.setTime();
    409                     break;
    410407                case "cmt":
    411408                case "desc":

  • trunk/src/org/openstreetmap/josm/io/OsmChangesetContentParser.java

    r7082 r7801  
    8484                data.put(currentPrimitive, currentModificationType);
    8585                break;
    86             case "osmChange":
    87                 // do nothing
    88                 break;
    8986            case "create":
    90                 currentModificationType = null;
    91                 break;
    9287            case "modify":
    93                 currentModificationType = null;
    94                 break;
    9588            case "delete":
    9689                currentModificationType = null;
    9790                break;
     91            case "osmChange":
    9892            case "tag":
    9993            case "nd":

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/DNSName.java

    r7592 r7801  
    101101     * Return the type of the GeneralName.
    102102     */
     103    @Override
    103104    public int getType() {
    104105        return GeneralNameInterface.NAME_DNS;
    … …  
    119120     * @exception IOException on encoding errors.
    120121     */
     122    @Override
    121123    public void encode(DerOutputStream out) throws IOException {
    122124        out.putIA5String(name);

  • trunk/src/org/openstreetmap/josm/tools/Diff.java

    r7005 r7801  
    912912                            && equivs[start] == equivs[end]
    913913                                                       && !other_changed[1+j]
    914                                                                          && end != i_end
    915914                                                                         && !((preceding >= 0 && start == preceding)
    916915                                                                                 || (other_preceding >= 0
Note: See TracChangeset for help on using the changeset viewer.