Skip to content

Commit 2165b3e

Browse files
committed
Fix method declaration detection when method name is string literal
Fixes #612
1 parent 349fb33 commit 2165b3e

File tree

5 files changed

+77
-9
lines changed

5 files changed

+77
-9
lines changed

TypeScript.YAML-tmLanguage

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -788,12 +788,12 @@ repository:
788788
endCaptures:
789789
'0': { name: punctuation.definition.block.ts }
790790
patterns:
791-
- include: "#string"
792791
- include: '#comment'
793792
- include: '#decorator'
794793
- include: '#method-declaration'
795794
- include: '#indexer-declaration'
796795
- include: '#field-declaration'
796+
- include: "#string"
797797
- include: '#type-annotation'
798798
- include: '#variable-initializer'
799799
- include: '#access-modifier'

TypeScript.tmLanguage

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2474,10 +2474,6 @@
24742474
</dict>
24752475
<key>patterns</key>
24762476
<array>
2477-
<dict>
2478-
<key>include</key>
2479-
<string>#string</string>
2480-
</dict>
24812477
<dict>
24822478
<key>include</key>
24832479
<string>#comment</string>
@@ -2498,6 +2494,10 @@
24982494
<key>include</key>
24992495
<string>#field-declaration</string>
25002496
</dict>
2497+
<dict>
2498+
<key>include</key>
2499+
<string>#string</string>
2500+
</dict>
25012501
<dict>
25022502
<key>include</key>
25032503
<string>#type-annotation</string>

TypeScriptReact.tmLanguage

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2478,10 +2478,6 @@
24782478
</dict>
24792479
<key>patterns</key>
24802480
<array>
2481-
<dict>
2482-
<key>include</key>
2483-
<string>#string</string>
2484-
</dict>
24852481
<dict>
24862482
<key>include</key>
24872483
<string>#comment</string>
@@ -2502,6 +2498,10 @@
25022498
<key>include</key>
25032499
<string>#field-declaration</string>
25042500
</dict>
2501+
<dict>
2502+
<key>include</key>
2503+
<string>#string</string>
2504+
</dict>
25052505
<dict>
25062506
<key>include</key>
25072507
<string>#type-annotation</string>

tests/baselines/Issue612.baseline.txt

Lines changed: 64 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,64 @@
1+
original file
2+
-----------------------------------
3+
class C {
4+
'm'?(): void;
5+
}
6+
const x = 0;
7+
-----------------------------------
8+
9+
Grammar: TypeScript.tmLanguage
10+
-----------------------------------
11+
>class C {
12+
^^^^^
13+
source.ts meta.class.ts storage.type.class.ts
14+
^
15+
source.ts meta.class.ts
16+
^
17+
source.ts meta.class.ts entity.name.type.class.ts
18+
^
19+
source.ts meta.class.ts
20+
^
21+
source.ts meta.class.ts punctuation.definition.block.ts
22+
> 'm'?(): void;
23+
^^^^
24+
source.ts meta.class.ts
25+
^
26+
source.ts meta.class.ts meta.method.declaration.ts string.quoted.single.ts punctuation.definition.string.begin.ts
27+
^
28+
source.ts meta.class.ts meta.method.declaration.ts string.quoted.single.ts
29+
^
30+
source.ts meta.class.ts meta.method.declaration.ts string.quoted.single.ts punctuation.definition.string.end.ts
31+
^
32+
source.ts meta.class.ts meta.method.declaration.ts keyword.operator.optional.ts
33+
^
34+
source.ts meta.class.ts meta.method.declaration.ts meta.parameters.ts punctuation.definition.parameters.begin.ts
35+
^
36+
source.ts meta.class.ts meta.method.declaration.ts meta.parameters.ts punctuation.definition.parameters.end.ts
37+
^
38+
source.ts meta.class.ts meta.method.declaration.ts meta.return.type.ts keyword.operator.type.annotation.ts
39+
^
40+
source.ts meta.class.ts meta.method.declaration.ts meta.return.type.ts
41+
^^^^
42+
source.ts meta.class.ts meta.method.declaration.ts meta.return.type.ts support.type.primitive.ts
43+
^
44+
source.ts meta.class.ts punctuation.terminator.statement.ts
45+
>}
46+
^
47+
source.ts meta.class.ts punctuation.definition.block.ts
48+
>const x = 0;
49+
^^^^^
50+
source.ts meta.var.expr.ts storage.type.ts
51+
^
52+
source.ts meta.var.expr.ts
53+
^
54+
source.ts meta.var.expr.ts meta.var-single-variable.expr.ts meta.definition.variable.ts variable.other.readwrite.ts
55+
^
56+
source.ts meta.var.expr.ts meta.var-single-variable.expr.ts
57+
^
58+
source.ts meta.var.expr.ts keyword.operator.assignment.ts
59+
^
60+
source.ts meta.var.expr.ts
61+
^
62+
source.ts meta.var.expr.ts constant.numeric.decimal.ts
63+
^
64+
source.ts punctuation.terminator.statement.ts

tests/cases/Issue612.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
class C {
2+
'm'?(): void;
3+
}
4+
const x = 0;

0 commit comments

Comments
 (0)