Feature: Handling of closing parenthesis on function calls

[matthewhughes934]

Hi, thanks for creating this tool.

I was wondering if there were any plans, or existing discussions (I didn't see any with a quick search of the issues on this repo) around handling the location of closing parenthesis for function calls.

To demonstrate my thoughts:

package pkg

func f() {
	s := []string{
		"foo",
		"bar"}

	m := map[string]int{
		"foo": 1,
		"bar": 2,}

	foo(
		"wat",
		"hmm",
		"something",)

	foo(
		"wat",
		"hmm",
		"something",
	)
}

The result of diff -U1 file.go <(gofumpt file.go):

@@ -5,3 +5,4 @@
 		"foo",
-		"bar"}
+		"bar",
+	}
 
@@ -9,3 +10,4 @@
 		"foo": 1,
-		"bar": 2,}
+		"bar": 2,
+	}
 
@@ -14,3 +16,3 @@
 		"hmm",
-		"something",)
+		"something")

So for the slice and map instance, the closing brace is forced on a newline with a trailing comma added as need. What do you think about making this also be the case for the closing parenthesis on multi-line function calls? So that both calls to foo above would be formatted as:

	foo(
		"wat",
		"hmm",
		"something",
	)

[mvdan]

Duplicate of #74 :) See the discussion there, and feel free to leave a comment if you have anything to add.

[matthewhughes934]

Duplicate of #74 :)

Ah thanks, looks like my search-fu skills were lacking