You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by afs <gi...@git.apache.org> on 2018/04/27 22:30:55 UTC
[GitHub] jena pull request #410: JENA-1535: Parse a specified language if given --syn...
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/410
JENA-1535: Parse a specified language if given --syntax.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena riot-syntax
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/410.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #410
----
commit 3ff8c96495862271a20f3298950ebb67e5d4a6ee
Author: Andy Seaborne <an...@...>
Date: 2018-04-27T22:27:38Z
JENA-1535: Parse a specified language if given --syntax.
----
---
[GitHub] jena pull request #410: JENA-1535: Parse a specified language if given --syn...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/jena/pull/410
---
[GitHub] jena pull request #410: JENA-1535: Parse a specified language if given --syn...
Posted by kinow <gi...@git.apache.org>.
Github user kinow commented on a diff in the pull request:
https://github.com/apache/jena/pull/410#discussion_r184821458
--- Diff: jena-cmds/src/main/java/riotcmd/CmdLangParse.java ---
@@ -218,8 +218,14 @@ public ParseRecord parseFile(String filename) {
RDFParserBuilder builder = RDFParser.create();
if ( baseURI != null )
builder.base(baseURI);
- Lang lang = selectLang(null, null, RDFLanguages.NQUADS) ;
- builder.lang(lang);
+ if ( modLangParse.getLang() != null )
+ // Always use the command line specified synatx.
--- End diff --
s/synatx/syntax
---
[GitHub] jena pull request #410: JENA-1535: Parse a specified language if given --syn...
Posted by afs <gi...@git.apache.org>.
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/410#discussion_r184823377
--- Diff: jena-cmds/src/main/java/riotcmd/CmdLangParse.java ---
@@ -218,8 +218,14 @@ public ParseRecord parseFile(String filename) {
RDFParserBuilder builder = RDFParser.create();
if ( baseURI != null )
builder.base(baseURI);
- Lang lang = selectLang(null, null, RDFLanguages.NQUADS) ;
- builder.lang(lang);
+ if ( modLangParse.getLang() != null )
+ // Always use the command line specified synatx.
--- End diff --
Done!
---