Merge pull request #408 from allevato/closure-attrs

Ensure there is a break between closure attributes and a capture list.

Author: allevato

Sources/SwiftFormatPrettyPrint/TokenStreamCreator.swift

@@ -1103,7 +1103,8 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
   override func visit(_ node: ClosureSignatureSyntax) -> SyntaxVisitorContinueKind {
     before(node.firstToken, tokens: .open)
 
-    arrangeAttributeList(node.attributes, suppressFinalBreak: node.input == nil)
+    arrangeAttributeList(
+      node.attributes, suppressFinalBreak: node.input == nil && node.capture == nil)
 
     if let input = node.input {
       // We unconditionally put a break before the `in` keyword below, so we should only put a break

Tests/SwiftFormatPrettyPrintTests/ClosureExprTests.swift

@@ -492,6 +492,7 @@ final class ClosureExprTests: PrettyPrintTestCase {
       let b = { @MainActor in print("hello world") }
       let c = { @MainActor param in print("hi") }
       let d = { @MainActor (a: Int) async -> Int in print("hi") }
+      let e = { @MainActor [weak self] in print("hi") }
       """
 
     let expected =
@@ -507,6 +508,9 @@ final class ClosureExprTests: PrettyPrintTestCase {
         @MainActor (a: Int) async -> Int in
         print("hi")
       }
+      let e = { @MainActor [weak self] in
+        print("hi")
+      }
 
       """