fixes #684 (second)

Author: wumpz

src/main/java/net/sf/jsqlparser/statement/execute/Execute.java

@@ -36,6 +36,7 @@ public class Execute implements Statement {
     private EXEC_TYPE execType = EXEC_TYPE.EXECUTE;
     private String name;
     private ExpressionList exprList;
+    private boolean parenthesis = false;
 
     public String getName() {
         return name;
@@ -71,6 +72,14 @@ public void setExecType(EXEC_TYPE execType) {
         this.execType = execType;
     }
 
+    public boolean isParenthesis() {
+        return parenthesis;
+    }
+
+    public void setParenthesis(boolean parenthesis) {
+        this.parenthesis = parenthesis;
+    }
+
     @Override
     public void accept(StatementVisitor statementVisitor) {
         statementVisitor.visit(this);
@@ -80,7 +89,7 @@ public void accept(StatementVisitor statementVisitor) {
     public String toString() {
         return execType.name() + " " + name
                 + (exprList != null && exprList.getExpressions() != null ? " "
-                + PlainSelect.getStringList(exprList.getExpressions(), true, false) : "");
+                + PlainSelect.getStringList(exprList.getExpressions(), true, parenthesis) : "");
     }
 
     public static enum EXEC_TYPE {

src/main/java/net/sf/jsqlparser/util/deparser/ExecuteDeParser.java

@@ -52,16 +52,23 @@ public void setBuffer(StringBuilder buffer) {
 
     public void deParse(Execute execute) {
         buffer.append(execute.getExecType().name()).append(" ").append(execute.getName());
+        if (execute.isParenthesis()) {
+            buffer.append(" (");
+        } else if (execute.getExprList() != null) {
+            buffer.append(" ");
+        }
         if (execute.getExprList() != null) {
             List<Expression> expressions = execute.getExprList().getExpressions();
             for (int i = 0; i < expressions.size(); i++) {
                 if (i > 0) {
-                    buffer.append(",");
+                    buffer.append(", ");
                 }
-                buffer.append(" ");
                 expressions.get(i).accept(expressionVisitor);
             }
         }
+        if (execute.isParenthesis()) {
+            buffer.append(")");
+        }
     }
 
     public ExpressionVisitor getExpressionVisitor() {

src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt

@@ -2905,7 +2905,11 @@ Execute Execute(): {
 
     funcName=RelObjectNameList() { execute.setName(funcName); }
 
-    [ expressionList=SimpleExpressionList() ]
+    ( 
+        LOOKAHEAD(3) expressionList=SimpleExpressionList() 
+        |
+        ("(" expressionList=SimpleExpressionList() ")" { execute.setParenthesis(true); })
+    )?
 
     { 
         execute.setExprList(expressionList);

src/test/java/net/sf/jsqlparser/statement/execute/ExecuteTest.java

@@ -75,4 +75,9 @@ public void testAcceptCall() throws JSQLParserException {
     public void testCallWithMultiname() throws JSQLParserException {
         assertSqlCanBeParsedAndDeparsed("CALL BAR.FOO");
     }
+
+    @Test
+    public void testAcceptCallWithParenthesis() throws JSQLParserException {
+        assertSqlCanBeParsedAndDeparsed("CALL myproc ('a', 2, 'b')");
+    }
 }