Skip to content

Commit a882faf

Browse files
committed
Some additional deprecation fixes missed by the other cherrypicks.
1 parent 822d85e commit a882faf

File tree

3 files changed

+26
-27
lines changed

3 files changed

+26
-27
lines changed

Sources/SwiftFormat/PrettyPrint/TokenStreamCreator.swift

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -853,7 +853,7 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
853853
override func visit(_ node: TupleExprSyntax) -> SyntaxVisitorContinueKind {
854854
// We'll do nothing if it's a zero-element tuple, because we just want to keep the empty `()`
855855
// together.
856-
let elementCount = node.elementList.count
856+
let elementCount = node.elements.count
857857

858858
if elementCount == 1 {
859859
// A tuple with one element is a parenthesized expression; add a group around it to keep it
@@ -875,9 +875,9 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
875875
after(node.leftParen, tokens: .break(.open, size: 0), .open)
876876
before(node.rightParen, tokens: .break(.close, size: 0), .close)
877877

878-
insertTokens(.break(.same), betweenElementsOf: node.elementList)
878+
insertTokens(.break(.same), betweenElementsOf: node.elements)
879879

880-
for element in node.elementList {
880+
for element in node.elements {
881881
arrangeAsTupleExprElement(element)
882882
}
883883
}
@@ -2424,7 +2424,7 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
24242424
return .visitChildren
24252425
}
24262426

2427-
override func visit(_ node: AccessPathComponentSyntax) -> SyntaxVisitorContinueKind {
2427+
override func visit(_ node: ImportPathComponentSyntax) -> SyntaxVisitorContinueKind {
24282428
return .visitChildren
24292429
}
24302430

@@ -3300,7 +3300,7 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
33003300
private func startsWithOpenDelimiter(_ node: Syntax) -> Bool {
33013301
guard let token = node.firstToken(viewMode: .sourceAccurate) else { return false }
33023302
switch token.tokenKind {
3303-
case .leftBrace, .leftParen, .leftSquareBracket: return true
3303+
case .leftBrace, .leftParen, .leftSquare: return true
33043304
default: return false
33053305
}
33063306
}
@@ -3407,8 +3407,8 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
34073407
return true
34083408
case .tryExpr(let tryExpr):
34093409
return isCompoundExpression(tryExpr.expression)
3410-
case .tupleExpr(let tupleExpr) where tupleExpr.elementList.count == 1:
3411-
return isCompoundExpression(tupleExpr.elementList.first!.expression)
3410+
case .tupleExpr(let tupleExpr) where tupleExpr.elements.count == 1:
3411+
return isCompoundExpression(tupleExpr.elements.first!.expression)
34123412
default:
34133413
return false
34143414
}
@@ -3442,7 +3442,7 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
34423442
/// not parenthesized.
34433443
private func parenthesizedLeftmostExpr(of expr: ExprSyntax) -> TupleExprSyntax? {
34443444
switch Syntax(expr).as(SyntaxEnum.self) {
3445-
case .tupleExpr(let tupleExpr) where tupleExpr.elementList.count == 1:
3445+
case .tupleExpr(let tupleExpr) where tupleExpr.elements.count == 1:
34463446
return tupleExpr
34473447
case .infixOperatorExpr(let infixOperatorExpr):
34483448
return parenthesizedLeftmostExpr(of: infixOperatorExpr.leftOperand)
@@ -3604,7 +3604,7 @@ fileprivate final class TokenStreamCreator: SyntaxVisitor {
36043604
)
36053605
}
36063606

3607-
if let innerExpr = parenthesizedExpr.elementList.first?.expression,
3607+
if let innerExpr = parenthesizedExpr.elements.first?.expression,
36083608
let stringLiteralExpr = innerExpr.as(StringLiteralExprSyntax.self),
36093609
stringLiteralExpr.openingQuote.tokenKind == .multilineStringQuote
36103610
{

Sources/SwiftFormat/Rules/ReturnVoidInsteadOfEmptyTuple.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@ public final class ReturnVoidInsteadOfEmptyTuple: SyntaxFormatRule {
3939
return super.visit(node)
4040
}
4141

42-
// Make sure that function types nested in the argument list are also rewritten (for example,
42+
// Make sure that function types nested in the parameter list are also rewritten (for example,
4343
// `(Int -> ()) -> ()` should become `(Int -> Void) -> Void`).
44-
let arguments = visit(node.arguments)
44+
let parameters = visit(node.parameters)
4545
let voidKeyword = makeVoidIdentifierType(toReplace: returnType)
4646
var rewrittenNode = node
4747
rewrittenNode.parameters = parameters

Sources/SwiftFormat/Rules/UseShorthandTypeNames.swift

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,7 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
154154
valueType: typeArguments.1.argument,
155155
leftSquare: TokenSyntax.leftSquareToken(leadingTrivia: leadingTrivia),
156156
colon: TokenSyntax.colonToken(trailingTrivia: .spaces(1)),
157-
rightSquareBracket:
158-
TokenSyntax.rightSquareBracketToken(trailingTrivia: trailingTrivia))
157+
rightSquare: TokenSyntax.rightSquareToken(trailingTrivia: trailingTrivia))
159158
newNode = ExprSyntax(dictTypeExpr)
160159

161160
case "Optional":
@@ -267,18 +266,18 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
267266
/// have a valid expression representation.
268267
private func makeArrayTypeExpression(
269268
elementType: TypeSyntax,
270-
leftSquareBracket: TokenSyntax,
271-
rightSquareBracket: TokenSyntax
269+
leftSquare: TokenSyntax,
270+
rightSquare: TokenSyntax
272271
) -> ArrayExprSyntax? {
273272
guard let elementTypeExpr = expressionRepresentation(of: elementType) else {
274273
return nil
275274
}
276275
return ArrayExprSyntax(
277-
leftSquare: leftSquareBracket,
276+
leftSquare: leftSquare,
278277
elements: ArrayElementListSyntax([
279278
ArrayElementSyntax(expression: elementTypeExpr, trailingComma: nil),
280279
]),
281-
rightSquare: rightSquareBracket)
280+
rightSquare: rightSquare)
282281
}
283282

