Skip to content

Commit 61c0e74

Browse files
StevenWong12StevenWong12
committed
Rename some SyntaxClassification 
1 parent 5a914f9 commit 61c0e74

File tree

5 files changed

+83
-73
lines changed

5 files changed

+83
-73
lines changed

‎Sources/SwiftIDEUtils/SwiftIDEUtilsCompatibility.swift‎

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,29 @@
1515

1616
public extension SyntaxClassification {
1717
/// A `#` keyword like `#warning`.
18-
@available(*, deprecated, renamed: "poundDirective")
19-
static var poundDirectiveKeyword: SyntaxClassification {
20-
return .poundDirective
18+
@available(*, deprecated, renamed: "ifConfigDirective")
19+
static var poundDirective: SyntaxClassification {
20+
return .ifConfigDirective
21+
}
22+
23+
@available(*, deprecated, renamed: "ifConfigDirective")
24+
static var buildConfigId: SyntaxClassification {
25+
return .ifConfigDirective
26+
}
27+
28+
@available(*, deprecated, message: "String interpolation anchors are now classified as .none")
29+
static var stringInterpolationAnchor: SyntaxClassification {
30+
return .none
31+
}
32+
33+
@available(*, deprecated, renamed: "type")
34+
static var typeIdentifier: SyntaxClassification {
35+
return .type
36+
}
37+
38+
@available(*, deprecated, renamed: "operator")
39+
static var operatorIdentifier: SyntaxClassification {
40+
return .operator
2141
}
2242
}
2343

‎Sources/SwiftIDEUtils/SyntaxClassification.swift‎

Lines changed: 18 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ public enum SyntaxClassification {
1717
case attribute
1818
/// A block comment starting with `/**` and ending with `*/.
1919
case blockComment
20-
/// A build configuration directive like `#if`, `#elseif`, `#else`.
21-
case buildConfigId
2220
/// A doc block comment starting with `/**` and ending with `*/.
2321
case docBlockComment
2422
/// A doc line comment starting with `///`.
@@ -31,6 +29,8 @@ public enum SyntaxClassification {
3129
case floatingLiteral
3230
/// A generic identifier.
3331
case identifier
32+
/// A directive starting with `#if`, `#elseif`or `#else`.
33+
case ifConfigDirective
3434
/// An integer literal.
3535
case integerLiteral
3636
/// A Swift keyword, including contextual keywords.
@@ -40,17 +40,13 @@ public enum SyntaxClassification {
4040
/// The token should not receive syntax coloring.
4141
case none
4242
/// An identifier referring to an operator.
43-
case operatorIdentifier
44-
/// A `#` token like `#warning`.
45-
case poundDirective
43+
case `operator`
4644
/// A regex literal, including multiline regex literals.
4745
case regexLiteral
48-
/// The opening and closing parenthesis of string interpolation.
49-
case stringInterpolationAnchor
5046
/// A string literal including multiline string literals.
5147
case stringLiteral
5248
/// An identifier referring to a type.
53-
case typeIdentifier
49+
case type
5450
}
5551

5652
extension SyntaxClassification {
@@ -71,26 +67,22 @@ extension SyntaxClassification {
7167
return (.keyword, false)
7268
case \DeclModifierSyntax.name:
7369
return (.attribute, false)
74-
case \ExpressionSegmentSyntax.leftParen:
75-
return (.stringInterpolationAnchor, true)
76-
case \ExpressionSegmentSyntax.rightParen:
77-
return (.stringInterpolationAnchor, true)
7870
case \IfConfigClauseSyntax.poundKeyword:
79-
return (.buildConfigId, false)
71+
return (.ifConfigDirective, false)
8072
case \IfConfigClauseSyntax.condition:
81-
return (.buildConfigId, false)
73+
return (.ifConfigDirective, false)
8274
case \IfConfigDeclSyntax.poundEndif:
83-
return (.buildConfigId, false)
75+
return (.ifConfigDirective, false)
8476
case \MemberTypeIdentifierSyntax.name:
85-
return (.typeIdentifier, false)
77+
return (.type, false)
8678
case \OperatorDeclSyntax.name:
87-
return (.operatorIdentifier, false)
79+
return (.operator, false)
8880
case \PrecedenceGroupAssociativitySyntax.associativityLabel:
8981
return (.keyword, false)
9082
case \PrecedenceGroupRelationSyntax.higherThanOrLowerThanLabel:
9183
return (.keyword, false)
9284
case \SimpleTypeIdentifierSyntax.name:
93-
return (.typeIdentifier, false)
85+
return (.type, false)
9486
default:
9587
return nil
9688
}
@@ -109,7 +101,7 @@ extension RawTokenKind {
109101
case .backtick:
110102
return .none
111103
case .binaryOperator:
112-
return .operatorIdentifier
104+
return .operator
113105
case .colon:
114106
return .none
115107
case .comma:
@@ -147,29 +139,29 @@ extension RawTokenKind {
147139
case .period:
148140
return .none
149141
case .postfixOperator:
150-
return .operatorIdentifier
142+
return .operator
151143
case .postfixQuestionMark:
152144
return .none
153145
case .pound:
154146
return .none
155147
case .poundAvailable:
156148
return .none
157149
case .poundElse:
158-
return .poundDirective
150+
return .ifConfigDirective
159151
case .poundElseif:
160-
return .poundDirective
152+
return .ifConfigDirective
161153
case .poundEndif:
162-
return .poundDirective
154+
return .ifConfigDirective
163155
case .poundIf:
164-
return .poundDirective
156+
return .ifConfigDirective
165157
case .poundSourceLocation:
166-
return .poundDirective
158+
return .keyword
167159
case .poundUnavailable:
168160
return .none
169161
case .prefixAmpersand:
170162
return .none
171163
case .prefixOperator:
172-
return .operatorIdentifier
164+
return .operator
173165
case .rawStringPoundDelimiter:
174166
return .none
175167
case .regexLiteralPattern:

‎Sources/SwiftParser/Expressions.swift‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1082,11 +1082,13 @@ extension Parser {
10821082
forDirective: Bool,
10831083
flavor: ExprFlavor
10841084
) -> RawExprSyntax {
1085+
/*
10851086
if forDirective == true,
10861087
let directiveExpr = self.parsePrimaryExprForDirective()
10871088
{
10881089
return RawExprSyntax(directiveExpr)
10891090
}
1091+
*/
10901092

10911093
switch self.at(anyIn: PrimaryExpressionStart.self) {
10921094
case (.integerLiteral, let handle)?:

‎Sources/lit-test-helper/ClassifiedSyntaxTreePrinter.swift‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,14 @@ extension SyntaxClassification {
1919
case .none: return ""
2020
case .keyword: return "kw"
2121
case .identifier: return "id"
22-
case .typeIdentifier: return "type"
22+
case .type: return "type"
2323
case .dollarIdentifier: return "dollar"
24-
case .operatorIdentifier: return "op"
24+
case .operator: return "op"
2525
case .integerLiteral: return "int"
2626
case .floatingLiteral: return "float"
2727
case .stringLiteral: return "str"
28-
case .stringInterpolationAnchor: return "anchor"
2928
case .regexLiteral: return "regex"
30-
case .poundDirective: return "#kw"
31-
case .buildConfigId: return "#id"
29+
case .ifConfigDirective: return "#kw"
3230
case .attribute: return "attr-builtin"
3331
case .editorPlaceholder: return "placeholder"
3432
case .lineComment: return "comment-line"

‎Tests/SwiftIDEUtilsTest/ClassificationTests.swift‎

Lines changed: 37 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public class ClassificationTests: XCTestCase {
9090
expected: [
9191
ClassificationSpec(source: "let", kind: .keyword),
9292
ClassificationSpec(source: "x", kind: .identifier),
93-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
93+
ClassificationSpec(source: "Int", kind: .type),
9494
]
9595
)
9696
}
@@ -103,11 +103,11 @@ public class ClassificationTests: XCTestCase {
103103
expected: [
104104
ClassificationSpec(source: "let", kind: .keyword),
105105
ClassificationSpec(source: "x", kind: .identifier),
106-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
106+
ClassificationSpec(source: "Int", kind: .type),
107107
ClassificationSpec(source: "4", kind: .integerLiteral),
108-
ClassificationSpec(source: "+", kind: .operatorIdentifier),
108+
ClassificationSpec(source: "+", kind: .operator),
109109
ClassificationSpec(source: "5", kind: .integerLiteral),
110-
ClassificationSpec(source: "/", kind: .operatorIdentifier),
110+
ClassificationSpec(source: "/", kind: .operator),
111111
ClassificationSpec(source: "6", kind: .integerLiteral),
112112
]
113113
)
@@ -117,7 +117,7 @@ public class ClassificationTests: XCTestCase {
117117
expected: [
118118
ClassificationSpec(source: "infix", kind: .keyword),
119119
ClassificationSpec(source: "operator", kind: .keyword),
120-
ClassificationSpec(source: "*--*", kind: .operatorIdentifier),
120+
ClassificationSpec(source: "*--*", kind: .operator),
121121
]
122122
)
123123
}
@@ -131,13 +131,13 @@ public class ClassificationTests: XCTestCase {
131131
ClassificationSpec(source: "func", kind: .keyword),
132132
ClassificationSpec(source: "foo", kind: .identifier),
133133
ClassificationSpec(source: "x", kind: .identifier),
134-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
134+
ClassificationSpec(source: "Int", kind: .type),
135135
ClassificationSpec(source: "y", kind: .identifier),
136-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
137-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
136+
ClassificationSpec(source: "Int", kind: .type),
137+
ClassificationSpec(source: "Int", kind: .type),
138138
ClassificationSpec(source: "return", kind: .keyword),
139139
ClassificationSpec(source: "x", kind: .identifier),
140-
ClassificationSpec(source: "+", kind: .operatorIdentifier),
140+
ClassificationSpec(source: "+", kind: .operator),
141141
ClassificationSpec(source: "y", kind: .identifier),
142142
]
143143
)
@@ -219,18 +219,18 @@ public class ClassificationTests: XCTestCase {
219219
#endif
220220
""",
221221
expected: [
222-
ClassificationSpec(source: "#if", kind: .poundDirective),
223-
ClassificationSpec(source: "os", kind: .buildConfigId),
224-
ClassificationSpec(source: "macOS", kind: .buildConfigId),
222+
ClassificationSpec(source: "#if", kind: .ifConfigDirective),
223+
ClassificationSpec(source: "os", kind: .ifConfigDirective),
224+
ClassificationSpec(source: "macOS", kind: .ifConfigDirective),
225225
ClassificationSpec(source: "var", kind: .keyword),
226226
ClassificationSpec(source: "x", kind: .identifier),
227-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
228-
ClassificationSpec(source: "#elseif", kind: .poundDirective),
227+
ClassificationSpec(source: "Int", kind: .type),
228+
ClassificationSpec(source: "#elseif", kind: .ifConfigDirective),
229229
ClassificationSpec(source: "var", kind: .keyword),
230230
ClassificationSpec(source: "x", kind: .identifier),
231-
ClassificationSpec(source: "Float", kind: .typeIdentifier),
232-
ClassificationSpec(source: "#else", kind: .poundDirective),
233-
ClassificationSpec(source: "#endif", kind: .poundDirective),
231+
ClassificationSpec(source: "Float", kind: .type),
232+
ClassificationSpec(source: "#else", kind: .ifConfigDirective),
233+
ClassificationSpec(source: "#endif", kind: .ifConfigDirective),
234234
]
235235
)
236236
}
@@ -242,10 +242,10 @@ public class ClassificationTests: XCTestCase {
242242
#else
243243
""",
244244
expected: [
245-
ClassificationSpec(source: "#if", kind: .poundDirective),
246-
ClassificationSpec(source: "!", kind: .operatorIdentifier),
247-
ClassificationSpec(source: "CONF", kind: .buildConfigId),
248-
ClassificationSpec(source: "#else", kind: .poundDirective),
245+
ClassificationSpec(source: "#if", kind: .ifConfigDirective),
246+
ClassificationSpec(source: "!", kind: .operator),
247+
ClassificationSpec(source: "CONF", kind: .ifConfigDirective),
248+
ClassificationSpec(source: "#else", kind: .ifConfigDirective),
249249
]
250250
)
251251
}
@@ -269,7 +269,7 @@ public class ClassificationTests: XCTestCase {
269269
assertClassification(
270270
#"#sourceLocation(file: "x", line: 1)"#,
271271
expected: [
272-
ClassificationSpec(source: "#sourceLocation", kind: .poundDirective),
272+
ClassificationSpec(source: "#sourceLocation", kind: .keyword),
273273
ClassificationSpec(source: "file", kind: .keyword),
274274
ClassificationSpec(source: #""x""#, kind: .stringLiteral),
275275
ClassificationSpec(source: "line", kind: .keyword),
@@ -330,7 +330,7 @@ public class ClassificationTests: XCTestCase {
330330
ClassificationSpec(source: "OSX", kind: .keyword),
331331
ClassificationSpec(source: "10", kind: .integerLiteral),
332332
ClassificationSpec(source: "10", kind: .integerLiteral),
333-
ClassificationSpec(source: "*", kind: .operatorIdentifier),
333+
ClassificationSpec(source: "*", kind: .operator),
334334
]
335335
)
336336
}
@@ -345,7 +345,7 @@ public class ClassificationTests: XCTestCase {
345345
ClassificationSpec(source: "@IBOutlet", kind: .attribute),
346346
ClassificationSpec(source: "var", kind: .keyword),
347347
ClassificationSpec(source: "foo", kind: .identifier),
348-
ClassificationSpec(source: "String", kind: .typeIdentifier),
348+
ClassificationSpec(source: "String", kind: .type),
349349
]
350350
)
351351
}
@@ -363,8 +363,8 @@ public class ClassificationTests: XCTestCase {
363363
ClassificationSpec(source: "indirect", kind: .keyword),
364364
ClassificationSpec(source: "case", kind: .keyword),
365365
ClassificationSpec(source: "cons", kind: .identifier),
366-
ClassificationSpec(source: "T", kind: .typeIdentifier),
367-
ClassificationSpec(source: "List", kind: .typeIdentifier),
366+
ClassificationSpec(source: "T", kind: .type),
367+
ClassificationSpec(source: "List", kind: .type),
368368
]
369369
)
370370
}
@@ -410,7 +410,7 @@ public class ClassificationTests: XCTestCase {
410410
expected: [
411411
ClassificationSpec(source: "infix", kind: .keyword),
412412
ClassificationSpec(source: "operator", kind: .keyword),
413-
ClassificationSpec(source: "*-*", kind: .operatorIdentifier),
413+
ClassificationSpec(source: "*-*", kind: .operator),
414414
ClassificationSpec(source: "FunnyPrecedence", kind: .identifier),
415415
]
416416
)
@@ -441,7 +441,7 @@ public class ClassificationTests: XCTestCase {
441441
expected: [
442442
ClassificationSpec(source: "protocol", kind: .keyword),
443443
ClassificationSpec(source: "Prot2", kind: .identifier),
444-
ClassificationSpec(source: "Prot", kind: .typeIdentifier),
444+
ClassificationSpec(source: "Prot", kind: .type),
445445
]
446446
)
447447
}
@@ -453,13 +453,13 @@ public class ClassificationTests: XCTestCase {
453453
ClassificationSpec(source: "func", kind: .keyword),
454454
ClassificationSpec(source: "genFn", kind: .identifier),
455455
ClassificationSpec(source: "T", kind: .identifier),
456-
ClassificationSpec(source: "Prot", kind: .typeIdentifier),
457-
ClassificationSpec(source: "T", kind: .typeIdentifier),
458-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
456+
ClassificationSpec(source: "Prot", kind: .type),
457+
ClassificationSpec(source: "T", kind: .type),
458+
ClassificationSpec(source: "Int", kind: .type),
459459
ClassificationSpec(source: "where", kind: .keyword),
460-
ClassificationSpec(source: "T", kind: .typeIdentifier),
461-
ClassificationSpec(source: "Blarg", kind: .typeIdentifier),
462-
ClassificationSpec(source: "Prot2", kind: .typeIdentifier),
460+
ClassificationSpec(source: "T", kind: .type),
461+
ClassificationSpec(source: "Blarg", kind: .type),
462+
ClassificationSpec(source: "Prot2", kind: .type),
463463
]
464464
)
465465
}
@@ -469,9 +469,7 @@ public class ClassificationTests: XCTestCase {
469469
#""This is string \(1) interpolation""#,
470470
expected: [
471471
ClassificationSpec(source: #""This is string "#, kind: .stringLiteral),
472-
ClassificationSpec(source: "(", kind: .stringInterpolationAnchor),
473472
ClassificationSpec(source: "1", kind: .integerLiteral),
474-
ClassificationSpec(source: ")", kind: .stringInterpolationAnchor),
475473
ClassificationSpec(source: #" interpolation""#, kind: .stringLiteral),
476474
]
477475
)
@@ -531,12 +529,12 @@ public class ClassificationTests: XCTestCase {
531529
ClassificationSpec(source: "func", kind: .keyword),
532530
ClassificationSpec(source: "keywordInCaseAndLocalArgLabel", kind: .identifier),
533531
ClassificationSpec(source: "for", kind: .identifier),
534-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
532+
ClassificationSpec(source: "Int", kind: .type),
535533
ClassificationSpec(source: "for", kind: .identifier),
536534
ClassificationSpec(source: "in", kind: .identifier),
537-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
535+
ClassificationSpec(source: "Int", kind: .type),
538536
ClassificationSpec(source: "class", kind: .identifier),
539-
ClassificationSpec(source: "Int", kind: .typeIdentifier),
537+
ClassificationSpec(source: "Int", kind: .type),
540538
]
541539
)
542540
}

0 commit comments

Comments
 (0)