AfflictionNetworks/depot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

We need parens so that operators are evaluated in the right order.

Browse Source 
This adds extra params around a bunch of things. I'll let the gods of Depot
sort that out.

BTW: Arithmetic.Div is still totally broken. It somehow booches the prepared
statement. I tried changing the operator String to "//" and that sorta worked,
but then the next operator over was missing...
I'm working around this using Mul, but it'd be nice..
This commit is contained in:
Ray Greenwell
2008-05-16 02:10:07 +00:00
parent 31b886aed2
commit 195fabe2cc
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/java/com/samskivert/jdbc/depot/BuildVisitor.java
+2
View File
@@ -188,9 +188,11 @@ public abstract class BuildVisitor implements ExpressionVisitor
public void visit (BinaryOperator binaryOperator) public void visit (BinaryOperator binaryOperator)
throws Exception throws Exception
{ {
_builder.append('(');
binaryOperator.getLeftHandSide().accept(this); binaryOperator.getLeftHandSide().accept(this);
_builder.append(binaryOperator.operator()); _builder.append(binaryOperator.operator());
binaryOperator.getRightHandSide().accept(this); binaryOperator.getRightHandSide().accept(this);
_builder.append(')');
} }
public void visit (IsNull isNull) public void visit (IsNull isNull)