284283
/// Returns a `DictionaryExprSyntax` whose single key/value pair is the expression representations
@@ -287,9 +286,9 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
287286
private func makeDictionaryTypeExpression(
288287
keyType: TypeSyntax,
289288
valueType: TypeSyntax,
290-
leftSquareBracket: TokenSyntax,
289+
leftSquare: TokenSyntax,
291290
colon: TokenSyntax,
292-
rightSquareBracket: TokenSyntax
291+
rightSquare: TokenSyntax
293292
) -> DictionaryExprSyntax? {
294293
guard
295294
let keyTypeExpr = expressionRepresentation(of: keyType),
@@ -305,9 +304,9 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
305304
trailingComma: nil),
306305
])
307306
return DictionaryExprSyntax(
308-
leftSquare: leftSquareBracket,
307+
leftSquare: leftSquare,
309308
content: .elements(dictElementList),
310-
rightSquare: rightSquareBracket)
309+
rightSquare: rightSquare)
311310
}
312311

313312
/// Returns an `OptionalChainingExprSyntax` whose wrapped expression is the expression
@@ -417,7 +416,7 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
417416
valueType: dictionaryType.value,
418417
leftSquare: dictionaryType.leftSquare,
419418
colon: dictionaryType.colon,
420-
rightSquareBracket: dictionaryType.rightSquareBracket)
419+
rightSquare: dictionaryType.rightSquare)
421420
return ExprSyntax(result)
422421

423422
case .optionalType(let optionalType):
@@ -430,7 +429,7 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
430429
case .functionType(let functionType):
431430
let result = makeFunctionTypeExpression(
432431
leftParen: functionType.leftParen,
433-
argumentTypes: functionType.arguments,
432+
parameters: functionType.parameters,
434433
rightParen: functionType.rightParen,
435434
effectSpecifiers: functionType.effectSpecifiers,
436435
arrow: functionType.returnClause.arrow,
@@ -442,7 +441,7 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
442441
guard let elementExprs = expressionRepresentation(of: tupleType.elements) else { return nil }
443442
let result = TupleExprSyntax(
444443
leftParen: tupleType.leftParen,
445-
elementList: elementExprs,
444+
elements: elementExprs,
446445
rightParen: tupleType.rightParen)
447446
return ExprSyntax(result)
448447

@@ -474,22 +473,22 @@ public final class UseShorthandTypeNames: SyntaxFormatRule {
474473

475474
private func makeFunctionTypeExpression(
476475
leftParen: TokenSyntax,
477-
argumentTypes: TupleTypeElementListSyntax,
476+
parameters: TupleTypeElementListSyntax,
478477
rightParen: TokenSyntax,
479478
effectSpecifiers: TypeEffectSpecifiersSyntax?,
480479
arrow: TokenSyntax,
481480
returnType: TypeSyntax
482481
) -> SequenceExprSyntax? {
483482
guard
484-
let argumentTypeExprs = expressionRepresentation(of: argumentTypes),
483+
let parameterExprs = expressionRepresentation(of: parameters),
485484
let returnTypeExpr = expressionRepresentation(of: returnType)
486485
else {
487486
return nil
488487
}
489488

490489
let tupleExpr = TupleExprSyntax(
491490
leftParen: leftParen,
492-
elementList: argumentTypeExprs,
491+
elements: parameterExprs,
493492
rightParen: rightParen)
494493
let arrowExpr = ArrowExprSyntax(
495494
effectSpecifiers: effectSpecifiers,

0 commit comments

Comments
 (0)