Changeset 16643 in josm


Ignore:
Timestamp:
2020-06-14T20:19:59+02:00 (4 years ago)
Author:
simon04
Message:

see #19334 - ​https://errorprone.info/bugpattern/StringSplitter

Location:
trunk
Files:
72 edited

Legend:

Unmodified
Added
Removed

  • trunk/build.xml

    r16632 r16643  
    302302            <!-- Undocumented argument to ignore "Sun internal proprietary API" warning, see http://stackoverflow.com/a/13862308/2257172 -->
    303303            <compilerarg value="-XDignore.symbol.file"/>
    304             <compilerarg value="-Xplugin:ErrorProne -XepExcludedPaths:.*/parsergen/.* -Xep:ReferenceEquality:OFF -Xep:FutureReturnValueIgnored:OFF -Xep:StringSplitter:OFF -Xep:JdkObsolete:OFF -Xep:EqualsGetClass:OFF -Xep:UndefinedEquals:OFF -Xep:BadImport:OFF -Xep:AnnotateFormatMethod:OFF"/>
     304            <compilerarg value="-Xplugin:ErrorProne -XepExcludedPaths:.*/parsergen/.* -Xep:ReferenceEquality:OFF -Xep:FutureReturnValueIgnored:OFF -Xep:JdkObsolete:OFF -Xep:EqualsGetClass:OFF -Xep:UndefinedEquals:OFF -Xep:BadImport:OFF -Xep:AnnotateFormatMethod:OFF"/>
    305305            <compilerarg line="-Xmaxwarns 1000"/>
    306306            <classpath>

  • trunk/src/com/kitfox/svg/Text.java

    r14346 r16643  
    268268    {
    269269        //Get font
    270         String[] families = fontFamily.split(",");
     270        String[] families = fontFamily.split(",", -1);
    271271        Font font = null;
    272272        for (int i = 0; i < families.length; ++i)

  • trunk/src/com/kitfox/svg/util/FontSystem.java

    r14331 r16643  
    7878    public static FontSystem createFont(String fontFamily, int fontStyle, int fontWeight, int fontSize)
    7979    {
    80         String[] families = fontFamily.split(",");
     80        String[] families = fontFamily.split(",", -1);
    8181        for (String fontName: families)
    8282        {

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

    r16452 r16643  
    369369        final Collection<String> extensionsPlusArchive = new LinkedHashSet<>();
    370370        final Collection<String> extensionsForDescription = new LinkedHashSet<>();
    371         for (String e : extensions.split(",")) {
     371        for (String e : extensions.split(",", -1)) {
    372372            extensionsPlusArchive.add(e);
    373373            if (addArchiveExtension != AddArchiveExtension.NONE) {
    … …  
    420420     */
    421421    public boolean acceptName(String filename) {
    422         return Utils.hasExtension(filename, extensions.split(","));
     422        return Utils.hasExtension(filename, extensions.split(",", -1));
    423423    }
    424424

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

    r16505 r16643  
    121121        final List<String> cmd = new ArrayList<>();
    122122        cmd.add("/usr/bin/osascript");
    123         for (String line : RESTART_APPLE_SCRIPT.split("\n")) {
     123        for (String line : RESTART_APPLE_SCRIPT.split("\n", -1)) {
    124124            cmd.add("-e");
    125125            cmd.add(line);
    … …  
    142142            throw new IOException("Unable to retrieve sun.java.command property");
    143143        }
    144         String[] mainCommand = javaCommand.split(" ");
     144        String[] mainCommand = javaCommand.split(" ", -1);
    145145        if (javaCommand.endsWith(".jnlp") && jnlp == null) {
    146146            // see #11751 - jnlp on Linux

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

    r16438 r16643  
    8686
    8787        } else if (GpxUrlPattern.TRACKPOINTS_BBOX.matches(url)) {
    88             String[] table = url.split("\\?|=|&");
     88            String[] table = url.split("\\?|=|&", -1);
    8989            for (int i = 0; i < table.length; i++) {
    9090                if ("bbox".equals(table[i]) && i < table.length-1)

  • trunk/src/org/openstreetmap/josm/data/Bounds.java

    r16553 r16643  
    251251    public Bounds(String asString, String separator, ParseMethod parseMethod, boolean roundToOsmPrecision) {
    252252        CheckParameterUtil.ensureParameterNotNull(asString, "asString");
    253         String[] components = asString.split(separator);
     253        String[] components = asString.split(separator, -1);
    254254        if (components.length != 4)
    255255            throw new IllegalArgumentException(

  • trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

    r16553 r16643  
    451451        String cacheControl = urlConn.getHeaderField("Cache-Control");
    452452        if (cacheControl != null) {
    453             for (String token: cacheControl.split(",")) {
     453            for (String token: cacheControl.split(",", -1)) {
    454454                try {
    455455                    if (token.startsWith("max-age=")) {

  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r16605 r16643  
    400400            if (e.shapes != null) {
    401401                try {
    402                     for (String s : e.shapes.split(";")) {
     402                    for (String s : e.shapes.split(";", -1)) {
    403403                        bounds.addShape(new Shape(s, ","));
    404404                    }
    … …  
    410410        if (e.projections != null && !e.projections.isEmpty()) {
    411411            // split generates null element on empty string which gives one element Array[null]
    412             setServerProjections(Arrays.asList(e.projections.split(",")));
     412            setServerProjections(Arrays.asList(e.projections.split(",", -1)));
    413413        }
    414414        attributionText = Utils.intern(e.attribution_text);
    … …  
    666666            Matcher m = Pattern.compile(".*\\{PROJ\\(([^)}]+)\\)\\}.*").matcher(url.toUpperCase(Locale.ENGLISH));
    667667            if (m.matches()) {
    668                 setServerProjections(Arrays.asList(m.group(1).split(",")));
     668                setServerProjections(Arrays.asList(m.group(1).split(",", -1)));
    669669            }
    670670        }

  • trunk/src/org/openstreetmap/josm/data/imagery/Shape.java

    r16436 r16643  
    2929    public Shape(String asString, String separator) {
    3030        CheckParameterUtil.ensureParameterNotNull(asString, "asString");
    31         String[] components = asString.split(separator);
     31        String[] components = asString.split(separator, -1);
    3232        if (components.length % 2 != 0)
    3333            throw new IllegalArgumentException(MessageFormat.format("Even number of doubles expected in string, got {0}: {1}",

  • trunk/src/org/openstreetmap/josm/data/imagery/WMTSTileSource.java

    r16553 r16643  
    746746
    747747    private static <T> T parseCoor(String coor, boolean switchXY, BiFunction<String, String, T> function) {
    748         String[] parts = coor.split(" ");
     748        String[] parts = coor.split(" ", -1);
    749749        if (switchXY) {
    750750            return function.apply(parts[1], parts[0]);

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

    r15735 r16643  
    698698        Map<String, Collection<String>> originalSplitValues = new LinkedHashMap<>();
    699699        for (String v : originalValues) {
    700             List<String> vs = Arrays.asList(SPLIT_VALUES_PATTERN.split(v));
     700            List<String> vs = Arrays.asList(SPLIT_VALUES_PATTERN.split(v, -1));
    701701            originalSplitValues.put(v, vs);
    702702            values.addAll(vs);

  • trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java

    r16581 r16643  
    190190                        // add leading/trailing space in order to get expected split (e.g. "a--" => {"a", ""})
    191191                        String rangeS = ' ' + tokenizer.readTextOrNumber() + ' ';
    192                         String[] rangeA = rangeS.split("/");
     192                        String[] rangeA = rangeS.split("/", -1);
    193193                        if (rangeA.length == 1) {
    194194                            return new KeyValue(keyword, rangeS.trim(), regexSearch, caseSensitive);

  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r16626 r16643  
    358358
    359359        Pattern keyPattern = Pattern.compile("\\+(?<key>[a-zA-Z0-9_]+)(=(?<value>.*))?");
    360         String[] parts = Utils.WHITE_SPACES_PATTERN.split(trimmedPref);
     360        String[] parts = Utils.WHITE_SPACES_PATTERN.split(trimmedPref, -1);
    361361        for (String part : parts) {
    362362            Matcher m = keyPattern.matcher(part);
    … …  
    511511     */
    512512    public Datum parseToWGS84(String paramList, Ellipsoid ellps) throws ProjectionConfigurationException {
    513         String[] numStr = paramList.split(",");
     513        String[] numStr = paramList.split(",", -1);
    514514
    515515        if (numStr.length != 3 && numStr.length != 7)
    … …  
    621621     */
    622622    public static Bounds parseBounds(String boundsStr) throws ProjectionConfigurationException {
    623         String[] numStr = boundsStr.split(",");
     623        String[] numStr = boundsStr.split(",", -1);
    624624        if (numStr.length != 4)
    625625            throw new ProjectionConfigurationException(tr("Unexpected number of arguments for parameter ''+bounds'' (must be 4)"));

  • trunk/src/org/openstreetmap/josm/data/projection/ProjectionCLI.java

    r15716 r16643  
    166166
    167167    private EastNorth parseEastNorth(String s, ToDoubleFunction<String> parser) {
    168         String[] en = s.split("[;, ]+");
     168        String[] en = s.split("[;, ]+", -1);
    169169        if (en.length != 2)
    170170            throw new IllegalArgumentException(tr("Expected two coordinates, separated by white space, found {0} in ''{1}''", en.length, s));

  • trunk/src/org/openstreetmap/josm/data/validation/OsmValidator.java

    r16447 r16643  
    371371
    372372            ArrayList<String> ignoredElementList = new ArrayList<>();
    373             String[] osmobjects = elemId1Pattern.split(key);
     373            String[] osmobjects = elemId1Pattern.split(key, -1);
    374374            for (int i = 1; i < osmobjects.length; i++) {
    375375                String osmid = osmobjects[i];

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

    r13206 r16643  
    137137            return false;
    138138        }
    139         String[] octets = inet6Address.split(":");
     139        String[] octets = inet6Address.split(":", -1);
    140140        if (containsCompressedZeroes) {
    141141            List<String> octetList = new ArrayList<>(Arrays.asList(octets));

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

    r16628 r16643  
    268268     */
    269269    static List<String> expandHouseNumber(String houseNumber) {
    270         return Arrays.asList(houseNumber.split(",|;"));
     270        return Arrays.asList(houseNumber.split(",|;", -1));
    271271    }
    272272

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

    r16154 r16643  
    106106            return false;
    107107        }
    108         final String[] parts = key.replace(":conditional", "").split(":");
     108        final String[] parts = key.replace(":conditional", "").split(":", -1);
    109109        return isKeyValid3Parts(parts) || isKeyValid1Part(parts) || isKeyValid2Parts(parts);
    110110    }
    … …  
    178178                while (i + 1 <= m.groupCount() && m.group(i + 1) != null) {
    179179                    final String restrictionValue = m.group(i);
    180                     final String[] conditions = m.group(i + 1).replace("(", "").replace(")", "").split("\\s+(AND|and)\\s+");
     180                    final String[] conditions = m.group(i + 1).replace("(", "").replace(")", "").split("\\s+(AND|and)\\s+", -1);
    181181                    r.add(new ConditionalValue(restrictionValue, Arrays.asList(conditions)));
    182182                    i += 3;
    … …  
    197197            for (final ConditionalValue conditional : ConditionalValue.parse(value)) {
    198198                // validate restriction value
    199                 if (isTransportationMode(key.split(":")[0]) && !isRestrictionValue(conditional.restrictionValue)) {
     199                if (isTransportationMode(key.split(":", -1)[0]) && !isRestrictionValue(conditional.restrictionValue)) {
    200200                    return tr("{0} is not a valid restriction value", conditional.restrictionValue);
    201201                }

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

    r16630 r16643  
    9393            }
    9494            Map<Integer, Boolean> connections = new HashMap<>();
    95             String[] toLanes = TO_LANE_PATTERN.split(lane[1]);
     95            String[] toLanes = TO_LANE_PATTERN.split(lane[1], -1);
    9696            for (int j = 0; j < toLanes.length; j++) {
    9797                String toLane = toLanes[j].trim();

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

    r16337 r16643  
    9696        List<TestError> errors = new ArrayList<>();
    9797        String values = v != null ? v : p.get(k);
    98         for (String value : values.split(";")) {
     98        for (String value : values.split(";", -1)) {
    9999            if (!validator.isValid(value)) {
    100100                Supplier<Command> fix = null;

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

    r16172 r16643  
    139139                .map(e -> ((LiteralExpression) e).getLiteral())
    140140                .filter(l -> l instanceof String)
    141                 .map(l -> ((String) l).split(",")[0])
     141                .map(l -> ((String) l).split(",", -1)[0])
    142142                .findFirst();
    143143    }

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

    r16198 r16643  
    102102        Check if this is the case. */
    103103
    104         String[] splitNames = NAME_SPLIT_PATTERN.split(name);
     104        String[] splitNames = NAME_SPLIT_PATTERN.split(name, -1);
    105105        if (splitNames.length == 1) {
    106106            /* The name is not composed of multiple parts. Complain. */

  • trunk/src/org/openstreetmap/josm/gui/MainApplication.java

    r16623 r16643  
    11501150    private static void processOffline(ProgramArguments args) {
    11511151        for (String offlineNames : args.get(Option.OFFLINE)) {
    1152             for (String s : offlineNames.split(",")) {
     1152            for (String s : offlineNames.split(",", -1)) {
    11531153                try {
    11541154                    NetworkManager.setOffline(OnlineResource.valueOf(s.toUpperCase(Locale.ENGLISH)));

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

    r16438 r16643  
    298298            Set<String> results = instantiateSortedSet();
    299299            for (String value: values) {
    300                 for (String part: value.split(Pattern.quote(separator))) {
     300                for (String part: value.split(Pattern.quote(separator), -1)) {
    301301                    results.add(part);
    302302                }

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

    r15716 r16643  
    362362     */
    363363    public static EastNorth parseEastNorth(String s) {
    364         String[] en = s.split("[;, ]+");
     364        String[] en = s.split("[;, ]+", -1);
    365365        if (en.length != 2) return null;
    366366        try {

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

    r13994 r16643  
    4646     */
    4747    public List<String> findHashTags() {
    48         return Arrays.stream(comment.split("\\s"))
     48        return Arrays.stream(comment.split("\\s", -1))
    4949                .map(s -> Utils.strip(s, ",;"))
    5050                .filter(s -> s.matches("#[a-zA-Z][a-zA-Z_\\-0-9]+"))

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

    r16438 r16643  
    170170        if (!opts.isEmpty()) {
    171171            return JOptionPane.showOptionDialog(MainApplication.getMainFrame(), text, "Question",
    172                     JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.QUESTION_MESSAGE, null, opts.split(";"), 0);
     172                    JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.QUESTION_MESSAGE, null, opts.split(";", -1), 0);
    173173        } else {
    174174            return JOptionPane.showOptionDialog(MainApplication.getMainFrame(), text, "Question",
    … …  
    298298        final List<String> removeList = new ArrayList<>();
    299299        final List<String> deleteList = new ArrayList<>();
    300         Collections.addAll(installList, install.toLowerCase(Locale.ENGLISH).split(";"));
    301         Collections.addAll(removeList, uninstall.toLowerCase(Locale.ENGLISH).split(";"));
    302         Collections.addAll(deleteList, delete.toLowerCase(Locale.ENGLISH).split(";"));
     300        Collections.addAll(installList, install.toLowerCase(Locale.ENGLISH).split(";", -1));
     301        Collections.addAll(removeList, uninstall.toLowerCase(Locale.ENGLISH).split(";", -1));
     302        Collections.addAll(deleteList, delete.toLowerCase(Locale.ENGLISH).split(";", -1));
    303303        installList.remove("");
    304304        removeList.remove("");

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

    r16628 r16643  
    11281128            StringBuilder line = new StringBuilder();
    11291129            StringBuilder ret = new StringBuilder();
    1130             for (String s: text.split(" ")) {
     1130            for (String s: text.split(" ", -1)) {
    11311131                if (g.getFontMetrics().stringWidth(line.toString() + s) > tileSource.getTileSize()) {
    11321132                    ret.append(line).append('\n');
    … …  
    11391139        }
    11401140        int offset = 0;
    1141         for (String s: textToDraw.split("\n")) {
     1141        for (String s: textToDraw.split("\n", -1)) {
    11421142            g.setColor(Color.black);
    11431143            g.drawString(s, x + 1, y + offset + 1);
    … …  
    16321632            if (tileLoader instanceof TMSCachedTileLoader) {
    16331633                int offset = 200;
    1634                 for (String part: ((TMSCachedTileLoader) tileLoader).getStats().split("\n")) {
     1634                for (String part: ((TMSCachedTileLoader) tileLoader).getStats().split("\n", -1)) {
    16351635                    offset += 15;
    16361636                    myDrawString(g, tr("Cache stats: {0}", part), 50, offset);

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

    r16548 r16643  
    392392                            String jvmId = reader.readLine();
    393393                            if (jvmId != null) {
    394                                 String pid = jvmId.split("@")[0];
     394                                String pid = jvmId.split("@", -1)[0];
    395395                                skipFile = jvmPerfDataFileExists(pid);
    396396                            }

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

    r16553 r16643  
    797797                            .orElse(k);
    798798                    if (k.startsWith("extension")) {
    799                         String[] chain = k.split(":");
     799                        String[] chain = k.split(":", -1);
    800800                        if (chain.length >= 3 && "segment".equals(chain[2])) {
    801801                            segExts.addFlat(chain, v);

  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelper.java

    r16069 r16643  
    10171017
    10181018                // use comma as separator
    1019                 String[] column = line.split(",");
     1019                String[] column = line.split(",", -1);
    10201020
    10211021                // empty or comment line

  • trunk/src/org/openstreetmap/josm/gui/mappaint/RenderingCLI.java

    r16321 r16643  
    286286            break;
    287287        case ANCHOR:
    288             String[] parts = arg.split(",");
     288            String[] parts = arg.split(",", -1);
    289289            if (parts.length != 2)
    290290                throw new OptionParseException(

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

    r16628 r16643  
    11591159     */
    11601160    public static boolean inside(Environment env, String codes) { // NO_UCD (unused code)
    1161         return Arrays.stream(codes.toUpperCase(Locale.ENGLISH).split(","))
     1161        return Arrays.stream(codes.toUpperCase(Locale.ENGLISH).split(",", -1))
    11621162                .anyMatch(code -> Territories.isIso3166Code(code.trim(), center(env)));
    11631163    }

  • trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

    r13207 r16643  
    198198
    199199        for (String setCookie: setCookies) {
    200             String[] kvPairs = setCookie.split(";");
     200            String[] kvPairs = setCookie.split(";", -1);
    201201            if (kvPairs.length == 0) {
    202202                continue;
    … …  
    204204            for (String kvPair : kvPairs) {
    205205                kvPair = kvPair.trim();
    206                 String[] kv = kvPair.split("=");
     206                String[] kv = kvPair.split("=", -1);
    207207                if (kv.length != 2) {
    208208                    continue;

  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreference.java

    r16438 r16643  
    511511
    512512        private void activatePlugins(JTextArea textField, boolean deleteNotInList) {
    513             String[] lines = textField.getText().split("\n");
     513            String[] lines = textField.getText().split("\n", -1);
    514514            List<String> toActivate = new ArrayList<>();
    515515            List<String> notFound = new ArrayList<>();

  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItem.java

    r16179 r16643  
    8787            return TYPE_CACHE.get(types);
    8888        Set<TaggingPresetType> result = EnumSet.noneOf(TaggingPresetType.class);
    89         for (String type : Arrays.asList(types.split(","))) {
     89        for (String type : Arrays.asList(types.split(",", -1))) {
    9090            try {
    9191                TaggingPresetType presetType = TaggingPresetType.fromString(type);

  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetSelector.java

    r16179 r16643  
    130130            String locName = preset.getLocaleName();
    131131            if (locName != null) {
    132                 Collections.addAll(collection, locName.toLowerCase(Locale.ENGLISH).split("\\s"));
     132                Collections.addAll(collection, locName.toLowerCase(Locale.ENGLISH).split("\\s", -1));
    133133            }
    134134        }
    … …  
    282282
    283283            if (searchText.contains("/")) {
    284                 groupWords = searchText.substring(0, searchText.lastIndexOf('/')).split("[\\s/]");
    285                 nameWords = searchText.substring(searchText.indexOf('/') + 1).split("\\s");
     284                groupWords = searchText.substring(0, searchText.lastIndexOf('/')).split("[\\s/]", -1);
     285                nameWords = searchText.substring(searchText.indexOf('/') + 1).split("\\s", -1);
    286286            } else {
    287287                groupWords = null;
    288                 nameWords = searchText.split("\\s");
     288                nameWords = searchText.split("\\s", -1);
    289289            }
    290290

  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/ComboMultiSelect.java

    r16544 r16643  
    174174            } else {
    175175                String s = o.toString();
    176                 Set<String> parts = new TreeSet<>(Arrays.asList(s.split(String.valueOf(delimiter))));
     176                Set<String> parts = new TreeSet<>(Arrays.asList(s.split(String.valueOf(delimiter), -1)));
    177177                ListModel<PresetListEntry> lm = getModel();
    178178                int[] intParts = new int[lm.getSize()];
    … …  
    440440
    441441        if (values_from != null) {
    442             String[] classMethod = values_from.split("#");
     442            String[] classMethod = values_from.split("#", -1);
    443443            if (classMethod.length == 2) {
    444444                try {

  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java

    r16630 r16643  
    6767        AutoCompletingTextField textField = new AutoCompletingTextField();
    6868        if (alternative_autocomplete_keys != null) {
    69             initAutoCompletionField(textField, (key + ',' + alternative_autocomplete_keys).split(","));
     69            initAutoCompletionField(textField, (key + ',' + alternative_autocomplete_keys).split(",", -1));
    7070        } else {
    7171            initAutoCompletionField(textField, key);
    … …  
    126126
    127127            // first, one button for each auto_increment value
    128             for (final String ai : auto_increment.split(",")) {
     128            for (final String ai : auto_increment.split(",", -1)) {
    129129                JToggleButton aibutton = new JToggleButton(ai);
    130130                aibutton.setToolTipText(tr("Select auto-increment of {0} for this field", ai));

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

    r16438 r16643  
    8787                    expr = expr.replace("+", "\\+");
    8888                    // split search string on whitespace, do case-insensitive AND search
    89                     List<RowFilter<Object, Object>> andFilters = Arrays.stream(expr.split("\\s+"))
     89                    List<RowFilter<Object, Object>> andFilters = Arrays.stream(expr.split("\\s+", -1))
    9090                            .map(word -> RowFilter.regexFilter("(?i)" + word))
    9191                            .collect(Collectors.toList());

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

    r15716 r16643  
    480480
    481481        protected Date[] parseTime(String value) throws ChangesetQueryUrlException {
    482             String[] dates = value.split(",");
     482            String[] dates = value.split(",", -1);
    483483            if (dates.length == 0 || dates.length > 2)
    484484                throw new ChangesetQueryUrlException(
    … …  
    495495                return Collections.<Long>emptySet();
    496496            } else {
    497                 return Stream.of(value.split(",")).map(Long::valueOf).collect(Collectors.toSet());
     497                return Stream.of(value.split(",", -1)).map(Long::valueOf).collect(Collectors.toSet());
    498498            }
    499499        }
    … …  
    560560        protected Map<String, String> createMapFromQueryString(String query) {
    561561            Map<String, String> queryParams = new HashMap<>();
    562             String[] keyValuePairs = query.split("&");
     562            String[] keyValuePairs = query.split("&", -1);
    563563            for (String keyValuePair: keyValuePairs) {
    564                 String[] kv = keyValuePair.split("=");
     564                String[] kv = keyValuePair.split("=", -1);
    565565                queryParams.put(kv[0], kv.length > 1 ? kv[1] : "");
    566566            }

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

    r16038 r16643  
    147147                String schemaLocation = atts.getValue(GpxConstants.XML_URI_XSD, "schemaLocation");
    148148                if (schemaLocation != null) {
    149                     String[] schemaLocations = schemaLocation.split(" ");
     149                    String[] schemaLocations = schemaLocation.split(" ", -1);
    150150                    for (int i = 0; i < schemaLocations.length - 1; i += 2) {
    151151                        final String schemaURI = schemaLocations[i];

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

    r15736 r16643  
    197197            // write the email address
    198198            if (attr.containsKey(META_AUTHOR_EMAIL)) {
    199                 String[] tmp = data.getString(META_AUTHOR_EMAIL).split("@");
     199                String[] tmp = data.getString(META_AUTHOR_EMAIL).split("@", -1);
    200200                if (tmp.length == 2) {
    201201                    inline("email", "id=\"" + encode(tmp[0]) + "\" domain=\"" + encode(tmp[1]) +'\"');

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

    r16419 r16643  
    284284                    currentResult.lat = Double.parseDouble(atts.getValue("lat"));
    285285                    currentResult.lon = Double.parseDouble(atts.getValue("lon"));
    286                     String[] bbox = atts.getValue("boundingbox").split(",");
     286                    String[] bbox = atts.getValue("boundingbox").split(",", -1);
    287287                    currentResult.bounds = new Bounds(
    288288                            Double.parseDouble(bbox[0]), Double.parseDouble(bbox[2]),

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

    r13849 r16643  
    9090            try {
    9191                String[] token = new String(Base64.getDecoder().decode(auth.substring(BASIC_AUTH.length())),
    92                         StandardCharsets.UTF_8).split(":");
     92                        StandardCharsets.UTF_8).split(":", -1);
    9393                if (token.length == 2) {
    9494                    return token[0];

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

    r16629 r16643  
    336336            final String errorIndicator = "Error</strong>: ";
    337337            if (ex.getMessage() != null && ex.getMessage().contains(errorIndicator)) {
    338                 final String errorPlusRest = ex.getMessage().split(errorIndicator)[1];
     338                final String errorPlusRest = ex.getMessage().split(errorIndicator, -1)[1];
    339339                if (errorPlusRest != null) {
    340                     ex.setErrorHeader(errorPlusRest.split("</")[0].replaceAll(".*::request_read_and_idx::", ""));
     340                    ex.setErrorHeader(errorPlusRest.split("</", -1)[0].replaceAll(".*::request_read_and_idx::", ""));
    341341                }
    342342            }

  • trunk/src/org/openstreetmap/josm/io/nmea/NmeaReader.java

    r16436 r16643  
    313313            // if there is no * or other meanities it will throw
    314314            // and result in a malformed packet.
    315             String[] chkstrings = s.split("\\*");
     315            String[] chkstrings = s.split("\\*", -1);
    316316            if (chkstrings.length > 1) {
    317317                byte[] chb = chkstrings[0].getBytes(StandardCharsets.UTF_8);
    … …  
    329329            }
    330330            // now for the content
    331             String[] e = chkstrings[0].split(",");
     331            String[] e = chkstrings[0].split(",", -1);
    332332            String accu;
    333333

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/FeaturesHandler.java

    r16550 r16643  
    2424    protected void handleRequest() throws RequestHandlerErrorException, RequestHandlerBadRequestException {
    2525        String q = args.get("q");
    26         Collection<String> handlers = q == null ? null : Arrays.asList(q.split("[,\\s]+"));
     26        Collection<String> handlers = q == null ? null : Arrays.asList(q.split("[,\\s]+", -1));
    2727        content = RequestProcessor.getHandlersInfoAsJSON(handlers).toString();
    2828        contentType = "application/json";

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    r16328 r16643  
    330330        if (args != null && args.containsKey("select")) {
    331331            toSelect.clear();
    332             for (String item : args.get("select").split(",")) {
     332            for (String item : args.get("select").split(",", -1)) {
    333333                if (!item.isEmpty()) {
    334334                    if (CURRENT_SELECTION.equalsIgnoreCase(item)) {

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java

    r16576 r16643  
    244244
    245245    protected final String[] splitArg(String arg, Pattern splitter) {
    246         return splitter.split(args != null ? args.get(arg) : "");
     246        return splitter.split(args != null ? args.get(arg) : "", -1);
    247247    }
    248248
    … …  
    259259            return r;
    260260        }
    261         for (String kv : uri.getRawQuery().split("&")) {
     261        for (String kv : uri.getRawQuery().split("&", -1)) {
    262262            final String[] kvs = Utils.decodeUrl(kv).split("=", 2);
    263263            r.put(kvs[0], kvs.length > 1 ? kvs[1] : null);

  • trunk/src/org/openstreetmap/josm/io/rtklib/RtkLibPosReader.java

    r15496 r16643  
    8686                    } else if (!line.startsWith("%")) {
    8787                        try {
    88                             String[] fields = line.split("[ ]+");
     88                            String[] fields = line.split("[ ]+", -1);
    8989                            WayPoint currentwp = new WayPoint(new LatLon(
    9090                                    Double.parseDouble(fields[IDX_LAT]),

  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r16553 r16643  
    487487                    String depStr = e.getAttribute("depends");
    488488                    if (!depStr.isEmpty()) {
    489                         for (String sd : depStr.split(",")) {
     489                        for (String sd : depStr.split(",", -1)) {
    490490                            Integer d = null;
    491491                            try {

  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r16488 r16643  
    10031003            String systemProp = Utils.getSystemProperty("josm.plugins");
    10041004            if (systemProp != null) {
    1005                 plugins.addAll(Arrays.asList(systemProp.split(",")));
     1005                plugins.addAll(Arrays.asList(systemProp.split(",", -1)));
    10061006                Logging.debug("josm.plugins system property set to ''{0}''. Plugins list is now {1}", systemProp, plugins);
    10071007            }

  • trunk/src/org/openstreetmap/josm/plugins/PluginInformation.java

    r16182 r16643  
    307307        String classPath = attr.getValue(Attributes.Name.CLASS_PATH);
    308308        if (classPath != null) {
    309             for (String entry : classPath.split(" ")) {
     309            for (String entry : classPath.split(" ", -1)) {
    310310                File entryFile;
    311311                if (new File(entry).isAbsolute() || file == null) {
    … …  
    510510    public boolean matches(String filter) {
    511511        if (filter == null) return true;
    512         String[] words = filter.split("\\s+");
     512        String[] words = filter.split("\\s+", -1);
    513513        for (String word: words) {
    514514            if (matches(word, name)
    … …  
    556556        List<String> requiredPlugins = new ArrayList<>();
    557557        if (pluginList != null) {
    558             for (String s : pluginList.split(";")) {
     558            for (String s : pluginList.split(";", -1)) {
    559559                String plugin = s.trim();
    560560                if (!plugin.isEmpty()) {

  • trunk/src/org/openstreetmap/josm/plugins/PluginListParser.java

    r15588 r16643  
    7777                }
    7878                addPluginInformation(ret, name, url, manifest.toString());
    79                 String[] x = line.split(";");
     79                String[] x = line.split(";", -1);
    8080                if (x.length != 2)
    8181                    throw new IOException(tr("Illegal entry in plugin list.") + " " + line);

  • trunk/src/org/openstreetmap/josm/plugins/ReadRemotePluginInformationTask.java

    r15716 r16643  
    221221            panel.add(new JLabel(firstMessage), GBC.eol().insets(0, 0, 0, 10));
    222222            StringBuilder b = new StringBuilder();
    223             for (String part : msg.split("(?<=\\G.{200})")) {
     223            for (String part : msg.split("(?<=\\G.{200})", -1)) {
    224224                b.append(part).append('\n');
    225225            }

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

    r16630 r16643  
    107107        if (m.matches()) {
    108108            OsmPrimitive n = new Node(Long.parseLong(m.group(1)));
    109             for (String s : m.group(2).split(",")) {
     109            for (String s : m.group(2).split(",", -1)) {
    110110                refs.add(new Relation(Long.parseLong(s)));
    111111            }
    … …  
    115115        if (m.matches()) {
    116116            OsmPrimitive n = new Node(Long.parseLong(m.group(1)));
    117             for (String s : m.group(2).split(",")) {
     117            for (String s : m.group(2).split(",", -1)) {
    118118                refs.add(new Way(Long.parseLong(s)));
    119119            }
    … …  
    123123        if (m.matches()) {
    124124            OsmPrimitive n = new Relation(Long.parseLong(m.group(1)));
    125             for (String s : m.group(2).split(",")) {
     125            for (String s : m.group(2).split(",", -1)) {
    126126                refs.add(new Relation(Long.parseLong(s)));
    127127            }
    … …  
    131131        if (m.matches()) {
    132132            OsmPrimitive n = new Way(Long.parseLong(m.group(1)));
    133             for (String s : m.group(2).split(",")) {
     133            for (String s : m.group(2).split(",", -1)) {
    134134                refs.add(new Relation(Long.parseLong(s)));
    135135            }
    … …  
    140140        if (m.matches()) {
    141141            OsmPrimitive n = OsmPrimitiveType.WAY.newInstance(Long.parseLong(m.group(1)), true);
    142             for (String s : m.group(2).split(",")) {
     142            for (String s : m.group(2).split(",", -1)) {
    143143                refs.add(new Node(Long.parseLong(s)));
    144144            }
    … …  
    149149        if (m.matches()) {
    150150            OsmPrimitive n = OsmPrimitiveType.RELATION.newInstance(Long.parseLong(m.group(1)), true);
    151             for (String s : m.group(2).split(",")) {
     151            for (String s : m.group(2).split(",", -1)) {
    152152                refs.add(new Node(Long.parseLong(s)));
    153153            }
    … …  
    158158        if (m.matches()) {
    159159            OsmPrimitive n = OsmPrimitiveType.RELATION.newInstance(Long.parseLong(m.group(1)), true);
    160             for (String s : m.group(2).split(",")) {
     160            for (String s : m.group(2).split(",", -1)) {
    161161                refs.add(new Way(Long.parseLong(s)));
    162162            }

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

    r16486 r16643  
    19761976                                        String value = ((Element) item).getAttribute("value");
    19771977                                        if (!value.isEmpty()) {
    1978                                             String[] s = value.split(" ");
     1978                                            String[] s = value.split(" ", -1);
    19791979                                            if (s.length == 3) {
    19801980                                                return parseRGB(s);

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

    r15665 r16643  
    5858            return null;
    5959        }
    60         String[] args = url.substring(i+1).split("&");
     60        String[] args = url.substring(i+1).split("&", -1);
    6161        Map<String, String> map = new HashMap<>();
    6262        for (String arg : args) {
    … …  
    6969        try {
    7070            if (map.containsKey("bbox")) {
    71                 String[] bbox = map.get("bbox").split(",");
     71                String[] bbox = map.get("bbox").split(",", -1);
    7272                b = new Bounds(
    7373                        Double.parseDouble(bbox[1]), Double.parseDouble(bbox[0]),
    … …  
    103103        if (endIndex == -1) endIndex = url.length();
    104104        String coordPart = url.substring(startIndex+(url.contains("#map=") ? "#map=".length() : "#".length()), endIndex);
    105         String[] parts = coordPart.split("/");
     105        String[] parts = coordPart.split("/", -1);
    106106        if (parts.length < 3) {
    107107            Logging.warn(tr("URL does not contain {0}/{1}/{2}", tr("zoom"), tr("latitude"), tr("longitude")));

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

    r15543 r16643  
    315315                        while (result == null && (line = reader.readLine()) != null) {
    316316                            if (line.contains("=")) {
    317                                 String[] tokens = line.split("=");
     317                                String[] tokens = line.split("=", -1);
    318318                                if (tokens.length >= 2) {
    319319                                    // Description, if available, contains exactly what we need

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

    r16630 r16643  
    5050     */
    5151    public static Map<String, String> readTagsByRegexp(String text, String splitRegex, String tagRegex, boolean unescapeTextInQuotes) {
    52          String[] lines = text.split(splitRegex);
     52         String[] lines = text.split(splitRegex, -1);
    5353         Pattern p = Pattern.compile(tagRegex);
    5454         Map<String, String> tags = new LinkedHashMap<>();

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

    r16624 r16643  
    10511051            return null;
    10521052        } else {
    1053             return String.join("\n", limit(Arrays.asList(s.split("\\n")), maxLines, "..."));
     1053            return String.join("\n", limit(Arrays.asList(s.split("\\n", -1)), maxLines, "..."));
    10541054        }
    10551055    }
    … …  
    17071707                            "java.baseline.version.url",
    17081708                            Config.getUrls().getJOSMWebsite() + "/remote/oracle-java-update-baseline.version")))
    1709                     .connect().fetchContent().split("\n");
     1709                    .connect().fetchContent().split("\n", -1);
    17101710            if (getJavaVersion() <= 8) {
    17111711                for (String version : versions) {
    … …  
    18411841        // Workaround to https://bugs.openjdk.java.net/browse/JDK-4523159
    18421842        String urlPath = jarUrl.getPath().replace("%20", " ");
    1843         if (urlPath.startsWith("file:/") && urlPath.split("!").length > 2) {
     1843        if (urlPath.startsWith("file:/") && urlPath.split("!", -1).length > 2) {
    18441844            // Locate jar file
    18451845            int index = urlPath.lastIndexOf("!/");

  • trunk/test/functional/org/openstreetmap/josm/data/imagery/ImageryCompareTestIT.java

    r15135 r16643  
    4444        String comparison = HttpClient.create(new URL("https://josm.openstreetmap.de/wiki/ImageryCompare")).connect().fetchContent();
    4545        String rubricLine = null;
    46         for (String line : comparison.split("\n")) {
     46        for (String line : comparison.split("\n", -1)) {
    4747            boolean black = line.startsWith(BLACK_PREFIX);
    4848            if (black) {

  • trunk/test/unit/org/openstreetmap/josm/TestUtils.java

    r16162 r16643  
    598598    public static List<String> getIgnoredErrorMessages(Class<?> integrationTest) throws IOException {
    599599        return Arrays.stream(new WikiReader()
    600                 .read("https://josm.openstreetmap.de/wiki/IntegrationTestIgnores?format=txt")
    601                 .split("\\n"))
     600                .read("https://josm.openstreetmap.de/wiki/IntegrationTestIgnores?format=txt").split("\\n", -1))
    602601                .filter(s -> s.startsWith("|| " + integrationTest.getSimpleName() + " ||"))
    603602                .map(s -> s.substring(s.indexOf("{{{") + 3, s.indexOf("}}}")))

  • trunk/test/unit/org/openstreetmap/josm/data/osm/TagCollectionTest.java

    r14518 r16643  
    689689        TagCollection g = new TagCollection(Arrays.asList(new Tag("k", "b"), new Tag("k", "a"), new Tag("k", "b"),
    690690                new Tag("k", "c"), new Tag("k", "d")));
    691         assertEquals("a;b;c;d", Stream.of(g.getJoinedValues("k").split(";")).sorted().collect(Collectors.joining(";")));
     691        assertEquals("a;b;c;d", Stream.of(g.getJoinedValues("k").split(";", -1)).sorted().collect(Collectors.joining(";")));
    692692    }
    693693

  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionRefTest.java

    r16006 r16643  
    9999            debug = "debug".equals(args[0]);
    100100            if (args[args.length - 1].startsWith("EPSG:")) {
    101                 forcedCodes = Arrays.asList(args[args.length - 1].split(","));
     101                forcedCodes = Arrays.asList(args[args.length - 1].split(",", -1));
    102102            }
    103103        }
    … …  
    137137                    result.add(curEntry);
    138138                } else if (curEntry != null) {
    139                     String[] f = line.trim().split(",");
     139                    String[] f = line.trim().split(",", -1);
    140140                    double lon = Double.parseDouble(f[0]);
    141141                    double lat = Double.parseDouble(f[1]);
    … …  
    277277        List<String> args = new ArrayList<>();
    278278        args.add(CS2CS_EXE);
    279         args.addAll(Arrays.asList("-f %.9f +proj=longlat +datum=WGS84 +to".split(" ")));
     279        args.addAll(Arrays.asList("-f %.9f +proj=longlat +datum=WGS84 +to".split(" ", -1)));
    280280        // proj.4 cannot read our ntf_r93_b.gsb file
    281281        // possibly because it is big endian. Use equivalent
    … …  
    286286            def = def.replace("'", "\\'").replace("\"", "\\\"");
    287287        }
    288         args.addAll(Arrays.asList(def.split(" ")));
     288        args.addAll(Arrays.asList(def.split(" ", -1)));
    289289        ProcessBuilder pb = new ProcessBuilder(args);
    290290        pb.environment().put("PROJ_LIB", new File(PROJ_LIB_DIR).getAbsolutePath());

  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionRegressionTest.java

    r16445 r16643  
    125125
    126126    private static Pair<Double, Double> readLine(String expectedName, String input) {
    127         String[] fields = input.trim().split("[ ]+");
     127        String[] fields = input.trim().split("[ ]+", -1);
    128128        if (fields.length != 3) throw new AssertionError();
    129129        if (!fields[0].equals(expectedName)) throw new AssertionError();

  • trunk/test/unit/org/openstreetmap/josm/data/validation/tests/ValidatorTestUtils.java

    r15071 r16643  
    5151                    Set<Integer> expectedCodes = new TreeSet<>();
    5252                    if (!"none".equals(codes)) {
    53                         for (String code : codes.split(",")) {
     53                        for (String code : codes.split(",", -1)) {
    5454                            expectedCodes.add(Integer.parseInt(code));
    5555                        }

  • trunk/test/unit/org/openstreetmap/josm/gui/mappaint/RenderingCLIAreaTest.java

    r16618 r16643  
    154154
    155155    public RenderingCLIAreaTest(String args, Matcher<Double> scaleMatcher, Matcher<Bounds> boundsMatcher) {
    156         this.args = args.split("\\s+");
     156        this.args = args.split("\\s+", -1);
    157157        this.scaleMatcher = scaleMatcher;
    158158        this.boundsMatcher = boundsMatcher;

  • trunk/test/unit/org/openstreetmap/josm/gui/preferences/imagery/ImageryPreferenceTestIT.java

    r16438 r16643  
    248248        // Check if we have received an error message
    249249        String error = helper.detectErrorMessage(new String(data, StandardCharsets.UTF_8));
    250         String errorMsg = url + zoomMarker(zoom) + (error != null ? error.split("\\n")[0] : defaultMessage);
     250        String errorMsg = url + zoomMarker(zoom) + (error != null ? error.split("\\n", -1)[0] : defaultMessage);
    251251        addError(info, errorMsg);
    252252        return errorMsg;
    … …  
    344344
    345345    private static boolean isZoomError(String error) {
    346         String[] parts = error.split(ERROR_SEP);
     346        String[] parts = error.split(ERROR_SEP, -1);
    347347        String lastPart = parts.length > 0 ? parts[parts.length - 1].toLowerCase(Locale.ENGLISH) : "";
    348348        return lastPart.contains("bbox")

  • trunk/test/unit/org/openstreetmap/josm/tools/KeyboardUtilsTest.java

    r14308 r16643  
    108108    private static void testgetCharactersForKeyE00(String locale, Character... expected) {
    109109        if (locale.contains("_")) {
    110             String[] l = locale.split("_");
     110            String[] l = locale.split("_", -1);
    111111            testgetCharactersForKeyE00(new Locale(l[0], l[1]), expected);
    112112        } else {
Note: See TracChangeset for help on using the changeset viewer.