You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2016/10/26 21:43:30 UTC
[3/8] jena git commit: Use try-resource to reduce warnings.
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 <an...@apache.org>
Authored: Tue Oct 25 12:39:13 2016 +0100
Committer: Andy Seaborne <an...@apache.org>
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();