Merge pull request #124 from cleydyr/issue-110

[110] feche explicitamente o arquivo
cleydyr · Jul 31, 2019 · 8a57d74 · 8a57d74
2 parents 7892012 + 7f5e0be
commit 8a57d74
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/src/main/java/biblivre/core/file/BiblivreFile.java b/src/main/java/biblivre/core/file/BiblivreFile.java
@@ -93,10 +93,4 @@ public void copy(OutputStream out) throws IOException {
 	public abstract boolean exists();
 
 	public abstract void copy(OutputStream out, long start, long size) throws IOException;
-
-	@Override
-	protected void finalize() throws Throwable {
-		this.close();
-		super.finalize();
-	}
 }
diff --git a/src/main/java/biblivre/digitalmedia/Handler.java b/src/main/java/biblivre/digitalmedia/Handler.java
@@ -58,6 +58,8 @@ public void download(ExtendedRequest request, ExtendedResponse response) {
 		}
 
 		this.setFile(file);
+
+		this.setCallback(file::close);
 	}
 
 	public void upload(ExtendedRequest request, ExtendedResponse response) {
-Original file line number
+Diff line change
@@ Expand Up @@
     		}
     		this.setFile(file);
+    		this.setCallback(file::close);
     	}
     	public void upload(ExtendedRequest request, ExtendedResponse response) {
@@ Expand Down @@