[GitHub] jena pull request #463: Cleanup

2018-08-20 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r211266636 --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/core/DynamicDatasets.java --- @@ -56,8 +55,8 @@ public static DatasetGraph

[GitHub] jena pull request #463: Cleanup

2018-08-20 Thread rvesse
Github user rvesse commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r211191248 --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/core/DynamicDatasets.java --- @@ -56,8 +55,8 @@ public static DatasetGraph

[GitHub] jena pull request #463: Cleanup

2018-08-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jena/pull/463 ---

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210921706 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210896418 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/ListUtils.java --- @@ -34,49 +34,40 @@ { private ListUtils() {} -public

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210893804 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210891477 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210889580 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/ListUtils.java --- @@ -34,49 +34,40 @@ { private ListUtils() {} -public

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210856558 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/ListUtils.java --- @@ -34,49 +34,40 @@ { private ListUtils() {} -public

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210855762 --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/util/NodeUtils.java --- @@ -18,12 +18,9 @@ package org.apache.jena.sparql.util;

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210855919 --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/util/NodeUtils.java --- @@ -105,15 +102,23 @@ public String accept(Node x) { Iterator conv

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210857578 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/ServiceRouter.java --- @@ -120,10 +120,12 @@ protected boolean

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210857731 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -20,10 +20,9 @@ import static

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210857986 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/463 Cleanup This is a collection of small items, accumulated while working with the source code attached to a project. * Various comments/javadoc added and fixed and other code cleanup. * make