Merge pull request swiftlang#31098 from nathawes/indentation-fixes

Nathan Hawes · web-flow · commit c0b5cb9122ce · 2020-04-20T16:51:48.000-07:00
[SourceKit/CodeFormat] Indentation fixes for multi-line string, UnresolvedSpecializeExpr, and trailing target handling.
diff --git a/lib/IDE/Formatting.cpp b/lib/IDE/Formatting.cpp
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-closure-in.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-closure-in.swift
@@ -5,4 +5,4 @@ func foo() {
     ) in
 
 // RUN: %sourcekitd-test -req=format -line=6 -length=1 %s | %FileCheck --strict-whitespace %s
-// CHECK: key.sourcetext: "        "
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-comma.swift
@@ -0,0 +1,5 @@
+enum Foo {
+    case a,
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "         "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-empty.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-empty.swift
@@ -0,0 +1,5 @@
+enum Foo {
+    case
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-enum-case-no-comma.swift
@@ -0,0 +1,5 @@
+enum Foo {
+    case a
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "    "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-generic-param-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-generic-param-comma.swift
@@ -0,0 +1,4 @@
+struct Foo<T: Equatable,
+           
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "           "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-generic-param-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-generic-param-no-comma.swift
@@ -0,0 +1,4 @@
+struct Foo<T: Equatable
+
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: ""
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-condition.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-condition.swift
@@ -2,4 +2,4 @@ func foo(x: Bool) {
     guard let number = patterns.index(where: {
 
 // RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
-// CHECK: key.sourcetext: "        "
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    guard let number = Optional(23),
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "          "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-empty.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-empty.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    guard
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-guard-pattern-no-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    guard let number = Optional(23)
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "    "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    if let number = Optional(23),
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "       "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-empty.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-empty.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    if
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-if-pattern-no-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    if let number = Optional(23)
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "    "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-inherits-item-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-inherits-item-comma.swift
@@ -0,0 +1,4 @@
+struct Foo: Equatable,
+
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "            "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-inherits-item-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-inherits-item-no-comma.swift
@@ -0,0 +1,4 @@
+struct Foo: Equatable
+
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: ""
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-comma.swift
@@ -0,0 +1,5 @@
+class Foo {
+    func foo(x: Int,
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "             "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-empty.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-empty.swift
@@ -0,0 +1,5 @@
+class Foo {
+    func foo(
+        
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-parameter-list-no-comma.swift
@@ -0,0 +1,6 @@
+class Foo {
+    func foo(x: Int
+    
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "    "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-pattern-binding-decl-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-pattern-binding-decl-comma.swift
@@ -0,0 +1,4 @@
+public let x = 10,
+
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "           "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-pattern-binding-decl-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-pattern-binding-decl-no-comma.swift
@@ -0,0 +1,4 @@
+public let x = 10
+
+// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: ""
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    while let number = Optional(23),
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "          "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-empty.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-empty.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    while
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "        "
diff --git a/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-no-comma.swift b/test/SourceKit/CodeFormat/indent-trailing/trailing-while-pattern-no-comma.swift
@@ -0,0 +1,5 @@
+func foo(x: Bool) {
+    while let number = Optional(23)
+
+// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
+// CHECK: key.sourcetext: "    "
diff --git a/test/swift-indent/basic.swift b/test/swift-indent/basic.swift
@@ -308,6 +308,14 @@ let s = """
             c
     """
 
+func wantsToIndentContents() {
+    let dontLetItIndentMyValue = """
+a
+    b
+        c
+"""
+}
+
 print("""
     foo {
         bar()
@@ -905,6 +913,14 @@ let x = foo<Int,
             String,
             Int>()
 
+let x = foo<
+    Int,
+    String,
+    Int
+>()
+.filter { $0 > 10 }
+.count
+
 
 // Invalid elements should still be indented.
 //
