Use try-resource to reduce warnings. Project: http://git-wip-us.apache.org/repos/asf/jena/repo Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/7df4d4cd Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/7df4d4cd Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/7df4d4cd
Branch: refs/heads/master Commit: 7df4d4cd2db09bfea06999b040160d0321e42b9d Parents: 46e1648 Author: Andy Seaborne <[email protected]> Authored: Tue Oct 25 12:39:13 2016 +0100 Committer: Andy Seaborne <[email protected]> Committed: Tue Oct 25 12:39:13 2016 +0100 ---------------------------------------------------------------------- .../jena/permissions/query/QueryEngineTest.java | 22 ++++---------------- 1 file changed, 4 insertions(+), 18 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jena/blob/7df4d4cd/jena-permissions/src/test/java/org/apache/jena/permissions/query/QueryEngineTest.java ---------------------------------------------------------------------- diff --git a/jena-permissions/src/test/java/org/apache/jena/permissions/query/QueryEngineTest.java b/jena-permissions/src/test/java/org/apache/jena/permissions/query/QueryEngineTest.java index 12e13f3..75f2f85 100644 --- a/jena-permissions/src/test/java/org/apache/jena/permissions/query/QueryEngineTest.java +++ b/jena-permissions/src/test/java/org/apache/jena/permissions/query/QueryEngineTest.java @@ -122,9 +122,7 @@ public class QueryEngineTest { + " { ?foo a <http://example.com/class> ; " + "?bar [] ." + " } "; - final QueryExecution qexec = QueryExecutionFactory.create(query, - model); - try { + try ( QueryExecution qexec = QueryExecutionFactory.create(query, model) ) { final ResultSet results = qexec.execSelect(); int count = 0; for (; results.hasNext();) { @@ -132,8 +130,6 @@ public class QueryEngineTest { results.nextSolution(); } Assert.assertEquals(8, count); - } finally { - qexec.close(); } } finally { model.close(); @@ -164,9 +160,7 @@ public class QueryEngineTest { + " { ?foo a <http://example.com/class> ; " + "?bar [] ." + " } "; - final QueryExecution qexec = QueryExecutionFactory.create(query, - model); - try { + try ( QueryExecution qexec = QueryExecutionFactory.create(query, model) ) { final ResultSet results = qexec.execSelect(); int count = 0; for (; results.hasNext();) { @@ -174,8 +168,6 @@ public class QueryEngineTest { results.nextSolution(); } Assert.assertEquals(4, count); - } finally { - qexec.close(); } } finally { model.close(); @@ -201,8 +193,7 @@ public class QueryEngineTest { "http://example.com/securedModel";, baseModel); try { String query = "SELECT ?s ?p ?o WHERE " + " { ?s ?p ?o } "; - QueryExecution qexec = QueryExecutionFactory.create(query, model); - try { + try ( QueryExecution qexec = QueryExecutionFactory.create(query, model) ) { final ResultSet results = qexec.execSelect(); int count = 0; for (; results.hasNext();) { @@ -211,13 +202,10 @@ public class QueryEngineTest { } // 2x 3 values + type triple Assert.assertEquals(8, count); - } finally { - qexec.close(); } query = "SELECT ?s ?p ?o WHERE " + " { GRAPH ?g {?s ?p ?o } }"; - qexec = QueryExecutionFactory.create(query, model); - try { + try ( QueryExecution qexec = QueryExecutionFactory.create(query, model) ) { final ResultSet results = qexec.execSelect(); int count = 0; for (; results.hasNext();) { @@ -227,8 +215,6 @@ public class QueryEngineTest { // 2x 3 values + type triple // no named graphs so no results. Assert.assertEquals(0, count); - } finally { - qexec.close(); } } finally { model.close();
