Expose sort and deduplicate diagnostics in Public API

sheetalkamat · sheetalkamat · commit a05f669f43d4 · 2018-01-22T16:41:21.000-08:00
Fixes microsoft#20876
diff --git a/src/compiler/core.ts b/src/compiler/core.ts
@@ -16,6 +16,10 @@ namespace ts {
         // Update: We also consider a path like `C:\foo.ts` "relative" because we do not search for it in `node_modules` or treat it as an ambient module.
         return pathIsRelative(moduleName) || isRootedDiskPath(moduleName);
     }
+
+    export function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[] {
+        return sortAndDeduplicate(diagnostics, compareDiagnostics);
+    }
 }
 
 /* @internal */
@@ -1897,10 +1901,6 @@ namespace ts {
         return text1 ? Comparison.GreaterThan : Comparison.LessThan;
     }
 
-    export function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[] {
-        return sortAndDeduplicate(diagnostics, compareDiagnostics);
-    }
-
     export function normalizeSlashes(path: string): string {
         return path.replace(/\\/g, "/");
     }
diff --git a/tests/baselines/reference/api/tsserverlibrary.d.ts b/tests/baselines/reference/api/tsserverlibrary.d.ts
@@ -2814,6 +2814,7 @@ declare namespace ts {
 }
 declare namespace ts {
     function isExternalModuleNameRelative(moduleName: string): boolean;
+    function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[];
 }
 declare function setTimeout(handler: (...args: any[]) => void, timeout: number): any;
 declare function clearTimeout(handle: any): void;
diff --git a/tests/baselines/reference/api/typescript.d.ts b/tests/baselines/reference/api/typescript.d.ts
@@ -2814,6 +2814,7 @@ declare namespace ts {
 }
 declare namespace ts {
     function isExternalModuleNameRelative(moduleName: string): boolean;
+    function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[];
 }
 declare function setTimeout(handler: (...args: any[]) => void, timeout: number): any;
 declare function clearTimeout(handle: any): void;

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,10 @@ namespace ts {`
`16`	`16`	// Update: We also consider a path like `C:\foo.ts` "relative" because we do not search for it in `node_modules` or treat it as an ambient module.
`17`	`17`	`return pathIsRelative(moduleName) \|\| isRootedDiskPath(moduleName);`
`18`	`18`	`}`
	`19`	`+`
	`20`	`+ export function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[] {`
	`21`	`+ return sortAndDeduplicate(diagnostics, compareDiagnostics);`
	`22`	`+ }`
`19`	`23`	`}`
`20`	`24`
`21`	`25`	`/* @internal */`
`@@ -1897,10 +1901,6 @@ namespace ts {`
`1897`	`1901`	`return text1 ? Comparison.GreaterThan : Comparison.LessThan;`
`1898`	`1902`	`}`
`1899`	`1903`
`1900`		`- export function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[] {`
`1901`		`- return sortAndDeduplicate(diagnostics, compareDiagnostics);`
`1902`		`- }`
`1903`		`-`
`1904`	`1904`	`export function normalizeSlashes(path: string): string {`
`1905`	`1905`	`return path.replace(/\\/g, "/");`
`1906`	`1906`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2814,6 +2814,7 @@ declare namespace ts {`
`2814`	`2814`	`}`
`2815`	`2815`	`declare namespace ts {`
`2816`	`2816`	`function isExternalModuleNameRelative(moduleName: string): boolean;`
	`2817`	`+ function sortAndDeduplicateDiagnostics(diagnostics: ReadonlyArray<Diagnostic>): Diagnostic[];`
`2817`	`2818`	`}`
`2818`	`2819`	`declare function setTimeout(handler: (...args: any[]) => void, timeout: number): any;`
`2819`	`2820`	`declare function clearTimeout(handle: any): void;`