Ignore:
Timestamp:
2014-04-19T00:36:43+02:00 (6 years ago)
Author:
Don-vip
Message:

sonar - place Strings literals on the left side when checking for equality (can avoid NPEs)

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
2 edited

Legend:

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

    r6920 r6990  
    8181            String[] table = url.split("\\?|=|&");
    8282            for (int i = 0; i<table.length; i++) {
    83                 if (table[i].equals("bbox") && i<table.length-1 )
     83                if ("bbox".equals(table[i]) && i<table.length-1 )
    8484                    return download(newLayer, new Bounds(table[i+1], ",", ParseMethod.LEFT_BOTTOM_RIGHT_TOP), progressMonitor);
    8585            }
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r6883 r6990  
    502502                String mv = null;
    503503
    504                 if (key.equals("timestamp")) {
     504                if ("timestamp".equals(key)) {
    505505                    mv = DateUtils.fromDate(osm.getTimestamp());
    506506                } else {
     
    603603            }
    604604
    605             if (regexp && key.length() > 0 && !key.equals("*")) {
     605            if (regexp && key.length() > 0 && !"*".equals(key)) {
    606606                try {
    607607                    keyPattern = Pattern.compile(key, regexFlags(false));
     
    614614                keyPattern = null;
    615615            }
    616             if (regexp && this.value.length() > 0 && !this.value.equals("*")) {
     616            if (regexp && this.value.length() > 0 && !"*".equals(this.value)) {
    617617                try {
    618618                    valuePattern = Pattern.compile(this.value, regexFlags(false));
     
    769769        private String user;
    770770        public UserMatch(String user) {
    771             if (user.equals("anonymous")) {
     771            if ("anonymous".equals(user)) {
    772772                this.user = null;
    773773            } else {
     
    13141314            value = "";
    13151315        }
    1316         if (key.equals("type"))
     1316        if ("type".equals(key))
    13171317            return new ExactType(value);
    1318         else if (key.equals("user"))
     1318        else if ("user".equals(key))
    13191319            return new UserMatch(value);
    1320         else if (key.equals("role"))
     1320        else if ("role".equals(key))
    13211321            return new RoleMatch(value);
    13221322        else
Note: See TracChangeset for help on using the changeset viewer.