Changeset 6087 in josm


Ignore:
Timestamp:
2013-07-26T17:28:24+02:00 (8 years ago)
Author:
Don-vip
Message:

see #8902 - string.equals("") => string.isEmpty() (patch by shinigami)

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

Legend:

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

    r5962 r6087  
    238238        @Override
    239239        protected void buttonAction(int buttonIndex, ActionEvent evt) {
    240             if (buttonIndex == 0 && tBookmarkName.getText() != null && !"".equals(tBookmarkName.getText()) &&
     240            if (buttonIndex == 0 && tBookmarkName.getText() != null && !tBookmarkName.getText().isEmpty() &&
    241241                    OffsetBookmark.getBookmarkByName(layer, tBookmarkName.getText()) != null) {
    242242                if (!confirmOverwriteBookmark()) return;
     
    252252            if (getValue() != 1) {
    253253                layer.setOffset(oldDx, oldDy);
    254             } else if (tBookmarkName.getText() != null && !"".equals(tBookmarkName.getText())) {
     254            } else if (tBookmarkName.getText() != null && !tBookmarkName.getText().isEmpty()) {
    255255                OffsetBookmark.bookmarkOffset(tBookmarkName.getText(), layer);
    256256            }
  • trunk/src/org/openstreetmap/josm/actions/Map_Rectifier_WMSmenuAction.java

    r6069 r6087  
    126126            // If the contents match, they will be inserted into the text field and the corresponding
    127127            // service will be pre-selected.
    128             if(!clip.equals("") && tfWmsUrl.getText().equals("")
     128            if(!clip.isEmpty() && tfWmsUrl.getText().isEmpty()
    129129                    && (s.urlRegEx.matcher(clip).find() || s.idValidator.matcher(clip).matches())) {
    130130                serviceBtn.setSelected(true);
     
    133133            s.btn = serviceBtn;
    134134            group.add(serviceBtn);
    135             if(!s.url.equals("")) {
     135            if(!s.url.isEmpty()) {
    136136                panel.add(serviceBtn, GBC.std());
    137137                panel.add(new UrlLabel(s.url, tr("Visit Homepage")), GBC.eol().anchor(GridBagConstraints.EAST));
     
    142142
    143143        // Fallback in case no match was found
    144         if(tfWmsUrl.getText().equals("") && firstBtn != null) {
     144        if(tfWmsUrl.getText().isEmpty() && firstBtn != null) {
    145145            firstBtn.setSelected(true);
    146146        }
     
    175175                // We've reached the custom WMS URL service
    176176                // Just set the URL and hope everything works out
    177                 if(s.wmsUrl.equals("")) {
     177                if(s.wmsUrl.isEmpty()) {
    178178                    addWMSLayer(s.name + " (" + text + ")", text);
    179179                    break outer;
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r5909 r6087  
    680680        @Override public boolean match(OsmPrimitive osm) {
    681681            if (!osm.hasKeys() && osm.getUser() == null)
    682                 return search.equals("");
     682                return search.isEmpty();
    683683
    684684            for (String key: osm.keySet()) {
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r6084 r6087  
    441441        putDefault(key, def);
    442442        final String prop = properties.get(key);
    443         if (prop == null || prop.equals(""))
     443        if (prop == null || prop.isEmpty())
    444444            return def;
    445445        return prop;
     
    774774        putDefault("color."+colKey, ColorHelper.color2html(def));
    775775        String colStr = specName != null ? get("color."+specName) : "";
    776         if(colStr.equals("")) {
     776        if(colStr.isEmpty()) {
    777777            colStr = get("color."+colKey);
    778778        }
    779         return colStr.equals("") ? def : ColorHelper.html2color(colStr);
     779        return colStr.isEmpty() ? def : ColorHelper.html2color(colStr);
    780780    }
    781781
    782782    synchronized public Color getDefaultColor(String colKey) {
    783783        String colStr = defaults.get("color."+colKey);
    784         return colStr == null || "".equals(colStr) ? null : ColorHelper.html2color(colStr);
     784        return colStr == null || colStr.isEmpty() ? null : ColorHelper.html2color(colStr);
    785785    }
    786786
  • trunk/src/org/openstreetmap/josm/data/Version.java

    r5956 r6087  
    7979        Pattern p = Pattern.compile("^([^:]+):(.*)$");
    8080        for (String line: content.split("\n")) {
    81             if (line == null || line.trim().equals("")) {
     81            if (line == null || line.trim().isEmpty()) {
    8282                continue;
    8383            }
  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java

    r5729 r6087  
    127127    // some additional checks to respect extended URLs in preferences (legacy workaround)
    128128    private boolean isSimilar(String a, String b) {
    129         return Utils.equal(a, b) || (a != null && b != null && !"".equals(a) && !"".equals(b) && (a.contains(b) || b.contains(a)));
     129        return Utils.equal(a, b) || (a != null && b != null && !a.isEmpty() && !b.isEmpty() && (a.contains(b) || b.contains(a)));
    130130    }
    131131
  • trunk/src/org/openstreetmap/josm/data/osm/TagCollection.java

    r6084 r6087  
    602602            throw new IllegalStateException(tr("Tag collection cannot be applied to a primitive because there are keys with multiple values."));
    603603        for (Tag tag: tags) {
    604             if (tag.getValue() == null || tag.getValue().equals("")) {
     604            if (tag.getValue() == null || tag.getValue().isEmpty()) {
    605605                primitive.remove(tag.getKey());
    606606            } else {
  • trunk/src/org/openstreetmap/josm/gui/ExtendedDialog.java

    r6070 r6087  
    573573     */
    574574    private boolean toggleCheckState(String togglePref) {
    575         toggleable = togglePref != null && !togglePref.equals("");
     575        toggleable = togglePref != null && !togglePref.isEmpty();
    576576
    577577        toggleValue = Main.pref.getInteger("message."+togglePref+".value", -1);
  • trunk/src/org/openstreetmap/josm/gui/JosmUserIdentityManager.java

    r6084 r6087  
    102102    public void setPartiallyIdentified(String userName) throws IllegalArgumentException {
    103103        CheckParameterUtil.ensureParameterNotNull(userName, "userName");
    104         if (userName.trim().equals(""))
     104        if (userName.trim().isEmpty())
    105105            throw new IllegalArgumentException(MessageFormat.format("Expected non-empty value for parameter ''{0}'', got ''{1}''", "userName", userName));
    106106        this.userName = userName;
     
    120120    public void setFullyIdentified(String username, UserInfo userinfo) throws IllegalArgumentException {
    121121        CheckParameterUtil.ensureParameterNotNull(username, "username");
    122         if (username.trim().equals(""))
     122        if (username.trim().isEmpty())
    123123            throw new IllegalArgumentException(tr("Expected non-empty value for parameter ''{0}'', got ''{1}''", "userName", userName));
    124124        CheckParameterUtil.ensureParameterNotNull(userinfo, "userinfo");
     
    192192        String userName = CredentialsManager.getInstance().getUsername();
    193193        if (isAnonymous()) {
    194             if (userName != null && ! userName.trim().equals("")) {
     194            if (userName != null && !userName.trim().isEmpty()) {
    195195                setPartiallyIdentified(userName);
    196196            }
     
    256256            if (!(evt.getNewValue() instanceof StringSetting)) return;
    257257            String newValue = ((StringSetting) evt.getNewValue()).getValue();
    258             if (newValue == null || newValue.trim().equals("")) {
     258            if (newValue == null || newValue.trim().isEmpty()) {
    259259                setAnonymous();
    260260            } else if (isFullyIdentified()) {
  • trunk/src/org/openstreetmap/josm/gui/bbox/TileSelectionBBoxChooser.java

    r6084 r6087  
    604604            String value = getComponent().getText().trim();
    605605            try {
    606                 if (value.equals("")) {
     606                if (value.isEmpty()) {
    607607                    tileIndex = 0;
    608608                } else {
  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/PasteTagsConflictResolverDialog.java

    r6084 r6087  
    462462                case RELATION: msg = trn("{0} relation", "{0} relations", numPrimitives, numPrimitives); break;
    463463                }
    464                 text = text.equals("") ? msg : text + ", " + msg;
     464                text = text.isEmpty() ? msg : text + ", " + msg;
    465465            }
    466466            setText(text);
  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolver.java

    r6084 r6087  
    174174        if (!cbTagRelations.isSelected())
    175175            return null;
    176         if (tfKey.getText().trim().equals(""))
     176        if (tfKey.getText().trim().isEmpty())
    177177            return null;
    178         if (tfValue.getText().trim().equals(""))
     178        if (tfValue.getText().trim().isEmpty())
    179179            return null;
    180180        if (primitives == null || primitives.isEmpty())
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheTableCellRenderer.java

    r6084 r6087  
    4848    protected void renderUploadComment(Changeset cs) {
    4949        String comment = cs.get("comment");
    50         if (comment == null || comment.trim().equals("")) {
     50        if (comment == null || comment.trim().isEmpty()) {
    5151            setText(trc("changeset.upload-comment", "empty"));
    5252            setFont(UIManager.getFont("Table.font").deriveFont(Font.ITALIC));
     
    6969    protected void renderUser(Changeset cs) {
    7070        User user = cs.getUser();
    71         if (user == null || user.getName().trim().equals("")) {
     71        if (user == null || user.getName().trim().isEmpty()) {
    7272            setFont(UIManager.getFont("Table.font").deriveFont(Font.ITALIC));
    7373            setText(tr("anonymous"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/UrlBasedQueryPanel.java

    r6084 r6087  
    184184        protected void validate() {
    185185            String value = tfUrl.getText();
    186             if (value.trim().equals("")) {
     186            if (value.trim().isEmpty()) {
    187187                feedbackNone();
    188188                return;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r6085 r6087  
    10491049                            );
    10501050
    1051                     if (type != null && !type.equals("")) {
     1051                    if (type != null && !type.isEmpty()) {
    10521052                        uris.add(new URI(String.format("%s%sRelation:%s", base, lang, type)));
    10531053                        uris.add(new URI(String.format("%sRelation:%s", base, type)));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TagEditHelper.java

    r6084 r6087  
    340340            // is not Java 1.5
    341341            //value = java.text.Normalizer.normalize(value, java.text.Normalizer.Form.NFC);
    342             if (value.equals("")) {
     342            if (value.isEmpty()) {
    343343                value = null; // delete the key
    344344            }
    345345            String newkey = keys.getEditor().getItem().toString().trim();
    346346            //newkey = java.text.Normalizer.normalize(newkey, java.text.Normalizer.Form.NFC);
    347             if (newkey.equals("")) {
     347            if (newkey.isEmpty()) {
    348348                newkey = key;
    349349                value = null; // delete the key instead
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r6070 r6087  
    15731573
    15741574        protected boolean isEmptyRole() {
    1575             return tfRole.getText() == null || tfRole.getText().trim().equals("");
     1575            return tfRole.getText() == null || tfRole.getText().trim().isEmpty();
    15761576        }
    15771577
  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkSelection.java

    r6084 r6087  
    202202            );
    203203            b.setArea(currentArea);
    204             if (b.getName() != null && !b.getName().equals("")) {
     204            if (b.getName() != null && !b.getName().isEmpty()) {
    205205                ((DefaultListModel)bookmarks.getModel()).addElement(b);
    206206                bookmarks.save();
  • trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java

    r5886 r6087  
    230230            tfUserName.setText(username);
    231231            tfPassword.setText(password);
    232             cbSaveCredentials.setSelected(!username.equals("") && ! password.equals(""));
     232            cbSaveCredentials.setSelected(!username.isEmpty() && !password.isEmpty());
    233233        }
    234234
  • trunk/src/org/openstreetmap/josm/gui/io/FilenameCellEditor.java

    r6084 r6087  
    115115    @Override
    116116    public boolean stopCellEditing() {
    117         if (tfFileName.getText() == null || tfFileName.getText().trim().equals("")) {
     117        if (tfFileName.getText() == null || tfFileName.getText().trim().isEmpty()) {
    118118            value = null;
    119119        } else {
  • trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java

    r6084 r6087  
    226226    @Override
    227227    public boolean stopCellEditing() {
    228         if (tfFilename.getText() == null || tfFilename.getText().trim().equals("")) {
     228        if (tfFilename.getText() == null || tfFilename.getText().trim().isEmpty()) {
    229229            value = null;
    230230        } else {
  • trunk/src/org/openstreetmap/josm/gui/io/TagSettingsPanel.java

    r6084 r6087  
    5656            return;
    5757
    58         if (comment.equals("")) {
     58        if (comment.isEmpty()) {
    5959            pnlTagEditor.getModel().delete("comment");
    6060            return;
  • trunk/src/org/openstreetmap/josm/gui/oauth/ManualAuthorizationUI.java

    r6084 r6087  
    183183        @Override
    184184        public boolean isValid() {
    185             return ! getComponent().getText().trim().equals("");
     185            return !getComponent().getText().trim().isEmpty();
    186186        }
    187187
     
    203203        @Override
    204204        public boolean isValid() {
    205             return ! getComponent().getText().trim().equals("");
     205            return !getComponent().getText().trim().isEmpty();
    206206        }
    207207
  • trunk/src/org/openstreetmap/josm/gui/preferences/PluginPreference.java

    r6084 r6087  
    441441        public void filter() {
    442442            String expr = tfFilter.getText().trim();
    443             if (expr.equals("")) {
     443            if (expr.isEmpty()) {
    444444                expr = null;
    445445            }
  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r6084 r6087  
    10831083                        @Override
    10841084                        public int compare(String o1, String o2) {
    1085                             if (o1.equals("") && o2.equals(""))
     1085                            if (o1.isEmpty() && o2.isEmpty())
    10861086                                return 0;
    1087                             if (o1.equals("")) return 1;
    1088                             if (o2.equals("")) return -1;
     1087                            if (o1.isEmpty()) return 1;
     1088                            if (o2.isEmpty()) return -1;
    10891089                            return o1.compareTo(o2);
    10901090                        }
     
    12411241
    12421242                while ((line = reader.readLine()) != null && !canceled) {
    1243                     if (line.trim().equals("")) {
     1243                    if (line.trim().isEmpty()) {
    12441244                        continue; // skip empty lines
    12451245                    }
  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginListPanel.java

    r6084 r6087  
    4545    protected String formatPluginRemoteVersion(PluginInformation pi) {
    4646        StringBuilder sb = new StringBuilder();
    47         if (pi.version == null || pi.version.trim().equals("")) {
     47        if (pi.version == null || pi.version.trim().isEmpty()) {
    4848            sb.append(tr("unknown"));
    4949        } else {
     
    5858    protected String formatPluginLocalVersion(PluginInformation pi) {
    5959        if (pi == null) return tr("unknown");
    60         if (pi.localversion == null || pi.localversion.trim().equals(""))
     60        if (pi.localversion == null || pi.localversion.trim().isEmpty())
    6161            return tr("unknown");
    6262        return pi.localversion;
  • trunk/src/org/openstreetmap/josm/gui/preferences/server/OsmApiUrlInputPanel.java

    r6084 r6087  
    198198        protected void updateEnabledState() {
    199199            boolean enabled =
    200                 !tfOsmServerUrl.getText().trim().equals("")
     200                !tfOsmServerUrl.getText().trim().isEmpty()
    201201                && !tfOsmServerUrl.getText().trim().equals(lastTestedUrl);
    202202            if (enabled) {
     
    236236        @Override
    237237        public boolean isValid() {
    238             if (getComponent().getText().trim().equals(""))
     238            if (getComponent().getText().trim().isEmpty())
    239239                return false;
    240240
     
    249249        @Override
    250250        public void validate() {
    251             if (getComponent().getText().trim().equals("")) {
     251            if (getComponent().getText().trim().isEmpty()) {
    252252                feedbackInvalid(tr("OSM API URL must not be empty. Please enter the OSM API URL."));
    253253                return;
  • trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

    r6064 r6087  
    149149        case 1:
    150150            String v = (String)value;
    151             if (tag.getValueCount() > 1 && ! v.equals("")) {
     151            if (tag.getValueCount() > 1 && !v.isEmpty()) {
    152152                updateTagValue(tag, v);
    153153            } else if (tag.getValueCount() <= 1) {
     
    430430            // tag name holds an empty key. Don't apply it to the selection.
    431431            //
    432             if (tag.getName().trim().equals("") || tag.getValue().trim().equals("")) {
     432            if (tag.getName().trim().isEmpty() || tag.getValue().trim().isEmpty()) {
    433433                continue;
    434434            }
     
    476476        // tag name holds an empty key. Don't apply it to the selection.
    477477        //
    478         if (tag.getName().trim().equals(""))
     478        if (tag.getName().trim().isEmpty())
    479479            return null;
    480480
     
    517517        ArrayList<String> keys = new ArrayList<String>();
    518518        for (TagModel tag: tags) {
    519             if (!tag.getName().trim().equals("")) {
     519            if (!tag.getName().trim().isEmpty()) {
    520520                keys.add(tag.getName());
    521521            }
  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetItems.java

    r6074 r6087  
    9797                return "<b>"+DIFFERENT.replaceAll("<", "&lt;").replaceAll(">", "&gt;")+"</b>";
    9898
    99             if (value.equals(""))
     99            if (value.isEmpty())
    100100                return "&nbsp;";
    101101
     
    117117
    118118        private Integer parseInteger(String str) {
    119             if (str == null || "".equals(str))
     119            if (str == null || str.isEmpty())
    120120                return null;
    121121            try {
  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java

    r6084 r6087  
    150150                    @Override
    151151                    public void keyReleased(KeyEvent e) {
    152                         if (getText().equals("")) {
     152                        if (getText().isEmpty()) {
    153153                            applyFilter("");
    154154                        }
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r5366 r6087  
    302302    public static class ChangesetQueryUrlParser {
    303303        protected int parseUid(String value) throws ChangesetQueryUrlException {
    304             if (value == null || value.trim().equals(""))
     304            if (value == null || value.trim().isEmpty())
    305305                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid",value));
    306306            int id;
     
    316316
    317317        protected boolean parseOpen(String value) throws ChangesetQueryUrlException {
    318             if (value == null || value.trim().equals(""))
     318            if (value == null || value.trim().isEmpty())
    319319                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "open",value));
    320320            if (value.equals("true"))
     
    327327
    328328        protected boolean parseBoolean(String value, String parameter) throws ChangesetQueryUrlException {
    329             if (value == null || value.trim().equals(""))
     329            if (value == null || value.trim().isEmpty())
    330330                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter,value));
    331331            if (value.equals("true"))
     
    338338
    339339        protected Date parseDate(String value, String parameter) throws ChangesetQueryUrlException {
    340             if (value == null || value.trim().equals(""))
     340            if (value == null || value.trim().isEmpty())
    341341                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter,value));
    342342            if (value.endsWith("Z")) {
     
    436436                return new ChangesetQuery();
    437437            query = query.trim();
    438             if (query.equals(""))
     438            if (query.isEmpty())
    439439                return new ChangesetQuery();
    440440            Map<String,String> queryParams  = createMapFromQueryString(query);
  • trunk/src/org/openstreetmap/josm/io/DefaultProxySelector.java

    r5266 r6087  
    108108        String host = Main.pref.get(ProxyPreferencesPanel.PROXY_HTTP_HOST, null);
    109109        int port = parseProxyPortValue(ProxyPreferencesPanel.PROXY_HTTP_PORT, Main.pref.get(ProxyPreferencesPanel.PROXY_HTTP_PORT, null));
    110         if (host != null && ! host.trim().equals("") && port > 0) {
     110        if (host != null && !host.trim().isEmpty() && port > 0) {
    111111            httpProxySocketAddress = new InetSocketAddress(host,port);
    112112        } else {
     
    120120        host = Main.pref.get(ProxyPreferencesPanel.PROXY_SOCKS_HOST, null);
    121121        port = parseProxyPortValue(ProxyPreferencesPanel.PROXY_SOCKS_PORT, Main.pref.get(ProxyPreferencesPanel.PROXY_SOCKS_PORT, null));
    122         if (host != null && ! host.trim().equals("") && port > 0) {
     122        if (host != null && !host.trim().isEmpty() && port > 0) {
    123123            socksProxySocketAddress = new InetSocketAddress(host,port);
    124124        } else {
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r5874 r6087  
    235235    private boolean ParseNMEASentence(String s, NMEAParserState ps) {
    236236        try {
    237             if (s.equals(""))
     237            if (s.isEmpty())
    238238                throw new NullPointerException();
    239239
     
    304304                    // Ignore heights that are not in meters for now
    305305                    accu=e[GPGGA.HEIGHT.position];
    306                     if(!accu.equals("")) {
     306                    if(!accu.isEmpty()) {
    307307                        Double.parseDouble(accu);
    308308                        // if it throws it's malformed; this should only happen if the
    309309                        // device sends nonstandard data.
    310                         if(!accu.equals("")) {
     310                        if(!accu.isEmpty()) { // FIX ? same check
    311311                            currentwp.attr.put("ele", accu);
    312312                        }
     
    316316                accu=e[GPGGA.SATELLITE_COUNT.position];
    317317                int sat = 0;
    318                 if(!accu.equals("")) {
     318                if(!accu.isEmpty()) {
    319319                    sat = Integer.parseInt(accu);
    320320                    currentwp.attr.put("sat", accu);
     
    322322                // h-dilution
    323323                accu=e[GPGGA.HDOP.position];
    324                 if(!accu.equals("")) {
     324                if(!accu.isEmpty()) {
    325325                    currentwp.attr.put("hdop", Float.parseFloat(accu));
    326326                }
    327327                // fix
    328328                accu=e[GPGGA.QUALITY.position];
    329                 if(!accu.equals("")) {
     329                if(!accu.isEmpty()) {
    330330                    int fixtype = Integer.parseInt(accu);
    331331                    switch(fixtype) {
     
    353353                    // other values than (T)rue are ignored
    354354                    accu = e[GPVTG.COURSE.position];
    355                     if(!accu.equals("")) {
     355                    if(!accu.isEmpty()) {
    356356                        Double.parseDouble(accu);
    357357                        currentwp.attr.put("course", accu);
     
    362362                if(accu.startsWith("K")) {
    363363                    accu = e[GPVTG.SPEED_KMH.position];
    364                     if(!accu.equals("")) {
     364                    if(!accu.isEmpty()) {
    365365                        double speed = Double.parseDouble(accu);
    366366                        speed /= 3.6; // speed in m/s
     
    371371                // vdop
    372372                accu=e[GPGSA.VDOP.position];
    373                 if(!accu.equals("")) {
     373                if(!accu.isEmpty()) {
    374374                    currentwp.attr.put("vdop", Float.parseFloat(accu));
    375375                }
    376376                // hdop
    377377                accu=e[GPGSA.HDOP.position];
    378                 if(!accu.equals("")) {
     378                if(!accu.isEmpty()) {
    379379                    currentwp.attr.put("hdop", Float.parseFloat(accu));
    380380                }
    381381                // pdop
    382382                accu=e[GPGSA.PDOP.position];
    383                 if(!accu.equals("")) {
     383                if(!accu.isEmpty()) {
    384384                    currentwp.attr.put("pdop", Float.parseFloat(accu));
    385385                }
     
    412412                // speed
    413413                accu = e[GPRMC.SPEED.position];
    414                 if(!accu.equals("") && !currentwp.attr.containsKey("speed")) {
     414                if(!accu.isEmpty() && !currentwp.attr.containsKey("speed")) {
    415415                    double speed = Double.parseDouble(accu);
    416416                    speed *= 0.514444444; // to m/s
     
    419419                // course
    420420                accu = e[GPRMC.COURSE.position];
    421                 if(!accu.equals("") && !currentwp.attr.containsKey("course")) {
     421                if(!accu.isEmpty() && !currentwp.attr.containsKey("course")) {
    422422                    Double.parseDouble(accu);
    423423                    currentwp.attr.put("course", accu);
     
    464464        // return a zero latlon instead of null so it is logged as zero coordinate
    465465        // instead of malformed sentence
    466         if(widthNorth.equals("")&&lengthEast.equals("")) return new LatLon(0.0,0.0);
     466        if(widthNorth.isEmpty() && lengthEast.isEmpty()) return new LatLon(0.0,0.0);
    467467
    468468        // The format is xxDDLL.LLLL
  • trunk/src/org/openstreetmap/josm/io/OsmApiException.java

    r6070 r6087  
    170170            sb.append(tr(errorHeader));
    171171            sb.append(tr("(Code={0})", responseCode));
    172         } else if (errorBody != null && !errorBody.trim().equals("")) {
     172        } else if (errorBody != null && !errorBody.trim().isEmpty()) {
    173173            errorBody = errorBody.trim();
    174174            sb.append(tr(errorBody));
  • trunk/src/org/openstreetmap/josm/io/auth/AbstractCredentialsAgent.java

    r6070 r6087  
    4545         * (noSuccessWithLastResponse == true).
    4646         */
    47         } else if (noSuccessWithLastResponse || username.equals("") || password.equals("")) {
     47        } else if (noSuccessWithLastResponse || username.isEmpty() || password.isEmpty()) {
    4848            GuiHelper.runInEDTAndWait(new Runnable() {
    4949                @Override
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSGrabber.java

    r5926 r6087  
    4747        this.info = layer.getInfo();
    4848        this.baseURL = info.getUrl();
    49         if(layer.getInfo().getCookies() != null && !layer.getInfo().getCookies().equals("")) {
     49        if(layer.getInfo().getCookies() != null && !layer.getInfo().getCookies().isEmpty()) {
    5050            props.put("Cookie", layer.getInfo().getCookies());
    5151        }
  • trunk/src/org/openstreetmap/josm/io/session/GpxTracksSessionImporter.java

    r5684 r6087  
    3333            XPathExpression fileExp = xpath.compile("file/text()");
    3434            String fileStr = (String) fileExp.evaluate(elem, XPathConstants.STRING);
    35             if (fileStr == null || fileStr.equals("")) {
     35            if (fileStr == null || fileStr.isEmpty()) {
    3636                throw new IllegalDataException(tr("File name expected for layer no. {0}", support.getLayerIndex()));
    3737            }
  • trunk/src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java

    r6070 r6087  
    3737            XPathExpression fileExp = xpath.compile("file/text()");
    3838            String fileStr = (String) fileExp.evaluate(elem, XPathConstants.STRING);
    39             if (fileStr == null || fileStr.equals("")) {
     39            if (fileStr == null || fileStr.isEmpty()) {
    4040                throw new IllegalDataException(tr("File name expected for layer no. {0}", support.getLayerIndex()));
    4141            }
  • trunk/src/org/openstreetmap/josm/io/session/OsmDataSessionImporter.java

    r4815 r6087  
    3434            XPathExpression fileExp = xpath.compile("file/text()");
    3535            String fileStr = (String) fileExp.evaluate(elem, XPathConstants.STRING);
    36             if (fileStr == null || fileStr.equals("")) {
     36            if (fileStr == null || fileStr.isEmpty()) {
    3737                throw new IllegalDataException(tr("File name expected for layer no. {0}", support.getLayerIndex()));
    3838            }
  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r6084 r6087  
    11831183            PluginInformation pi = pp.getPluginInformation();
    11841184            pl.remove(pi.name);
    1185             pl.add(pi.name + " (" + (pi.localversion != null && !pi.localversion.equals("")
     1185            pl.add(pi.name + " (" + (pi.localversion != null && !pi.localversion.isEmpty()
    11861186                    ? pi.localversion : "unknown") + ")");
    11871187        }
     
    11981198            final PluginInformation info = p.getPluginInformation();
    11991199            String name = info.name
    1200             + (info.version != null && !info.version.equals("") ? " Version: " + info.version : "");
     1200            + (info.version != null && !info.version.isEmpty() ? " Version: " + info.version : "");
    12011201            pluginTab.add(new JLabel(name), GBC.std());
    12021202            pluginTab.add(Box.createHorizontalGlue(), GBC.std().fill(GBC.HORIZONTAL));
  • trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java

    r6070 r6087  
    429429    public static String explainGeneric(Exception e) {
    430430        String msg = e.getMessage();
    431         if (msg == null || msg.trim().equals("")) {
     431        if (msg == null || msg.trim().isEmpty()) {
    432432            msg = e.toString();
    433433        }
  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r6084 r6087  
    403403            if (subdir == null) {
    404404                subdir = "";
    405             } else if (!subdir.equals("")) {
     405            } else if (!subdir.isEmpty()) {
    406406                subdir += "/";
    407407            }
  • trunk/src/org/openstreetmap/josm/tools/WindowGeometry.java

    r6070 r6087  
    163163    protected void initFromPreferences(String preferenceKey) throws WindowGeometryException {
    164164        String value = Main.pref.get(preferenceKey);
    165         if (value == null || value.equals(""))
     165        if (value == null || value.isEmpty())
    166166            throw new WindowGeometryException(tr("Preference with key ''{0}'' does not exist. Cannot restore window geometry from preferences.", preferenceKey));
    167167        topLeft = new Point();
Note: See TracChangeset for help on using the changeset viewer.