Skip to content

Commit 5dcb937

Browse files
authored
Merge pull request #21343 from Microsoft/add2-semicolon-to-unused-class-members
Add semicolon to unused class member list
2 parents 5887169 + 97fb0fd commit 5dcb937

File tree

5 files changed

+32
-0
lines changed

5 files changed

+32
-0
lines changed

src/compiler/checker.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21030,6 +21030,7 @@ namespace ts {
2103021030
}
2103121031
break;
2103221032
case SyntaxKind.IndexSignature:
21033+
case SyntaxKind.SemicolonClassElement:
2103321034
// Can't be private
2103421035
break;
2103521036
default:
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
//// [unusedSemicolonInClass.ts]
2+
class Unused {
3+
;
4+
}
5+
6+
7+
//// [unusedSemicolonInClass.js]
8+
var Unused = /** @class */ (function () {
9+
function Unused() {
10+
}
11+
;
12+
return Unused;
13+
}());
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
=== tests/cases/compiler/unusedSemicolonInClass.ts ===
2+
class Unused {
3+
>Unused : Symbol(Unused, Decl(unusedSemicolonInClass.ts, 0, 0))
4+
5+
;
6+
}
7+
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
=== tests/cases/compiler/unusedSemicolonInClass.ts ===
2+
class Unused {
3+
>Unused : Unused
4+
5+
;
6+
}
7+
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
// @noUnusedLocals: true
2+
class Unused {
3+
;
4+
}

0 commit comments

Comments
 (0)