Omit parenthesis when no default arg

Author: Luma

autoload/vimlparser.vim

@@ -4979,7 +4979,7 @@ function! s:Compiler.compile_function(node) abort
     endif
     for i in range(len(rlist))
       if i < len(rlist) - len(default_args)
-        let left .= printf(' (%s)', rlist[i])
+        let left .= printf(' %s', rlist[i])
       else
         let left .= printf(' (%s %s)', rlist[i], default_args[i + len(default_args) - len(rlist)])
       endif

js/vimlparser.js

@@ -4476,16 +4476,16 @@ Compiler.prototype.compile_function = function(node) {
     var rlist = node.rlist.map((function(vval) { return this.compile(vval); }).bind(this));
     var default_args = node.default_args.map((function(vval) { return this.compile(vval); }).bind(this));
     if (!viml_empty(rlist)) {
-        var remaining = v:false;
+        var remaining = FALSE;
         if (rlist[rlist.length - 1] == "...") {
             viml_remove(rlist, -1);
-            var remaining = v:true;
+            var remaining = TRUE;
         }
         var __c11 = viml_range(viml_len(rlist));
         for (var __i11 = 0; __i11 < __c11.length; ++__i11) {
             var i = __c11[__i11];
             if (i < viml_len(rlist) - viml_len(default_args)) {
-                left += viml_printf(" (%s)", rlist[i]);
+                left += viml_printf(" %s", rlist[i]);
             }
             else {
                 left += viml_printf(" (%s %s)", rlist[i], default_args[i + viml_len(default_args) - viml_len(rlist)]);

py/vimlparser.py

@@ -3583,13 +3583,13 @@ def compile_function(self, node):
         rlist = [self.compile(vval) for vval in node.rlist]
         default_args = [self.compile(vval) for vval in node.default_args]
         if not viml_empty(rlist):
-            remaining = v:false
+            remaining = FALSE
             if rlist[-1] == "...":
                 viml_remove(rlist, -1)
-                remaining = v:true
+                remaining = TRUE
             for i in viml_range(viml_len(rlist)):
                 if i < viml_len(rlist) - viml_len(default_args):
-                    left += viml_printf(" (%s)", rlist[i])
+                    left += viml_printf(" %s", rlist[i])
                 else:
                     left += viml_printf(" (%s %s)", rlist[i], default_args[i + viml_len(default_args) - viml_len(rlist)])
             if remaining:

test/test1.ok

@@ -1,5 +1,5 @@
 ; test1
-(function (s:foo (a) (b) . ...)
+(function (s:foo a b . ...)
   (return 0))
 (function (s:bar (a 1) (b 2) . ...)
   (return 0))

test/test_xxx_funcarg_last_comma.ok

@@ -1,2 +1,2 @@
-(function (F (a)))
-(function (G (a)))
+(function (F a))
+(function (G a))