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 2013/05/07 21:31:07 UTC
svn commit: r1480041 -
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java
Author: andy
Date: Tue May 7 19:31:07 2013
New Revision: 1480041
URL: http://svn.apache.org/r1480041
Log:
parens for ASC
Modified:
jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java
Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java?rev=1480041&r1=1480040&r2=1480041&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/SortCondition.java Tue May 7 19:31:07 2013
@@ -58,6 +58,9 @@ public class SortCondition extends Print
IndentedWriter writer)
{
boolean explicitDirection = false ;
+ // Not always necessary but safe.
+ // At this point there must be brackets but some forms (e.g. ?x+?y)
+ // are going to put their own brackets in regardless.
boolean needParens = false ;
if ( direction != Query.ORDER_DEFAULT )
@@ -70,11 +73,8 @@ public class SortCondition extends Print
if ( direction == Query.ORDER_ASCENDING )
{
- // Not always necessary but safe.
- // At this point theer must be brackets but some forms (e.g. ?x+?y)
- // are going to put their own brackets in regardless.
writer.print("ASC") ;
- //needParens = true ;
+ needParens = true ;
}
if ( direction == Query.ORDER_DESCENDING )