[email protected] wrote:
+    /**
+     * Returns the nodes in the given query result as an {...@link Iterable}
+     * for use in a Java 5 for-each loop. The return value encapsulates
+     * the {...@link QueryResult#getNodes()} method call. Potential
+     * {...@link RepositoryException}s are converted to {...@link 
RuntimeException}s.
+     *
+     * @param result query result
+     * @return nodes in the query result
+     */
+    public static Iterable<Node> getNodes(final QueryResult result) {
+        return new Iterable<Node>() {
+            @SuppressWarnings("unchecked")
+            public Iterator<Node> iterator() {
+                try {
+                    return result.getNodes();
+                } catch (RepositoryException e) {
+                    throw new RuntimeException(
+                            "Unable to access nodes in " + result, e);
+                }
+            }
+        };
+    }

Jukka, is it really a good idea to catch the RepositoryException here, mapping it to an unchecked exception?

BR, Julian

Reply via email to