[GitHub] jena pull request #171: Using library code in a few classes

2016-10-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jena/pull/171 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] jena pull request #171: Using library code in a few classes

2016-10-06 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r82183679 --- Diff: jena-arq/src/main/java/org/apache/jena/riot/out/JsonLDWriter.java --- @@ -23,7 +23,6 @@ import java.io.OutputStreamWriter ; --- End diff --

[GitHub] jena pull request #171: Using library code in a few classes

2016-10-06 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r82183935 --- Diff: jena-arq/src/main/java/org/apache/jena/riot/out/JsonLDWriter.java --- @@ -98,10 +96,8 @@ private void serialize(Writer writer, DatasetGraph dataset,

[GitHub] jena pull request #171: Using library code in a few classes

2016-09-27 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r80686831 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/test/Gen.java --- @@ -32,52 +38,35 @@ return rand(numRand, low, high, false) ; }

[GitHub] jena pull request #171: Using library code in a few classes

2016-09-27 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r80686815 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/test/Gen.java --- @@ -95,19 +84,13 @@ } if ( !found )

[GitHub] jena pull request #171: Using library code in a few classes

2016-09-26 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r80601919 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/test/Gen.java --- @@ -32,52 +38,35 @@ return rand(numRand, low, high, false) ; }

[GitHub] jena pull request #171: Using library code in a few classes

2016-09-26 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/171#discussion_r80602233 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/test/Gen.java --- @@ -95,19 +84,13 @@ } if ( !found )

[GitHub] jena pull request #171: Using library code in a few classes

2016-09-25 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/jena/pull/171 Using library code in a few classes Just a couple of orthogonal commits that use library code in `Collections`, switch to using `default` methods where it seems appropriate, remove assignments with no