Commit 0cf100d
authored
Add constructor functions to aliasable expressions (microsoft#36108)
* Add constructor functions to aliasable expressions
Fixes microsoft#35228, at least the crash. There are still a couple of errors
that are probably incorrect.
Note that isJSConstructor relies on parent pointers and the ability to
merge symbols, so I had to move isAliasSymbolDeclaration (back?) to the
checker.
* add simple test case
* remove errors in test
* fix bad merge1 parent 49282d9 commit 0cf100dCopy full SHA for 0cf100d
File tree
Expand file treeCollapse file tree
11 files changed
+294
-30
lines changedOpen diff view settings
Filter options
- src/compiler
- tests
- baselines/reference
- cases/conformance/jsdoc/declarations
Expand file treeCollapse file tree
11 files changed
+294
-30
lines changedOpen diff view settings
Collapse file
+39-2Lines changed: 39 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
2181 | 2181 | |
2182 | 2182 | |
2183 | 2183 | |
| 2184 | + |
| 2185 | + |
| 2186 | + |
| 2187 | + |
| 2188 | + |
| 2189 | + |
| 2190 | + |
| 2191 | + |
| 2192 | + |
| 2193 | + |
| 2194 | + |
| 2195 | + |
| 2196 | + |
| 2197 | + |
| 2198 | + |
| 2199 | + |
| 2200 | + |
| 2201 | + |
| 2202 | + |
| 2203 | + |
| 2204 | + |
| 2205 | + |
| 2206 | + |
| 2207 | + |
| 2208 | + |
| 2209 | + |
| 2210 | + |
| 2211 | + |
| 2212 | + |
| 2213 | + |
| 2214 | + |
| 2215 | + |
| 2216 | + |
| 2217 | + |
| 2218 | + |
| 2219 | + |
| 2220 | + |
2184 | 2221 | |
2185 | 2222 | |
2186 | 2223 | |
| ||
5784 | 5821 | |
5785 | 5822 | |
5786 | 5823 | |
5787 | | - |
5788 | | - |
| 5824 | + |
| 5825 | + |
5789 | 5826 | |
5790 | 5827 | |
5791 | 5828 | |
|
Collapse file
+1-28Lines changed: 1 addition & 28 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
2738 | 2738 | |
2739 | 2739 | |
2740 | 2740 | |
2741 | | - |
2742 | | - |
2743 | | - |
2744 | | - |
2745 | | - |
2746 | | - |
2747 | | - |
2748 | | - |
2749 | | - |
2750 | | - |
2751 | | - |
2752 | | - |
2753 | | - |
2754 | | - |
2755 | | - |
2756 | | - |
2757 | | - |
2758 | | - |
2759 | | - |
2760 | | - |
2761 | | - |
2762 | | - |
2763 | | - |
2764 | | - |
2765 | | - |
2766 | | - |
2767 | | - |
2768 | 2741 | |
2769 | 2742 | |
2770 | 2743 | |
| ||
2775 | 2748 | |
2776 | 2749 | |
2777 | 2750 | |
2778 | | - |
| 2751 | + |
2779 | 2752 | |
2780 | 2753 | |
2781 | 2754 | |
|
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunction.js
Copy file name to clipboard+24Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunction.symbols
Copy file name to clipboard+28Lines changed: 28 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunction.types
Copy file name to clipboard+35Lines changed: 35 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
| 29 | + |
| 30 | + |
| 31 | + |
| 32 | + |
| 33 | + |
| 34 | + |
| 35 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunctionWithSub.errors.txt
Copy file name to clipboard+23Lines changed: 23 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunctionWithSub.js
Copy file name to clipboard+24Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunctionWithSub.symbols
Copy file name to clipboard+38Lines changed: 38 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
| 29 | + |
| 30 | + |
| 31 | + |
| 32 | + |
| 33 | + |
| 34 | + |
| 35 | + |
| 36 | + |
| 37 | + |
| 38 | + |
Collapse file
tests/baselines/reference/jsDeclarationsExportAssignedConstructorFunctionWithSub.types
Copy file name to clipboard+52Lines changed: 52 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
| 29 | + |
| 30 | + |
| 31 | + |
| 32 | + |
| 33 | + |
| 34 | + |
| 35 | + |
| 36 | + |
| 37 | + |
| 38 | + |
| 39 | + |
| 40 | + |
| 41 | + |
| 42 | + |
| 43 | + |
| 44 | + |
| 45 | + |
| 46 | + |
| 47 | + |
| 48 | + |
| 49 | + |
| 50 | + |
| 51 | + |
| 52 | + |
Collapse file
tests/cases/conformance/jsdoc/declarations/jsDeclarationsExportAssignedConstructorFunction.ts
Copy file name to clipboard+14Lines changed: 14 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
0 commit comments