Skip to content

Commit 5b3fe6e

Browse files
ajafffajafff
committed
update other interfaces and implementations
and accept baseline
1 parent 3b058a4 commit 5b3fe6e

File tree

6 files changed

+21
-21
lines changed

6 files changed

+21
-21
lines changed

‎src/compiler/resolutionCache.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ namespace ts {
55
startRecordingFilesWithChangedResolutions(): void;
66
finishRecordingFilesWithChangedResolutions(): Path[] | undefined;
77

8-
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames: string[] | undefined, redirectedReference?: ResolvedProjectReference): ResolvedModuleFull[];
8+
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames: string[] | undefined, redirectedReference?: ResolvedProjectReference): (ResolvedModuleFull | undefined)[];
99
getResolvedModuleWithFailedLookupLocationsFromCache(moduleName: string, containingFile: string): CachedResolvedModuleWithFailedLookupLocations | undefined;
10-
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[];
10+
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[];
1111

1212
invalidateResolutionOfFile(filePath: Path): void;
1313
removeResolutionsOfFile(filePath: Path): void;
@@ -250,7 +250,7 @@ namespace ts {
250250
loader: (name: string, containingFile: string, options: CompilerOptions, host: ModuleResolutionHost, redirectedReference?: ResolvedProjectReference) => T,
251251
getResolutionWithResolvedFileName: GetResolutionWithResolvedFileName<T, R>,
252252
reusedNames: string[] | undefined,
253-
logChanges: boolean): R[] {
253+
logChanges: boolean): (R | undefined)[] {
254254

255255
const path = resolutionHost.toPath(containingFile);
256256
const resolutionsInFile = cache.get(path) || cache.set(path, createMap()).get(path)!;
@@ -335,7 +335,7 @@ namespace ts {
335335
}
336336
}
337337

338-
function resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[] {
338+
function resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[] {
339339
return resolveNamesWithLocalCache<CachedResolvedTypeReferenceDirectiveWithFailedLookupLocations, ResolvedTypeReferenceDirective>(
340340
typeDirectiveNames, containingFile, redirectedReference,
341341
resolvedTypeReferenceDirectives, perDirectoryResolvedTypeReferenceDirectives,
@@ -344,7 +344,7 @@ namespace ts {
344344
);
345345
}
346346

347-
function resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames: string[] | undefined, redirectedReference?: ResolvedProjectReference): ResolvedModuleFull[] {
347+
function resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames: string[] | undefined, redirectedReference?: ResolvedProjectReference): (ResolvedModuleFull | undefined)[] {
348348
return resolveNamesWithLocalCache<CachedResolvedModuleWithFailedLookupLocations, ResolvedModuleFull>(
349349
moduleNames, containingFile, redirectedReference,
350350
resolvedModuleNames, perDirectoryResolvedModuleNames,

‎src/compiler/watch.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,9 +338,9 @@ namespace ts {
338338
getEnvironmentVariable?(name: string): string | undefined;
339339

340340
/** If provided, used to resolve the module names, otherwise typescript's default module resolution */
341-
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): ResolvedModule[];
341+
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): (ResolvedModule | undefined)[];
342342
/** If provided, used to resolve type reference directives, otherwise typescript's default resolution */
343-
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[];
343+
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[];
344344
}
345345

346346
/** Internal interface used to wire emit through same host */

‎src/server/project.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,15 +368,15 @@ namespace ts.server {
368368
return !this.isWatchedMissingFile(path) && this.directoryStructureHost.fileExists(file);
369369
}
370370

371-
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): ResolvedModuleFull[] {
371+
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): (ResolvedModuleFull | undefined)[] {
372372
return this.resolutionCache.resolveModuleNames(moduleNames, containingFile, reusedNames, redirectedReference);
373373
}
374374

375375
getResolvedModuleWithFailedLookupLocationsFromCache(moduleName: string, containingFile: string): ResolvedModuleWithFailedLookupLocations | undefined {
376376
return this.resolutionCache.getResolvedModuleWithFailedLookupLocationsFromCache(moduleName, containingFile);
377377
}
378378

