Revert "Convert TerminalLink.length to endIndex"

Tyriar · Tyriar · commit fe39b1c6ded2 · 2020-07-14T21:25:13.000-07:00
This reverts commit 48ce8ec. Fixes microsoft#101455
diff --git a/src/vs/vscode.proposed.d.ts b/src/vs/vscode.proposed.d.ts
@@ -1127,14 +1127,14 @@ declare module 'vscode' {
 
 	export interface TerminalLink {
 		/**
-		 * The 0-based start index of the link on [TerminalLinkContext.line](#TerminalLinkContext.line].
+		 * The start index of the link on [TerminalLinkContext.line](#TerminalLinkContext.line].
 		 */
 		startIndex: number;
 
 		/**
-		 * The 0-based end index of the link on [TerminalLinkContext.line](#TerminalLinkContext.line].
+		 * The length of the link on [TerminalLinkContext.line](#TerminalLinkContext.line]
 		 */
-		endIndex: number;
+		length: number;
 
 		/**
 		 * The tooltip text when you hover over this link.
diff --git a/src/vs/workbench/api/common/extHostTerminalService.ts b/src/vs/workbench/api/common/extHostTerminalService.ts
@@ -626,11 +626,10 @@ export abstract class BaseExtHostTerminalService implements IExtHostTerminalServ
 		for (const provideResult of provideResults) {
 			if (provideResult && provideResult.links.length > 0) {
 				result.push(...provideResult.links.map(providerLink => {
-					const endIndex = Math.max(providerLink.endIndex, providerLink.startIndex + 1);
 					const link = {
 						id: nextLinkId++,
 						startIndex: providerLink.startIndex,
-						length: endIndex - providerLink.startIndex,
+						length: providerLink.length,
 						label: providerLink.tooltip
 					};
 					cacheLinkMap.set(link.id, {
