Skip to content

Commit 61b160d

Browse files
committed
[nfc] Run swift-format over Swift ASTGen sources.
1 parent df00477 commit 61b160d

File tree

11 files changed

+164
-142
lines changed

11 files changed

+164
-142
lines changed

lib/ASTGen/Package.swift

Lines changed: 27 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -10,31 +10,31 @@
1010
import PackageDescription
1111

1212
let package = Package(
13-
name: "ASTGen",
14-
platforms: [
15-
.macOS(.v10_15)
16-
],
17-
products: [
18-
.library(name: "swiftASTGen", targets: ["swiftASTGen"]),
19-
],
20-
dependencies: [
21-
.package(path: "../../../swift-syntax"),
22-
],
23-
targets: [
24-
.target(
25-
name: "swiftASTGen",
26-
dependencies: [
27-
.product(name: "SwiftSyntax", package: "swift-syntax"),
28-
.product(name: "SwiftParser", package: "swift-syntax"),
29-
.product(name: "_SwiftSyntaxMacros", package: "swift-syntax")
30-
],
31-
path: ".",
32-
exclude: ["CMakeLists.txt"],
33-
swiftSettings: [
34-
.unsafeFlags([
35-
"-I", "../../include/swift/",
36-
"-I", "../../include"
37-
])
38-
]),
39-
]
13+
name: "ASTGen",
14+
platforms: [
15+
.macOS(.v10_15)
16+
],
17+
products: [
18+
.library(name: "swiftASTGen", targets: ["swiftASTGen"])
19+
],
20+
dependencies: [
21+
.package(path: "../../../swift-syntax")
22+
],
23+
targets: [
24+
.target(
25+
name: "swiftASTGen",
26+
dependencies: [
27+
.product(name: "SwiftSyntax", package: "swift-syntax"),
28+
.product(name: "SwiftParser", package: "swift-syntax"),
29+
.product(name: "_SwiftSyntaxMacros", package: "swift-syntax"),
30+
],
31+
path: ".",
32+
exclude: ["CMakeLists.txt"],
33+
swiftSettings: [
34+
.unsafeFlags([
35+
"-I", "../../include/swift/",
36+
"-I", "../../include",
37+
])
38+
])
39+
]
4040
)

lib/ASTGen/Sources/ASTGen/ASTGen.swift

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1+
import CASTBridging
12
import SwiftParser
23
import SwiftSyntax
34