379-
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[] {
379+
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[] {
380380
return this.resolutionCache.resolveTypeReferenceDirectives(typeDirectiveNames, containingFile, redirectedReference);
381381
}
382382

‎src/services/shims.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -331,29 +331,29 @@ namespace ts {
331331
private loggingEnabled = false;
332332
private tracingEnabled = false;
333333

334-
public resolveModuleNames: (moduleName: string[], containingFile: string) => ResolvedModuleFull[];
335-
public resolveTypeReferenceDirectives: (typeDirectiveNames: string[], containingFile: string) => ResolvedTypeReferenceDirective[];
334+
public resolveModuleNames: (moduleName: string[], containingFile: string) => (ResolvedModuleFull | undefined)[];
335+
public resolveTypeReferenceDirectives: (typeDirectiveNames: string[], containingFile: string) => (ResolvedTypeReferenceDirective | undefined)[];
336336
public directoryExists: (directoryName: string) => boolean;
337337

338338
constructor(private shimHost: LanguageServiceShimHost) {
339339
// if shimHost is a COM object then property check will become method call with no arguments.
340340
// 'in' does not have this effect.
341341
if ("getModuleResolutionsForFile" in this.shimHost) {
342-
this.resolveModuleNames = (moduleNames: string[], containingFile: string): ResolvedModuleFull[] => {
342+
this.resolveModuleNames = (moduleNames, containingFile) => {
343343
const resolutionsInFile = <MapLike<string>>JSON.parse(this.shimHost.getModuleResolutionsForFile!(containingFile)); // TODO: GH#18217
344344
return map(moduleNames, name => {
345345
const result = getProperty(resolutionsInFile, name);
346-
return result ? { resolvedFileName: result, extension: extensionFromPath(result), isExternalLibraryImport: false } : undefined!; // TODO: GH#18217
346+
return result ? { resolvedFileName: result, extension: extensionFromPath(result), isExternalLibraryImport: false } : undefined;
347347
});
348348
};
349349
}
350350
if ("directoryExists" in this.shimHost) {
351351
this.directoryExists = directoryName => this.shimHost.directoryExists(directoryName);
352352
}
353353
if ("getTypeReferenceDirectiveResolutionsForFile" in this.shimHost) {
354-
this.resolveTypeReferenceDirectives = (typeDirectiveNames: string[], containingFile: string) => {
354+
this.resolveTypeReferenceDirectives = (typeDirectiveNames, containingFile) => {
355355
const typeDirectivesForFile = <MapLike<ResolvedTypeReferenceDirective>>JSON.parse(this.shimHost.getTypeReferenceDirectiveResolutionsForFile!(containingFile)); // TODO: GH#18217
356-
return map(typeDirectiveNames, name => getProperty(typeDirectivesForFile, name)!); // TODO: GH#18217
356+
return map(typeDirectiveNames, name => getProperty(typeDirectivesForFile, name));
357357
};
358358
}
359359
}

‎tests/baselines/reference/api/tsserverlibrary.d.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4386,9 +4386,9 @@ declare namespace ts {
43864386
/** If provided is used to get the environment variable */
43874387
getEnvironmentVariable?(name: string): string | undefined;
43884388
/** If provided, used to resolve the module names, otherwise typescript's default module resolution */
4389-
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): ResolvedModule[];
4389+
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): (ResolvedModule | undefined)[];
43904390
/** If provided, used to resolve type reference directives, otherwise typescript's default resolution */
4391-
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[];
4391+
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[];
43924392
}
43934393
/**
43944394
* Host to create watch with root files and options
@@ -8151,9 +8151,9 @@ declare namespace ts.server {
81518151
readFile(fileName: string): string | undefined;
81528152
writeFile(fileName: string, content: string): void;
81538153
fileExists(file: string): boolean;
8154-
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): ResolvedModuleFull[];
8154+
resolveModuleNames(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): (ResolvedModuleFull | undefined)[];
81558155
getResolvedModuleWithFailedLookupLocationsFromCache(moduleName: string, containingFile: string): ResolvedModuleWithFailedLookupLocations | undefined;
8156-
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[];
8156+
resolveTypeReferenceDirectives(typeDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[];
81578157
directoryExists(path: string): boolean;
81588158
getDirectories(path: string): string[];
81598159
log(s: string): void;

‎tests/baselines/reference/api/typescript.d.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4386,9 +4386,9 @@ declare namespace ts {
43864386
/** If provided is used to get the environment variable */
43874387
getEnvironmentVariable?(name: string): string | undefined;
43884388
/** If provided, used to resolve the module names, otherwise typescript's default module resolution */
4389-
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): ResolvedModule[];
4389+
resolveModuleNames?(moduleNames: string[], containingFile: string, reusedNames?: string[], redirectedReference?: ResolvedProjectReference): (ResolvedModule | undefined)[];
43904390
/** If provided, used to resolve type reference directives, otherwise typescript's default resolution */
4391-
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): ResolvedTypeReferenceDirective[];
4391+
resolveTypeReferenceDirectives?(typeReferenceDirectiveNames: string[], containingFile: string, redirectedReference?: ResolvedProjectReference): (ResolvedTypeReferenceDirective | undefined)[];
43924392
}
43934393
/**
43944394
* Host to create watch with root files and options

0 commit comments

Comments
 (0)