Using try-with-resources in setContents(InputStream)

gastaldi · gastaldi · commit 314607a4a42d · 2015-01-30T11:07:05.000-02:00
diff --git a/resources/api/src/main/java/org/jboss/forge/addon/resource/AbstractFileResource.java b/resources/api/src/main/java/org/jboss/forge/addon/resource/AbstractFileResource.java
@@ -245,16 +245,14 @@ public T setContents(final InputStream data)
             }
          }
 
-         OutputStream out = getFileOperations().createOutputStream(file);
-         try
+         try (OutputStream out = getResourceOutputStream())
          {
             Streams.write(data, out);
+            out.flush();
          }
          finally
          {
             Streams.closeQuietly(data);
-            out.flush();
-            Streams.closeQuietly(out);
          }
       }
       catch (IOException e)

Original file line number	Diff line number	Diff line change
`@@ -245,16 +245,14 @@ public T setContents(final InputStream data)`
`245`	`245`	`}`
`246`	`246`	`}`
`247`	`247`
`248`		`- OutputStream out = getFileOperations().createOutputStream(file);`
`249`		`- try`
	`248`	`+ try (OutputStream out = getResourceOutputStream())`
`250`	`249`	`{`
`251`	`250`	`Streams.write(data, out);`
	`251`	`+ out.flush();`
`252`	`252`	`}`
`253`	`253`	`finally`
`254`	`254`	`{`
`255`	`255`	`Streams.closeQuietly(data);`
`256`		`- out.flush();`
`257`		`- Streams.closeQuietly(out);`
`258`	`256`	`}`
`259`	`257`	`}`
`260`	`258`	`catch (IOException e)`