Skip to content

Commit 3eda9c6

Browse files
author
Andy
authored
Make node type predicates public (#16121)
* Make node type predicates public * Rename isJSDocComment back to isJSDoc
1 parent 1cc860d commit 3eda9c6

File tree

5 files changed

+1500
-999
lines changed

5 files changed

+1500
-999
lines changed

src/compiler/checker.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21082,10 +21082,6 @@ namespace ts {
2108221082
}
2108321083
}
2108421084

21085-
function isAccessor(kind: SyntaxKind): boolean {
21086-
return kind === SyntaxKind.GetAccessor || kind === SyntaxKind.SetAccessor;
21087-
}
21088-
2108921085
function checkInheritedPropertiesAreIdentical(type: InterfaceType, typeNode: Node): boolean {
2109021086
const baseTypes = getBaseTypes(type);
2109121087
if (baseTypes.length < 2) {
@@ -24555,7 +24551,7 @@ namespace ts {
2455524551
function checkGrammarStatementInAmbientContext(node: Node): boolean {
2455624552
if (isInAmbientContext(node)) {
2455724553
// An accessors is already reported about the ambient context
24558-
if (isAccessor(node.parent.kind)) {
24554+
if (isAccessor(node.parent)) {
2455924555
return getNodeLinks(node).hasReportedStatementInAmbientContext = true;
2456024556
}
2456124557

src/compiler/factory.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3908,7 +3908,7 @@ namespace ts {
39083908
return bindingElement.right;
39093909
}
39103910

3911-
if (isSpreadExpression(bindingElement)) {
3911+
if (isSpreadElement(bindingElement)) {
39123912
// Recovery consistent with existing emit.
39133913
return getInitializerOfBindingOrAssignmentElement(<BindingOrAssignmentElement>bindingElement.expression);
39143914
}
@@ -3976,7 +3976,7 @@ namespace ts {
39763976
return getTargetOfBindingOrAssignmentElement(<BindingOrAssignmentElement>bindingElement.left);
39773977
}
39783978

3979-
if (isSpreadExpression(bindingElement)) {
3979+
if (isSpreadElement(bindingElement)) {
39803980
// `a` in `[...a] = ...`
39813981
return getTargetOfBindingOrAssignmentElement(<BindingOrAssignmentElement>bindingElement.expression);
39823982
}

src/compiler/transformers/es2015.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3396,7 +3396,7 @@ namespace ts {
33963396
else {
33973397
if (segments.length === 1) {
33983398
const firstElement = elements[0];
3399-
return needsUniqueCopy && isSpreadExpression(firstElement) && firstElement.expression.kind !== SyntaxKind.ArrayLiteralExpression
3399+
return needsUniqueCopy && isSpreadElement(firstElement) && firstElement.expression.kind !== SyntaxKind.ArrayLiteralExpression
34003400
? createArraySlice(segments[0])
34013401
: segments[0];
34023402
}
@@ -3407,7 +3407,7 @@ namespace ts {
34073407
}
34083408

34093409
function partitionSpread(node: Expression) {
3410-
return isSpreadExpression(node)
3410+
return isSpreadElement(node)
34113411
? visitSpanOfSpreads
34123412
: visitSpanOfNonSpreads;
34133413
}

src/compiler/transformers/module/system.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1502,7 +1502,7 @@ namespace ts {
15021502
if (isAssignmentExpression(node, /*excludeCompoundAssignment*/ true)) {
15031503
return hasExportedReferenceInDestructuringTarget(node.left);
15041504
}
1505-
else if (isSpreadExpression(node)) {
1505+
else if (isSpreadElement(node)) {
15061506
return hasExportedReferenceInDestructuringTarget(node.expression);
15071507
}
15081508
else if (isObjectLiteralExpression(node)) {

0 commit comments

Comments
 (0)