Ignore:
Timestamp:
2016-12-12T02:28:14+01:00 (3 years ago)
Author:
Don-vip
Message:

findbugs - RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE

Location:
trunk/src/org/openstreetmap/josm/io
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r10638 r11381  
    393393        protected Date[] parseTime(String value) throws ChangesetQueryUrlException {
    394394            String[] dates = value.split(",");
    395             if (dates == null || dates.length == 0 || dates.length > 2)
     395            if (dates.length == 0 || dates.length > 2)
    396396                throw new ChangesetQueryUrlException(
    397397                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "time", value));
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r10627 r11381  
    827827        StringBuilder urlBuilder = noteStringBuilder(note)
    828828            .append("/close");
    829         if (encodedMessage != null && !encodedMessage.trim().isEmpty()) {
     829        if (!encodedMessage.trim().isEmpty()) {
    830830            urlBuilder.append("?text=");
    831831            urlBuilder.append(encodedMessage);
     
    849849        StringBuilder urlBuilder = noteStringBuilder(note)
    850850            .append("/reopen");
    851         if (encodedMessage != null && !encodedMessage.trim().isEmpty()) {
     851        if (!encodedMessage.trim().isEmpty()) {
    852852            urlBuilder.append("?text=");
    853853            urlBuilder.append(encodedMessage);
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java

    r10739 r11381  
    243243            if (child != null) {
    244244                String baseURL = child.getAttribute("xlink:href");
    245                 if (baseURL != null && !baseURL.equals(serviceUrlStr)) {
     245                if (!baseURL.equals(serviceUrlStr)) {
    246246                    Main.info("GetCapabilities specifies a different service URL: " + baseURL);
    247247                    serviceUrl = new URL(baseURL);
  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r11374 r11381  
    380380        String activeAtt = layersEl.getAttribute("active");
    381381        try {
    382             active = (activeAtt != null && !activeAtt.isEmpty()) ? Integer.parseInt(activeAtt)-1 : -1;
     382            active = !activeAtt.isEmpty() ? (Integer.parseInt(activeAtt)-1) : -1;
    383383        } catch (NumberFormatException e) {
    384384            Main.warn("Unsupported value for 'active' layer attribute. Ignoring it. Error was: "+e.getMessage());
     
    407407                    if (idx == null) {
    408408                        error(tr("unexpected format of attribute ''index'' for element ''layer''"));
    409                     }
    410                     if (elems.containsKey(idx)) {
     409                    } else if (elems.containsKey(idx)) {
    411410                        error(tr("attribute ''index'' ({0}) for element ''layer'' must be unique", Integer.toString(idx)));
    412411                    }
     
    415414                    deps.putVoid(idx);
    416415                    String depStr = e.getAttribute("depends");
    417                     if (depStr != null && !depStr.isEmpty()) {
     416                    if (!depStr.isEmpty()) {
    418417                        for (String sd : depStr.split(",")) {
    419418                            Integer d = null;
Note: See TracChangeset for help on using the changeset viewer.