Ignore:
Timestamp:
2014-05-01T16:28:25+02:00 (6 years ago)
Author:
Don-vip
Message:

see #8465 - last batch of try-with-resources

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

Legend:

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

    r7005 r7037  
    3636import org.openstreetmap.josm.tools.CheckParameterUtil;
    3737import org.openstreetmap.josm.tools.GBC;
    38 import org.openstreetmap.josm.tools.Utils;
    3938
    4039public class GpxExporter extends FileExporter implements GpxConstants {
     
    176175        }
    177176
    178         OutputStream fo = null;
    179         try {
    180             fo = Compression.getCompressedFileOutputStream(file);
     177       
     178        try (OutputStream fo = Compression.getCompressedFileOutputStream(file)) {
    181179            new GpxWriter(fo).write(gpxData);
    182180            fo.flush();
     
    185183            JOptionPane.showMessageDialog(Main.parent, tr("Error while exporting {0}:\n{1}", fn, x.getMessage()),
    186184                    tr("Error"), JOptionPane.ERROR_MESSAGE);
    187         } finally {
    188             Utils.close(fo);
    189185        }
    190186    }
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r7012 r7037  
    174174        Collection<Collection<WayPoint>> currentTrack = new ArrayList<>();
    175175
    176         BufferedReader rd = null;
    177         try {
    178             rd = new BufferedReader(new InputStreamReader(source, Utils.UTF_8));
    179 
     176        try (BufferedReader rd = new BufferedReader(new InputStreamReader(source, Utils.UTF_8))) {
    180177            StringBuilder sb = new StringBuilder(1024);
    181178            int loopstart_char = rd.read();
     
    209206        } catch (Exception e) {
    210207            Main.warn(e);
    211         } finally {
    212             Utils.close(rd);
    213         }
    214     }
     208        }
     209    }
     210
    215211    private static class NMEAParserState {
    216212        protected Collection<WayPoint> waypoints = new ArrayList<>();
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r7033 r7037  
    629629                    activeConnection.setDoOutput(true);
    630630                    activeConnection.setRequestProperty("Content-type", "text/xml");
    631                     OutputStream out = activeConnection.getOutputStream();
    632 
    633                     // It seems that certain bits of the Ruby API are very unhappy upon
    634                     // receipt of a PUT/POST message without a Content-length header,
    635                     // even if the request has no payload.
    636                     // Since Java will not generate a Content-length header unless
    637                     // we use the output stream, we create an output stream for PUT/POST
    638                     // even if there is no payload.
    639                     if (requestBody != null) {
    640                         try (BufferedWriter bwr = new BufferedWriter(new OutputStreamWriter(out, Utils.UTF_8))) {
    641                             bwr.write(requestBody);
    642                             bwr.flush();
     631                    try (OutputStream out = activeConnection.getOutputStream()) {
     632                        // It seems that certain bits of the Ruby API are very unhappy upon
     633                        // receipt of a PUT/POST message without a Content-length header,
     634                        // even if the request has no payload.
     635                        // Since Java will not generate a Content-length header unless
     636                        // we use the output stream, we create an output stream for PUT/POST
     637                        // even if there is no payload.
     638                        if (requestBody != null) {
     639                            try (BufferedWriter bwr = new BufferedWriter(new OutputStreamWriter(out, Utils.UTF_8))) {
     640                                bwr.write(requestBody);
     641                                bwr.flush();
     642                            }
    643643                        }
    644644                    }
    645                     Utils.close(out);
    646645                }
    647646
  • trunk/src/org/openstreetmap/josm/io/OsmImporter.java

    r6798 r7037  
    6262    @Override
    6363    public void importData(File file, ProgressMonitor progressMonitor) throws IOException, IllegalDataException {
    64         InputStream in = null;
    65         try {
    66             in = Compression.getUncompressedFileInputStream(file);
     64        try (InputStream in = Compression.getUncompressedFileInputStream(file)) {
    6765            importData(in, file, progressMonitor);
    6866        } catch (FileNotFoundException e) {
    6967            Main.error(e);
    7068            throw new IOException(tr("File ''{0}'' does not exist.", file.getName()), e);
    71         } finally {
    72             Utils.close(in);
    7369        }
    7470    }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpsServer.java

    r7034 r7037  
    2929
    3030import org.openstreetmap.josm.Main;
    31 import org.openstreetmap.josm.tools.Utils;
    3231
    3332/**
     
    6059                        Main.error(tr("Unable to find JOSM keystore at {0}. Remote control will not be available on HTTPS.", KEYSTORE_PATH));
    6160                    } else {
    62                         try {
    63                             ks.load(in, password);
    64                         } finally {
    65                             Utils.close(in);
    66                         }
     61                        ks.load(in, password);
    6762                       
    6863                        if (Main.isDebugEnabled()) {
Note: See TracChangeset for help on using the changeset viewer.