4-
import CASTBridging
5-
65
extension Array {
76
public func withBridgedArrayRef<T>(_ c: (BridgedArrayRef) -> T) -> T {
87
withUnsafeBytes { buf in
@@ -22,7 +21,7 @@ enum ASTNode {
2221
case stmt(UnsafeMutableRawPointer)
2322
case expr(UnsafeMutableRawPointer)
2423
case type(UnsafeMutableRawPointer)
25-
24+
2625
var rawValue: UnsafeMutableRawPointer {
2726
switch self {
2827
case .decl(let ptr):
@@ -35,7 +34,7 @@ enum ASTNode {
3534
return ptr
3635
}
3736
}
38-
37+
3938
func bridged() -> ASTNodeBridged {
4039
switch self {
4140
case .expr(let e):
@@ -61,17 +60,17 @@ class Boxed<Value> {
6160

6261
struct ASTGenVisitor: SyntaxTransformVisitor {
6362
typealias ResultType = ASTNode
64-
63+
6564
let ctx: UnsafeMutableRawPointer
6665
let base: UnsafePointer<UInt8>
6766

6867
@Boxed var declContext: UnsafeMutableRawPointer
6968

7069
// TODO: this some how messes up the witness table when I uncomment it locally :/
71-
// public func visit<T>(_ node: T?) -> [UnsafeMutableRawPointer]? {
72-
// if let node = node { return visit(node) }
73-
// return nil
74-
// }
70+
// public func visit<T>(_ node: T?) -> [UnsafeMutableRawPointer]? {
71+
// if let node = node { return visit(node) }
72+
// return nil
73+
// }
7574

7675
@_disfavoredOverload
7776
public func visit(_ node: SourceFileSyntax) -> ASTNode {

lib/ASTGen/Sources/ASTGen/Decls.swift

Lines changed: 22 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1+
import CASTBridging
12
import SwiftParser
23
import SwiftSyntax
34

4-
import CASTBridging
5-
65
extension ASTGenVisitor {
76
public func visit(_ node: TypealiasDeclSyntax) -> ASTNode {
87
let aliasLoc = self.base.advanced(by: node.typealiasKeyword.position.utf8Offset).raw
@@ -13,7 +12,8 @@ extension ASTGenVisitor {
1312
}
1413
let nameLoc = self.base.advanced(by: node.identifier.position.utf8Offset).raw
1514
let genericParams = node.genericParameterClause.map(self.visit).map { $0.rawValue }
16-
let out = TypeAliasDecl_create(self.ctx, self.declContext, aliasLoc, equalLoc, name, nameLoc, genericParams)
15+
let out = TypeAliasDecl_create(
16+
self.ctx, self.declContext, aliasLoc, equalLoc, name, nameLoc, genericParams)
1717

1818
let oldDeclContext = declContext
1919
declContext = out.declContext
@@ -43,10 +43,10 @@ extension ASTGenVisitor {
4343
node.members.members.map(self.visit).withBridgedArrayRef { ref in
4444
NominalTypeDecl_setMembers(out.nominalDecl, ref)
4545
}
46-
46+
4747
return .decl(out.decl)
4848
}
49-
49+
5050
public func visit(_ node: ClassDeclSyntax) -> ASTNode {
5151
let loc = self.base.advanced(by: node.position.utf8Offset).raw
5252
var nameText = node.identifier.text
@@ -62,16 +62,16 @@ extension ASTGenVisitor {
6262
node.members.members.map(self.visit).withBridgedArrayRef { ref in
6363
NominalTypeDecl_setMembers(out.nominalDecl, ref)
6464
}
65-
65+
6666
return .decl(out.decl)
6767
}
68-
68+
6969
public func visit(_ node: VariableDeclSyntax) -> ASTNode {
7070
let pattern = visit(node.bindings.first!.pattern).rawValue
7171
let initializer = visit(node.bindings.first!.initializer!).rawValue
72-
72+
7373
let loc = self.base.advanced(by: node.position.utf8Offset).raw
74-
let isStateic = false // TODO: compute this
74+
let isStateic = false // TODO: compute this
7575
let isLet = node.letOrVarKeyword.tokenKind == .letKeyword
7676

7777
// TODO: don't drop "initializer" on the floor.
@@ -80,10 +80,10 @@ extension ASTGenVisitor {
8080

8181
public func visit(_ node: FunctionParameterSyntax) -> ASTNode {
8282
let loc = self.base.advanced(by: node.position.utf8Offset).raw
83-
83+
8484
let firstName: UnsafeMutableRawPointer?
8585
let secondName: UnsafeMutableRawPointer?
86-
86+
8787
if let nodeFirstName = node.firstName {
8888
var text = nodeFirstName.text
8989
firstName = text.withUTF8 { buf in
@@ -92,7 +92,7 @@ extension ASTGenVisitor {
9292
} else {
9393
firstName = nil
9494
}
95-
95+
9696
if let nodeSecondName = node.secondName {
9797
var text = nodeSecondName.text
9898
secondName = text.withUTF8 { buf in
@@ -101,18 +101,18 @@ extension ASTGenVisitor {
101101
} else {
102102
secondName = nil
103103
}
104-
104+
105105
return .decl(ParamDecl_create(ctx, loc, loc, firstName, loc, secondName, declContext))
106106
}
107107

108108
public func visit(_ node: FunctionDeclSyntax) -> ASTNode {
109109
let loc = self.base.advanced(by: node.position.utf8Offset).raw
110-
110+
111111
var nameText = node.identifier.text
112112
let name = nameText.withUTF8 { buf in
113113
return SwiftASTContext_getIdentifier(ctx, buf.baseAddress, buf.count)
114114
}
115-
115+
116116
let body: ASTNode?
117117
if let nodeBody = node.body {
118118
body = visit(nodeBody)
@@ -126,10 +126,13 @@ extension ASTGenVisitor {
126126
} else {
127127
returnType = nil
128128
}
129-
129+
130130
let params = node.signature.input.parameterList.map { visit($0) }
131-
return .decl(params.withBridgedArrayRef { ref in
132-
FuncDecl_create(ctx, loc, false, loc, name, loc, false, nil, false, nil, loc, ref, loc, body?.rawValue, returnType?.rawValue, declContext)
133-
})
131+
return .decl(
132+
params.withBridgedArrayRef { ref in
133+
FuncDecl_create(
134+
ctx, loc, false, loc, name, loc, false, nil, false, nil, loc, ref, loc, body?.rawValue,
135+
returnType?.rawValue, declContext)
136+
})
134137
}
135138
}

lib/ASTGen/Sources/ASTGen/Exprs.swift

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1+
import CASTBridging
12
import SwiftParser
23
import SwiftSyntax
34

4-
import CASTBridging
5-
65
extension ASTGenVisitor {
76
public func visit(_ node: ClosureExprSyntax) -> ASTNode {
87
let statements = node.statements.map(self.visit).map { $0.bridged() }
98
let loc = self.base.advanced(by: node.position.utf8Offset).raw
109

1110
let body = statements.withBridgedArrayRef { ref in
12-
BraceStmt_create(ctx, loc, ref, loc)
11+
BraceStmt_create(ctx, loc, ref, loc)
1312
}
14-
13+
1514
return .expr(ClosureExpr_create(ctx, body, declContext))
1615
}
1716

1817
public func visit(_ node: FunctionCallExprSyntax) -> ASTNode {
1918
// Transform the trailing closure into an argument.
2019
if let trailingClosure = node.trailingClosure {
21-
let tupleElement = TupleExprElementSyntax(label: nil, colon: nil, expression: ExprSyntax(trailingClosure), trailingComma: nil)
22-
20+
let tupleElement = TupleExprElementSyntax(
21+
label: nil, colon: nil, expression: ExprSyntax(trailingClosure), trailingComma: nil)
22+
2323
return visit(node.addArgument(tupleElement).withTrailingClosure(nil))
2424
}
25-
25+
2626
let args = visit(node.argumentList).rawValue
2727
// TODO: hack
2828
let callee = visit(node.calledExpression).rawValue
@@ -32,7 +32,7 @@ extension ASTGenVisitor {
3232

3333
public func visit(_ node: IdentifierExprSyntax) -> ASTNode {
3434
let loc = self.base.advanced(by: node.position.utf8Offset).raw
35-
35+
3636
var text = node.identifier.text
3737
let id = text.withUTF8 { buf in
3838
return SwiftASTContext_getIdentifier(ctx, buf.baseAddress, buf.count)
@@ -43,35 +43,36 @@ extension ASTGenVisitor {
4343

4444
public func visit(_ node: IdentifierPatternSyntax) -> ASTNode {
4545
let loc = self.base.advanced(by: node.position.utf8Offset).raw
46-
46+
4747
var text = node.identifier.text
4848
let id = text.withUTF8 { buf in
4949
return SwiftASTContext_getIdentifier(ctx, buf.baseAddress, buf.count)
5050
}
5151

5252
return .expr(SwiftIdentifierExpr_create(ctx, id, loc))
5353
}
54-
54+
5555
public func visit(_ node: MemberAccessExprSyntax) -> ASTNode {
5656
let loc = self.base.advanced(by: node.position.utf8Offset).raw
5757
let base = visit(node.base!).rawValue
5858
var nameText = node.name.text
5959
let name = nameText.withUTF8 { buf in
6060
return SwiftASTContext_getIdentifier(ctx, buf.baseAddress, buf.count)
6161
}
62-
62+
6363
return .expr(UnresolvedDotExpr_create(ctx, base, loc, name, loc))
6464
}
6565

6666
public func visit(_ node: TupleExprElementListSyntax) -> ASTNode {
6767
let elements = node.map(self.visit).map { $0.rawValue }
68-
68+
6969
// TODO: find correct paren locs.
7070
let lParenLoc = self.base.advanced(by: node.position.utf8Offset).raw
7171
let rParenLoc = self.base.advanced(by: node.position.utf8Offset).raw
72-
73-
return .expr(elements.withBridgedArrayRef { elementsRef in
74-
SwiftTupleExpr_create(self.ctx, lParenLoc, elementsRef, rParenLoc)
75-
})
72+
73+
return .expr(
74+
elements.withBridgedArrayRef { elementsRef in
75+
SwiftTupleExpr_create(self.ctx, lParenLoc, elementsRef, rParenLoc)
76+
})
7677
}
7778
}

lib/ASTGen/Sources/ASTGen/Generics.swift

Lines changed: 25 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,18 @@
1+
import CASTBridging
12
import SwiftParser
23
import SwiftSyntax
34

4-
import CASTBridging
5-
65
extension ASTGenVisitor {
76
func visit(_ node: GenericParameterClauseSyntax) -> ASTNode {
87
let lAngleLoc = self.base.advanced(by: node.leftAngleBracket.position.utf8Offset).raw
9-
let whereLoc = node.genericWhereClause.map { self.base.advanced(by: $0.whereKeyword.position.utf8Offset).raw }
8+
let whereLoc = node.genericWhereClause.map {
9+
self.base.advanced(by: $0.whereKeyword.position.utf8Offset).raw
10+
}
1011
let rAngleLoc = self.base.advanced(by: node.rightAngleBracket.position.utf8Offset).raw
11-
return .decl(self.withBridgedParametersAndRequirements(node) { params, reqs in
12-
return GenericParamList_create(self.ctx, lAngleLoc, params, whereLoc, reqs, rAngleLoc)
13-
})
12+
return .decl(
13+
self.withBridgedParametersAndRequirements(node) { params, reqs in
14+
return GenericParamList_create(self.ctx, lAngleLoc, params, whereLoc, reqs, rAngleLoc)
15+
})
1416
}
1517

1618
func visit(_ node: GenericParameterSyntax) -> ASTNode {
@@ -21,7 +23,10 @@ extension ASTGenVisitor {
2123
let nameLoc = self.base.advanced(by: node.name.position.utf8Offset).raw
2224
let ellipsisLoc = node.ellipsis.map { self.base.advanced(by: $0.position.utf8Offset).raw }
2325

24-
return .decl(GenericTypeParamDecl_create(self.ctx, self.declContext, name, nameLoc, ellipsisLoc, node.indexInParent / 2, ellipsisLoc != nil))
26+
return .decl(
27+
GenericTypeParamDecl_create(
28+
self.ctx, self.declContext, name, nameLoc, ellipsisLoc, node.indexInParent / 2,
29+
ellipsisLoc != nil))
2530
}
2631
}
2732

@@ -41,7 +46,7 @@ extension ASTGenVisitor {
4146
}
4247

4348
let loweredRequirement = self.visit(requirement)
44-
GenericTypeParamDecl_setInheritedType(self.ctx, loweredParameter, loweredRequirement.rawValue);
49+
GenericTypeParamDecl_setInheritedType(self.ctx, loweredParameter, loweredRequirement.rawValue)
4550
}
4651

4752
if let nodeRequirements = node.genericWhereClause?.requirementList {
@@ -51,20 +56,22 @@ extension ASTGenVisitor {
5156
let firstType = self.visit(conformance.leftTypeIdentifier).rawValue
5257
let separatorLoc = self.base.advanced(by: conformance.colon.position.utf8Offset).raw
5358
let secondType = self.visit(conformance.rightTypeIdentifier).rawValue
54-
requirements.append(BridgedRequirementRepr(
55-
SeparatorLoc: separatorLoc,
56-
Kind: .typeConstraint,
57-
FirstType: firstType,
58-
SecondType: secondType))
59+
requirements.append(
60+
BridgedRequirementRepr(
61+
SeparatorLoc: separatorLoc,
62+
Kind: .typeConstraint,
63+
FirstType: firstType,
64+
SecondType: secondType))
5965
case .sameTypeRequirement(let sameType):
6066
let firstType = self.visit(sameType.leftTypeIdentifier).rawValue
6167
let separatorLoc = self.base.advanced(by: sameType.equalityToken.position.utf8Offset).raw
6268
let secondType = self.visit(sameType.rightTypeIdentifier).rawValue
63-
requirements.append(BridgedRequirementRepr(
64-
SeparatorLoc: separatorLoc,
65-
Kind: .sameType,
66-
FirstType: firstType,
67-
SecondType: secondType))
69+
requirements.append(
70+
BridgedRequirementRepr(
71+
SeparatorLoc: separatorLoc,
72+
Kind: .sameType,
73+
FirstType: firstType,
74+
SecondType: secondType))
6875
case .layoutRequirement(_):
6976
fatalError("Cannot handle layout requirements!")
7077
}

0 commit comments

Comments
 (0)