Skip to content

Commit 17a29ff

Browse files
uhyouhyo
committed
revert unnecessary change
1 parent cb326d2 commit 17a29ff

File tree

3 files changed

+39
-49
lines changed

3 files changed

+39
-49
lines changed

‎src/compiler/checker.ts

+9-19
Original file line numberDiff line numberDiff line change
@@ -5969,29 +5969,19 @@ namespace ts {
59695969
return parameterNode;
59705970

59715971
function cloneBindingName(node: BindingName): BindingName {
5972-
return elideInitializerAndPropertyRenamingAndSetEmitFlags(node) as BindingName;
5973-
function elideInitializerAndPropertyRenamingAndSetEmitFlags(node: Node): Node {
5972+
return elideInitializerAndSetEmitFlags(node) as BindingName;
5973+
function elideInitializerAndSetEmitFlags(node: Node): Node {
59745974
if (context.tracker.trackSymbol && isComputedPropertyName(node) && isLateBindableName(node)) {
59755975
trackComputedName(node.expression, context.enclosingDeclaration, context);
59765976
}
5977-
let visited = visitEachChild(node, elideInitializerAndPropertyRenamingAndSetEmitFlags, nullTransformationContext, /*nodesVisitor*/ undefined, elideInitializerAndPropertyRenamingAndSetEmitFlags)!;
5977+
let visited = visitEachChild(node, elideInitializerAndSetEmitFlags, nullTransformationContext, /*nodesVisitor*/ undefined, elideInitializerAndSetEmitFlags)!;
59785978
if (isBindingElement(visited)) {
5979-
if (visited.propertyName && isIdentifier(visited.propertyName) && isIdentifier(visited.name)) {
5980-
visited = factory.updateBindingElement(
5981-
visited,
5982-
visited.dotDotDotToken,
5983-
/* propertyName*/ undefined,
5984-
visited.propertyName,
5985-
/*initializer*/ undefined);
5986-
}
5987-
else {
5988-
visited = factory.updateBindingElement(
5989-
visited,
5990-
visited.dotDotDotToken,
5991-
visited.propertyName,
5992-
visited.name,
5993-
/*initializer*/ undefined);
5994-
}
5979+
visited = factory.updateBindingElement(
5980+
visited,
5981+
visited.dotDotDotToken,
5982+
visited.propertyName,
5983+
visited.name,
5984+
/*initializer*/ undefined);
59955985
}
59965986
if (!nodeIsSynthesized(visited)) {
59975987
visited = factory.cloneNode(visited);

‎tests/baselines/reference/renamingDestructuredPropertyInFunctionType.types

+15-15
Original file line numberDiff line numberDiff line change
@@ -10,45 +10,45 @@ type F1 = (arg: number) => any; // OK
1010
>arg : number
1111

1212
type F2 = ({ a: string }: O) => any; // Error
13-
>F2 : ({ a }: O) => any
13+
>F2 : ({ a: string }: O) => any
1414
>a : any
1515
>string : string
1616

1717
type F3 = ({ a: string, b, c }: O) => any; // Error
18-
>F3 : ({ a, b, c }: O) => any
18+
>F3 : ({ a: string, b, c }: O) => any
1919
>a : any
2020
>string : string
2121
>b : number
2222
>c : number
2323

2424
type F4 = ({ a: string }: O) => any; // Error
25-
>F4 : ({ a }: O) => any
25+
>F4 : ({ a: string }: O) => any
2626
>a : any
2727
>string : string
2828

2929
type F5 = ({ a: string, b, c }: O) => any; // Error
30-
>F5 : ({ a, b, c }: O) => any
30+
>F5 : ({ a: string, b, c }: O) => any
3131
>a : any
3232
>string : string
3333
>b : number
3434
>c : number
3535

3636
type F6 = ({ a: string }) => typeof string; // OK
37-
>F6 : ({ a }: { a: any; }) => any
37+
>F6 : ({ a: string }: { a: any; }) => any
3838
>a : any
3939
>string : any
4040
>string : any
4141

4242
type F7 = ({ a: string, b: number }) => typeof number; // Error
43-
>F7 : ({ a, b }: { a: any; b: any; }) => any
43+
>F7 : ({ a: string, b: number }: { a: any; b: any; }) => any
4444
>a : any
4545
>string : any
4646
>b : any
4747
>number : any
4848
>number : any
4949

5050
type F8 = ({ a, b: number }) => typeof number; // OK
51-
>F8 : ({ a, b }: { a: any; b: any; }) => any
51+
>F8 : ({ a, b: number }: { a: any; b: any; }) => any
5252
>a : any
5353
>b : any
5454
>number : any
@@ -65,45 +65,45 @@ type G1 = (arg: number) => any; // OK
6565
>arg : number
6666

6767
type G2 = ({ a: string }: O) => any; // Error
68-
>G2 : ({ a }: O) => any
68+
>G2 : ({ a: string }: O) => any
6969
>a : any
7070
>string : string
7171

7272
type G3 = ({ a: string, b, c }: O) => any; // Error
73-
>G3 : ({ a, b, c }: O) => any
73+
>G3 : ({ a: string, b, c }: O) => any
7474
>a : any
7575
>string : string
7676
>b : number
7777
>c : number
7878

7979
type G4 = ({ a: string }: O) => any; // Error
80-
>G4 : ({ a }: O) => any
80+
>G4 : ({ a: string }: O) => any
8181
>a : any
8282
>string : string
8383

8484
type G5 = ({ a: string, b, c }: O) => any; // Error
85-
>G5 : ({ a, b, c }: O) => any
85+
>G5 : ({ a: string, b, c }: O) => any
8686
>a : any
8787
>string : string
8888
>b : number
8989
>c : number
9090

9191
type G6 = ({ a: string }) => typeof string; // OK
92-
>G6 : ({ a }: { a: any; }) => any
92+
>G6 : ({ a: string }: { a: any; }) => any
9393
>a : any
9494
>string : any
9595
>string : any
9696

9797
type G7 = ({ a: string, b: number }) => typeof number; // Error
98-
>G7 : ({ a, b }: { a: any; b: any; }) => any
98+
>G7 : ({ a: string, b: number }: { a: any; b: any; }) => any
9999
>a : any
100100
>string : any
101101
>b : any
102102
>number : any
103103
>number : any
104104

105105
type G8 = ({ a, b: number }) => typeof number; // OK
106-
>G8 : ({ a, b }: { a: any; b: any; }) => any
106+
>G8 : ({ a, b: number }: { a: any; b: any; }) => any
107107
>a : any
108108
>b : any
109109
>number : any
@@ -121,7 +121,7 @@ interface I {
121121
>arg : number
122122

123123
method2({ a: string }): any; // Error
124-
>method2 : ({ a }: { a: any; }) => any
124+
>method2 : ({ a: string }: { a: any; }) => any
125125
>a : any
126126
>string : any
127127

‎tests/baselines/reference/renamingDestructuredPropertyInFunctionType2.types

+15-15
Original file line numberDiff line numberDiff line change
@@ -10,45 +10,45 @@ type F1 = (arg: number) => any;
1010
>arg : number
1111

1212
type F2 = ({ a: string }: O) => any;
13-
>F2 : ({ a }: O) => any
13+
>F2 : ({ a: string }: O) => any
1414
>a : any
1515
>string : string
1616

1717
type F3 = ({ a: string, b, c }: O) => any;
18-
>F3 : ({ a, b, c }: O) => any
18+
>F3 : ({ a: string, b, c }: O) => any
1919
>a : any
2020
>string : string
2121
>b : number
2222
>c : number
2323

2424
type F4 = ({ a: string }: O) => any;
25-
>F4 : ({ a }: O) => any
25+
>F4 : ({ a: string }: O) => any
2626
>a : any
2727
>string : string
2828

2929
type F5 = ({ a: string, b, c }: O) => any;
30-
>F5 : ({ a, b, c }: O) => any
30+
>F5 : ({ a: string, b, c }: O) => any
3131
>a : any
3232
>string : string
3333
>b : number
3434
>c : number
3535

3636
type F6 = ({ a: string }) => typeof string;
37-
>F6 : ({ a }: { a: any; }) => any
37+
>F6 : ({ a: string }: { a: any; }) => any
3838
>a : any
3939
>string : any
4040
>string : any
4141

4242
type F7 = ({ a: string, b: number }) => typeof number;
43-
>F7 : ({ a, b }: { a: any; b: any; }) => any
43+
>F7 : ({ a: string, b: number }: { a: any; b: any; }) => any
4444
>a : any
4545
>string : any
4646
>b : any
4747
>number : any
4848
>number : any
4949

5050
type F8 = ({ a, b: number }) => typeof number;
51-
>F8 : ({ a, b }: { a: any; b: any; }) => any
51+
>F8 : ({ a, b: number }: { a: any; b: any; }) => any
5252
>a : any
5353
>b : any
5454
>number : any
@@ -65,45 +65,45 @@ type G1 = (arg: number) => any;
6565
>arg : number
6666

6767
type G2 = ({ a: string }: O) => any;
68-
>G2 : ({ a }: O) => any
68+
>G2 : ({ a: string }: O) => any
6969
>a : any
7070
>string : string
7171

7272
type G3 = ({ a: string, b, c }: O) => any;
73-
>G3 : ({ a, b, c }: O) => any
73+
>G3 : ({ a: string, b, c }: O) => any
7474
>a : any
7575
>string : string
7676
>b : number
7777
>c : number
7878

7979
type G4 = ({ a: string }: O) => any;
80-
>G4 : ({ a }: O) => any
80+
>G4 : ({ a: string }: O) => any
8181
>a : any
8282
>string : string
8383

8484
type G5 = ({ a: string, b, c }: O) => any;
85-
>G5 : ({ a, b, c }: O) => any
85+
>G5 : ({ a: string, b, c }: O) => any
8686
>a : any
8787
>string : string
8888
>b : number
8989
>c : number
9090

9191
type G6 = ({ a: string }) => typeof string;
92-
>G6 : ({ a }: { a: any; }) => any
92+
>G6 : ({ a: string }: { a: any; }) => any
9393
>a : any
9494
>string : any
9595
>string : any
9696

9797
type G7 = ({ a: string, b: number }) => typeof number;
98-
>G7 : ({ a, b }: { a: any; b: any; }) => any
98+
>G7 : ({ a: string, b: number }: { a: any; b: any; }) => any
9999
>a : any
100100
>string : any
101101
>b : any
102102
>number : any
103103
>number : any
104104

105105
type G8 = ({ a, b: number }) => typeof number;
106-
>G8 : ({ a, b }: { a: any; b: any; }) => any
106+
>G8 : ({ a, b: number }: { a: any; b: any; }) => any
107107
>a : any
108108
>b : any
109109
>number : any
@@ -121,7 +121,7 @@ interface I {
121121
>arg : number
122122

123123
method2({ a: string }): any;
124-
>method2 : ({ a }: { a: any; }) => any
124+
>method2 : ({ a: string }: { a: any; }) => any
125125
>a : any
126126
>string : any
127127

0 commit comments

Comments
 (0)