You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2014/05/23 15:38:20 UTC

svn commit: r1597093 - in /jena/trunk/jena-arq/src/main/java/org/apache/jena/riot: out/NodeFormatterTTL.java out/OutputProfile.java writer/WriterStream.java

Author: andy
Date: Fri May 23 13:38:19 2014
New Revision: 1597093

URL: http://svn.apache.org/r1597093
Log:
Inversion: output control should incorporate a NodeFormatter implementation,
not make the NodeFormatter implementation vary due to output setup.

Removed:
    jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/OutputProfile.java
Modified:
    jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/NodeFormatterTTL.java
    jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/writer/WriterStream.java

Modified: jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/NodeFormatterTTL.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/NodeFormatterTTL.java?rev=1597093&r1=1597092&r2=1597093&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/NodeFormatterTTL.java (original)
+++ jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/out/NodeFormatterTTL.java Fri May 23 13:38:19 2014
@@ -40,10 +40,7 @@ public class NodeFormatterTTL extends No
     private final String      baseIRI ;
     private final IRI         iriResolver ;
 
-    // Replace with a single "OutputPolicy"
-    public NodeFormatterTTL(String baseIRI, PrefixMap prefixMap) // OutputPolicy
-                                                                 // outputPolicy)
-    {
+    public NodeFormatterTTL(String baseIRI, PrefixMap prefixMap) {
         this(baseIRI, prefixMap, NodeToLabel.createBNodeByLabelEncoded()) ;
     }
 

Modified: jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/writer/WriterStream.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/writer/WriterStream.java?rev=1597093&r1=1597092&r2=1597093&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/writer/WriterStream.java (original)
+++ jena/trunk/jena-arq/src/main/java/org/apache/jena/riot/writer/WriterStream.java Fri May 23 13:38:19 2014
@@ -30,8 +30,12 @@ import com.hp.hpl.jena.graph.Triple ;
 import com.hp.hpl.jena.sparql.core.DatasetGraph ;
 import com.hp.hpl.jena.sparql.core.Quad ;
 
+/** Library class with helpers for writing streams of RDF (as {@linkplain StreamRDF}) */ 
 public class WriterStream
 {
+    // In development.
+    // This is really utilities for StreamRDF and should go into StreamRDFLib.
+    // It's the nature of the StreamRDF that does the writing. 
     private WriterStream() {}
     
     protected static void finish(StreamRDF dest)