Ignore:
Timestamp:
2014-04-29T18:06:29+02:00 (6 years ago)
Author:
Don-vip
Message:

see #8465 - use of try-with-resource

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

Legend:

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

    r7005 r7030  
    6060        Map<String, Object> config = new HashMap<>(1);
    6161        config.put(JsonGenerator.PRETTY_PRINTING, pretty);
    62         JsonWriter writer = Json.createWriterFactory(config).createWriter(stringWriter);
    63         JsonObjectBuilder object = Json.createObjectBuilder()
    64                 .add("type", "FeatureCollection")
    65                 .add("generator", "JOSM");
    66         appendLayerBounds(layer.data, object);
    67         appendLayerFeatures(layer.data, object);
    68         writer.writeObject(object.build());
    69         String result = stringWriter.toString();
    70         writer.close();
    71         return result;
     62        try (JsonWriter writer = Json.createWriterFactory(config).createWriter(stringWriter)) {
     63            JsonObjectBuilder object = Json.createObjectBuilder()
     64                    .add("type", "FeatureCollection")
     65                    .add("generator", "JOSM");
     66            appendLayerBounds(layer.data, object);
     67            appendLayerFeatures(layer.data, object);
     68            writer.writeObject(object.build());
     69            return stringWriter.toString();
     70        }
    7271    }
    7372   
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r7012 r7030  
    632632                    // even if there is no payload.
    633633                    if (requestBody != null) {
    634                         BufferedWriter bwr = new BufferedWriter(new OutputStreamWriter(out, Utils.UTF_8));
    635                         try {
     634                        try (BufferedWriter bwr = new BufferedWriter(new OutputStreamWriter(out, Utils.UTF_8))) {
    636635                            bwr.write(requestBody);
    637636                            bwr.flush();
    638                         } finally {
    639                             bwr.close();
    640637                        }
    641638                    }
     
    671668                    // Unauthorized, see #3887.
    672669                    //
    673                     BufferedReader in = new BufferedReader(new InputStreamReader(i, Utils.UTF_8));
    674670                    String s;
    675                     try {
     671                    try (BufferedReader in = new BufferedReader(new InputStreamReader(i, Utils.UTF_8))) {
    676672                        while((s = in.readLine()) != null) {
    677673                            responseBody.append(s);
    678674                            responseBody.append("\n");
    679675                        }
    680                     } finally {
    681                         in.close();
    682676                    }
    683677                }
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java

    r7005 r7030  
    141141        URLConnection openConnection = Utils.openHttpConnection(getCapabilitiesUrl);
    142142        InputStream inputStream = openConnection.getInputStream();
    143         BufferedReader br = new BufferedReader(UTFInputStreamReader.create(inputStream));
    144         String line;
    145143        StringBuilder ba = new StringBuilder();
    146         try {
     144        try (BufferedReader br = new BufferedReader(UTFInputStreamReader.create(inputStream))) {
     145            String line;
    147146            while ((line = br.readLine()) != null) {
    148147                ba.append(line);
    149148                ba.append("\n");
    150149            }
    151         } finally {
    152             br.close();
    153150        }
    154151        String incomingData = ba.toString();
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r7005 r7030  
    383383
    384384    public static String getHandlerInfoAsJSON(String cmd) {
    385         StringWriter w = new StringWriter();
    386         PrintWriter r = new PrintWriter(w);
    387         RequestHandler handler = null;
    388         try {
    389             Class<?> c = handlers.get(cmd);
    390             if (c==null) return null;
    391             handler = handlers.get(cmd).newInstance();
    392         } catch (Exception ex) {
    393             Main.error(ex);
     385        try (StringWriter w = new StringWriter()) {
     386            PrintWriter r = new PrintWriter(w);
     387            RequestHandler handler = null;
     388            try {
     389                Class<?> c = handlers.get(cmd);
     390                if (c==null) return null;
     391                handler = handlers.get(cmd).newInstance();
     392            } catch (InstantiationException | IllegalAccessException ex) {
     393                Main.error(ex);
     394                return null;
     395            }
     396   
     397            printJsonInfo(cmd, r, handler);
     398            return w.toString();
     399        } catch (IOException e) {
     400            Main.error(e);
    394401            return null;
    395402        }
    396 
     403    }
     404
     405    private static void printJsonInfo(String cmd, PrintWriter r, RequestHandler handler) {
    397406        r.printf("{ \"request\" : \"%s\"", cmd);
    398407        if (handler.getUsage() != null) {
     
    438447        }
    439448        r.append("]}");
    440         try {
    441             return w.toString();
    442         } finally {
    443             try {
    444                 w.close();
    445             } catch (IOException ex) {
    446                 Main.warn(ex);
    447             }
    448         }
    449449    }
    450450
Note: See TracChangeset for help on using the changeset viewer.