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 2012/02/01 11:17:50 UTC

svn commit: r1239038 - in /incubator/jena/Jena2/ARQ/trunk/src: main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java

Author: andy
Date: Wed Feb  1 10:17:50 2012
New Revision: 1239038

URL: http://svn.apache.org/viewvc?rev=1239038&view=rev
Log:
JENA-207 FmtUtils.stringForNode() does not escape tab characters

Modified:
    incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java
    incubator/jena/Jena2/ARQ/trunk/src/test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java

Modified: incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java
URL: http://svn.apache.org/viewvc/incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java?rev=1239038&r1=1239037&r2=1239038&view=diff
==============================================================================
--- incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java (original)
+++ incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/util/FmtUtils.java Wed Feb  1 10:17:50 2012
@@ -528,7 +528,7 @@ public class FmtUtils
 //            }    
 
             // Whitespace
-            if ( singleLineString && ( c == '\n' || c == '\r' || c == '\f' ) )
+            if ( singleLineString && ( c == '\n' || c == '\r' || c == '\f' || c == '\t' ) )
             {
                 if (c == '\n') sbuff.append("\\n");
                 if (c == '\t') sbuff.append("\\t");

Modified: incubator/jena/Jena2/ARQ/trunk/src/test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java
URL: http://svn.apache.org/viewvc/incubator/jena/Jena2/ARQ/trunk/src/test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java?rev=1239038&r1=1239037&r2=1239038&view=diff
==============================================================================
--- incubator/jena/Jena2/ARQ/trunk/src/test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java (original)
+++ incubator/jena/Jena2/ARQ/trunk/src/test/java/com/hp/hpl/jena/sparql/resultset/TestResultSetFormat1.java Wed Feb  1 10:17:50 2012
@@ -87,10 +87,15 @@ public class TestResultSetFormat1
     static String[] $rs7 = {
         "(resultset (?x) (row))" } ;
     
+    static String[] $rs8 = {
+    	"(resultset (?x)",
+    	" (row (?x \"has \t tab character\"))",
+    	")" } ;
+        
     @Parameters
     public static Collection<Object[]> data()
     {
-        return Arrays.asList(new Object[][] { {$rs0}, {$rs1}, {$rs2}, {$rs3}, {$rs4}, {$rs5}, {$rs6} , {$rs7} } ) ;
+        return Arrays.asList(new Object[][] { {$rs0}, {$rs1}, {$rs2}, {$rs3}, {$rs4}, {$rs5}, {$rs6}, {$rs7}, {$rs8} } ) ;
     }
 
     private final String[] $rs ;