Skip to content

Commit c8327da

Browse files
committed
Remove unnecessary getModifiedTime
1 parent 59ad6ef commit c8327da

File tree

52 files changed

+183
-184
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+183
-184
lines changed

src/compiler/tsbuildPublic.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1412,8 +1412,7 @@ namespace ts {
14121412
// had its file touched but not had its contents changed - this allows us
14131413
// to skip a downstream typecheck
14141414
if (isDeclarationFileName(output)) {
1415-
const outputModifiedTime = getModifiedTime(host, output);
1416-
newestDeclarationFileContentChangedTime = newer(newestDeclarationFileContentChangedTime, outputModifiedTime);
1415+
newestDeclarationFileContentChangedTime = newer(newestDeclarationFileContentChangedTime, outputTime);
14171416
}
14181417
}
14191418
}

tests/baselines/reference/tsbuild/amdModulesWithOut/modules-and-globals-mixed-in-amd.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1013,7 +1013,7 @@ getModifiedTime:: {
10131013
"/src/app/file4.ts": 1,
10141014
"/src/app/module.js": 1,
10151015
"/src/app/module.js.map": 1,
1016-
"/src/app/module.d.ts": 3,
1016+
"/src/app/module.d.ts": 2,
10171017
"/src/app/module.d.ts.map": 1,
10181018
"/src/app/module.tsbuildinfo": 1,
10191019
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/amdModulesWithOut/multiple-emitHelpers-in-all-projects.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2190,7 +2190,7 @@ getModifiedTime:: {
21902190
"/src/app/file4.ts": 1,
21912191
"/src/app/module.js": 1,
21922192
"/src/app/module.js.map": 1,
2193-
"/src/app/module.d.ts": 3,
2193+
"/src/app/module.d.ts": 2,
21942194
"/src/app/module.d.ts.map": 1,
21952195
"/src/app/module.tsbuildinfo": 1,
21962196
"/src/app/tsconfig.json": 1
@@ -3842,7 +3842,7 @@ getModifiedTime:: {
38423842
"/src/app/file4.ts": 1,
38433843
"/src/app/module.js": 1,
38443844
"/src/app/module.js.map": 1,
3845-
"/src/app/module.d.ts": 3,
3845+
"/src/app/module.d.ts": 2,
38463846
"/src/app/module.d.ts.map": 1,
38473847
"/src/app/module.tsbuildinfo": 1,
38483848
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/amdModulesWithOut/multiple-prologues-in-all-projects.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1299,7 +1299,7 @@ getModifiedTime:: {
12991299
"/src/app/file4.ts": 1,
13001300
"/src/app/module.js": 1,
13011301
"/src/app/module.js.map": 1,
1302-
"/src/app/module.d.ts": 3,
1302+
"/src/app/module.d.ts": 2,
13031303
"/src/app/module.d.ts.map": 1,
13041304
"/src/app/module.tsbuildinfo": 1,
13051305
"/src/app/tsconfig.json": 1
@@ -2223,7 +2223,7 @@ getModifiedTime:: {
22232223
"/src/app/file4.ts": 1,
22242224
"/src/app/module.js": 1,
22252225
"/src/app/module.js.map": 1,
2226-
"/src/app/module.d.ts": 3,
2226+
"/src/app/module.d.ts": 2,
22272227
"/src/app/module.d.ts.map": 1,
22282228
"/src/app/module.tsbuildinfo": 1,
22292229
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/amdModulesWithOut/shebang-in-all-projects.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1035,7 +1035,7 @@ getModifiedTime:: {
10351035
"/src/app/file4.ts": 1,
10361036
"/src/app/module.js": 1,
10371037
"/src/app/module.js.map": 1,
1038-
"/src/app/module.d.ts": 3,
1038+
"/src/app/module.d.ts": 2,
10391039
"/src/app/module.d.ts.map": 1,
10401040
"/src/app/module.tsbuildinfo": 1,
10411041
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/amdModulesWithOut/stripInternal.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4940,7 +4940,7 @@ getModifiedTime:: {
49404940
"/src/app/file4.ts": 1,
49414941
"/src/app/module.js": 1,
49424942
"/src/app/module.js.map": 1,
4943-
"/src/app/module.d.ts": 3,
4943+
"/src/app/module.d.ts": 2,
49444944
"/src/app/module.d.ts.map": 1,
49454945
"/src/app/module.tsbuildinfo": 1,
49464946
"/src/app/tsconfig.json": 1
@@ -8781,7 +8781,7 @@ getModifiedTime:: {
87818781
"/src/app/file4.ts": 1,
87828782
"/src/app/module.js": 1,
87838783
"/src/app/module.js.map": 1,
8784-
"/src/app/module.d.ts": 2,
8784+
"/src/app/module.d.ts": 1,
87858785
"/src/app/module.d.ts.map": 1,
87868786
"/src/app/module.tsbuildinfo": 1,
87878787
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/amdModulesWithOut/triple-slash-refs-in-all-projects.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1252,7 +1252,7 @@ getModifiedTime:: {
12521252
"/src/app/tripleRef.d.ts": 1,
12531253
"/src/app/module.js": 1,
12541254
"/src/app/module.js.map": 1,
1255-
"/src/app/module.d.ts": 3,
1255+
"/src/app/module.d.ts": 2,
12561256
"/src/app/module.d.ts.map": 1,
12571257
"/src/app/module.tsbuildinfo": 1,
12581258
"/src/app/tsconfig.json": 1

tests/baselines/reference/tsbuild/containerOnlyReferenced/verify-that-subsequent-builds-after-initial-build-doesnt-build-anything.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,17 +289,17 @@ exitCode:: ExitStatus.Success
289289
getModifiedTime:: {
290290
"/src/src/folder/index.ts": 1,
291291
"/src/src/folder/index.js": 1,
292-
"/src/src/folder/index.d.ts": 2,
292+
"/src/src/folder/index.d.ts": 1,
293293
"/src/src/folder/tsconfig.tsbuildinfo": 1,
294294
"/src/src/folder/tsconfig.json": 1,
295295
"/src/src/folder2/index.ts": 1,
296296
"/src/src/folder2/index.js": 1,
297-
"/src/src/folder2/index.d.ts": 2,
297+
"/src/src/folder2/index.d.ts": 1,
298298
"/src/src/folder2/tsconfig.tsbuildinfo": 1,
299299
"/src/src/folder2/tsconfig.json": 1,
300300
"/src/tests/index.ts": 1,
301301
"/src/tests/index.js": 1,
302-
"/src/tests/index.d.ts": 2,
302+
"/src/tests/index.d.ts": 1,
303303
"/src/tests/tsconfig.tsbuildinfo": 1,
304304
"/src/tests/tsconfig.json": 1
305305
}

tests/baselines/reference/tsbuild/javascriptProjectEmit/modifies-outfile-js-projects-and-concatenates-them-correctly.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ exitCode:: ExitStatus.Success
513513
getModifiedTime:: {
514514
"/src/common/nominal.js": 1,
515515
"/src/common/common.js": 1,
516-
"/src/common/common.d.ts": 2,
516+
"/src/common/common.d.ts": 1,
517517
"/src/common/common.tsbuildinfo": 1,
518518
"/src/common/tsconfig.json": 1,
519519
"/src/tsconfig.base.json": 2,
@@ -522,7 +522,7 @@ getModifiedTime:: {
522522
"/src/sub-project/sub-project.d.ts": 1,
523523
"/src/sub-project-2/index.js": 1,
524524
"/src/sub-project-2/sub-project-2.js": 1,
525-
"/src/sub-project-2/sub-project-2.d.ts": 3,
525+
"/src/sub-project-2/sub-project-2.d.ts": 2,
526526
"/src/sub-project-2/sub-project-2.tsbuildinfo": 1,
527527
"/src/sub-project-2/tsconfig.json": 1
528528
}

tests/baselines/reference/tsbuild/outFile/rebuilds-completely-when-command-line-incremental-flag-changes-between-non-dts-changes.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -636,14 +636,14 @@ getModifiedTime:: {
636636
"/src/second/second_part2.ts": 1,
637637
"/src/2/second-output.js": 1,
638638
"/src/2/second-output.js.map": 1,
639-
"/src/2/second-output.d.ts": 2,
639+
"/src/2/second-output.d.ts": 1,
640640
"/src/2/second-output.d.ts.map": 1,
641641
"/src/2/second-output.tsbuildinfo": 1,
642642
"/src/second/tsconfig.json": 1,
643643
"/src/third/third_part1.ts": 1,
644644
"/src/third/thirdjs/output/third-output.js": 1,
645645
"/src/third/thirdjs/output/third-output.js.map": 1,
646-
"/src/third/thirdjs/output/third-output.d.ts": 3,
646+
"/src/third/thirdjs/output/third-output.d.ts": 2,
647647
"/src/third/thirdjs/output/third-output.d.ts.map": 1,
648648
"/src/third/tsconfig.json": 1
649649
}
@@ -824,14 +824,14 @@ getModifiedTime:: {
824824
"/src/second/second_part2.ts": 1,
825825
"/src/2/second-output.js": 1,
826826
"/src/2/second-output.js.map": 1,
827-
"/src/2/second-output.d.ts": 2,
827+
"/src/2/second-output.d.ts": 1,
828828
"/src/2/second-output.d.ts.map": 1,
829829
"/src/2/second-output.tsbuildinfo": 1,
830830
"/src/second/tsconfig.json": 1,
831831
"/src/third/third_part1.ts": 1,
832832
"/src/third/thirdjs/output/third-output.js": 1,
833833
"/src/third/thirdjs/output/third-output.js.map": 1,
834-
"/src/third/thirdjs/output/third-output.d.ts": 3,
834+
"/src/third/thirdjs/output/third-output.d.ts": 2,
835835
"/src/third/thirdjs/output/third-output.d.ts.map": 1,
836836
"/src/third/thirdjs/output/third-output.tsbuildinfo": 1
837837
}

0 commit comments

Comments
 (0)