diff options
author | Miguel Alcon Pinto <malcon@google.com> | 2015-11-04 17:36:02 +0000 |
---|---|---|
committer | John Field <jfield@google.com> | 2015-11-05 16:49:27 +0000 |
commit | 53fb4d0ef10dea9d58f4b17f08c59c8e81ebc09a (patch) | |
tree | 38ab19ff52f325ce6fa33d7e4e925c9c4fb89c7d /src/main/java/com/google/devtools/build/lib/analysis | |
parent | 5c67934f08c821af6fbf0eab422caed805d77ee4 (diff) |
Close some streams that we didn't
--
MOS_MIGRATED_REVID=107048547
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java b/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java index 2540839792..aebc209ced 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java @@ -361,17 +361,17 @@ public final class Runfiles { for (Artifact artifact : pruningManifest.getCandidateRunfiles()) { allowedRunfiles.put(artifact.getRootRelativePath().getPathString(), artifact); } - BufferedReader reader = new BufferedReader( - new InputStreamReader(pruningManifest.getManifestFile().getPath().getInputStream())); - String line; - while ((line = reader.readLine()) != null) { - Artifact artifact = allowedRunfiles.get(line); - if (artifact != null) { - manifest.put(artifact.getRootRelativePath(), artifact); + try (BufferedReader reader = new BufferedReader( + new InputStreamReader(pruningManifest.getManifestFile().getPath().getInputStream()))) { + String line; + while ((line = reader.readLine()) != null) { + Artifact artifact = allowedRunfiles.get(line); + if (artifact != null) { + manifest.put(artifact.getRootRelativePath(), artifact); + } } } } - manifest = filterListForObscuringSymlinks(eventHandler, location, manifest); // TODO(bazel-team): Create /dev/null-like Artifact to avoid nulls? |