Skip to content

Commit 2e0fdc6

Browse files
author
Nathan Hawes
authored
Merge pull request #31178 from nathawes/indentation-fixes-5.3
[5.3][SourceKit/CodeFormat] Indentation fixes for multi-line string, UnresolvedSpecializeExpr, and trailing target handling
2 parents e8c3927 + b24b8c3 commit 2e0fdc6

25 files changed

+320
-104
lines changed

lib/IDE/Formatting.cpp

Lines changed: 202 additions & 102 deletions
Large diffs are not rendered by default.

test/SourceKit/CodeFormat/indent-trailing/trailing-closure-in.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@ func foo() {
55
) in
66

77
// RUN: %sourcekitd-test -req=format -line=6 -length=1 %s | %FileCheck --strict-whitespace %s
8-
// CHECK: key.sourcetext: " "
8+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
enum Foo {
2+
case a,
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
enum Foo {
2+
case
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
enum Foo {
2+
case a
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
struct Foo<T: Equatable,
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
struct Foo<T: Equatable
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: ""

test/SourceKit/CodeFormat/indent-trailing/trailing-guard-condition.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ func foo(x: Bool) {
22
guard let number = patterns.index(where: {
33

44
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5-
// CHECK: key.sourcetext: " "
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
guard let number = Optional(23),
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
guard
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
guard let number = Optional(23)
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
if let number = Optional(23),
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
if
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
if let number = Optional(23)
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
struct Foo: Equatable,
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
struct Foo: Equatable
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: ""
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
class Foo {
2+
func foo(x: Int,
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
class Foo {
2+
func foo(
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
class Foo {
2+
func foo(x: Int
3+
4+
5+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
6+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
public let x = 10,
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: " "
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
public let x = 10
2+
3+
// RUN: %sourcekitd-test -req=format -line=2 -length=1 %s | %FileCheck --strict-whitespace %s
4+
// CHECK: key.sourcetext: ""
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
while let number = Optional(23),
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
while
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
func foo(x: Bool) {
2+
while let number = Optional(23)
3+
4+
// RUN: %sourcekitd-test -req=format -line=3 -length=1 %s | %FileCheck --strict-whitespace %s
5+
// CHECK: key.sourcetext: " "

test/swift-indent/basic.swift

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -308,6 +308,14 @@ let s = """
308308
c
309309
"""
310310

311+
func wantsToIndentContents() {
312+
let dontLetItIndentMyValue = """
313+
a
314+
b
315+
c
316+
"""
317+
}
318+
311319
print("""
312320
foo {
313321
bar()
@@ -905,6 +913,14 @@ let x = foo<Int,
905913
String,
906914
Int>()
907915

916+
let x = foo<
917+
Int,
918+
String,
919+
Int
920+
>()
921+
.filter { $0 > 10 }
922+
.count
923+
908924

909925
// Invalid elements should still be indented.
910926
//

0 commit comments

Comments
 (0)