[GitHub] jena pull request #294: Fix and tests for possible NPE (JENA-1405)

2017-10-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jena/pull/294 ---

[GitHub] jena pull request #294: Fix and tests for possible NPE (JENA-1405)

2017-10-25 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/294#discussion_r146905463 --- Diff: jena-arq/src/main/java/org/apache/jena/riot/adapters/AdapterFileManager.java --- @@ -285,6 +286,12 @@ protected Model readModelWorker(Model model, String

[GitHub] jena pull request #294: Fix and tests for possible NPE (JENA-1405)

2017-10-25 Thread rvesse
Github user rvesse commented on a diff in the pull request: https://github.com/apache/jena/pull/294#discussion_r146852236 --- Diff: jena-arq/src/main/java/org/apache/jena/riot/adapters/AdapterFileManager.java --- @@ -285,6 +286,12 @@ protected Model readModelWorker(Model model, Str

[GitHub] jena pull request #294: Fix and tests for possible NPE (JENA-1405)

2017-10-25 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/294#discussion_r146827609 --- Diff: jena-arq/src/main/java/org/apache/jena/riot/adapters/AdapterFileManager.java --- @@ -285,6 +286,12 @@ protected Model readModelWorker(Model model, Stri

[GitHub] jena pull request #294: Fix and tests for possible NPE (JENA-1405)

2017-10-25 Thread rvesse
GitHub user rvesse opened a pull request: https://github.com/apache/jena/pull/294 Fix and tests for possible NPE (JENA-1405) Fixes a possible NPE in AdaptorFileManager by aligning its behaviour with FileManager to throw NotFoundException Adds specific tests for these cas