Merge pull request #21343 from Microsoft/add2-semicolon-to-unused-class-members

Add semicolon to unused class member list
This commit is contained in:
Nathan Shively-Sanders 2018-01-22 13:28:07 -08:00 committed by GitHub
commit 5dcb937a48
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 32 additions and 0 deletions

View file

@ -21030,6 +21030,7 @@ namespace ts {
}
break;
case SyntaxKind.IndexSignature:
case SyntaxKind.SemicolonClassElement:
// Can't be private
break;
default:

View file

@ -0,0 +1,13 @@
//// [unusedSemicolonInClass.ts]
class Unused {
;
}
//// [unusedSemicolonInClass.js]
var Unused = /** @class */ (function () {
function Unused() {
}
;
return Unused;
}());

View file

@ -0,0 +1,7 @@
=== tests/cases/compiler/unusedSemicolonInClass.ts ===
class Unused {
>Unused : Symbol(Unused, Decl(unusedSemicolonInClass.ts, 0, 0))
;
}

View file

@ -0,0 +1,7 @@
=== tests/cases/compiler/unusedSemicolonInClass.ts ===
class Unused {
>Unused : Unused
;
}

View file

@ -0,0 +1,4 @@
// @noUnusedLocals: true
class Unused {
;
}