replace void 0 with undefined

src/vs/workbench/api/node/extHost.api.impl.ts

@@ -601,7 +601,7 @@ export function createApiFactory(
 				return extHostConfiguration.onDidChangeConfiguration(listener, thisArgs, disposables);
 			},
 			getConfiguration(section?: string, resource?: vscode.Uri): vscode.WorkspaceConfiguration {
-				resource = arguments.length === 1 ? void 0 : resource;
+				resource = arguments.length === 1 ? undefined : resource;
 				return extHostConfiguration.getConfiguration(section, resource, extension.identifier);
 			},
 			registerTextDocumentContentProvider(scheme: string, provider: vscode.TextDocumentContentProvider) {

src/vs/workbench/api/node/extHostApiCommands.ts

@@ -109,7 +109,7 @@ export class ExtHostApiCommands {
 			args: [
 				{ name: 'uri', description: 'Uri of a text document', constraint: URI },
 				{ name: 'position', description: 'Position in a text document', constraint: types.Position },
-				{ name: 'triggerCharacter', description: '(optional) Trigger signature help when the user types the character, like `,` or `(`', constraint: (value: any) => value === void 0 || typeof value === 'string' }
+				{ name: 'triggerCharacter', description: '(optional) Trigger signature help when the user types the character, like `,` or `(`', constraint: (value: any) => value === undefined || typeof value === 'string' }
 			],
 			returns: 'A promise that resolves to SignatureHelp.'
 		});
@@ -125,8 +125,8 @@ export class ExtHostApiCommands {
 			args: [
 				{ name: 'uri', description: 'Uri of a text document', constraint: URI },
 				{ name: 'position', description: 'Position in a text document', constraint: types.Position },
-				{ name: 'triggerCharacter', description: '(optional) Trigger completion when the user types the character, like `,` or `(`', constraint: (value: any) => value === void 0 || typeof value === 'string' },
-				{ name: 'itemResolveCount', description: '(optional) Number of completions to resolve (too large numbers slow down completions)', constraint: (value: any) => value === void 0 || typeof value === 'number' }
+				{ name: 'triggerCharacter', description: '(optional) Trigger completion when the user types the character, like `,` or `(`', constraint: (value: any) => value === undefined || typeof value === 'string' },
+				{ name: 'itemResolveCount', description: '(optional) Number of completions to resolve (too large numbers slow down completions)', constraint: (value: any) => value === undefined || typeof value === 'number' }
 			],
 			returns: 'A promise that resolves to a CompletionList-instance.'
 		});
@@ -142,7 +142,7 @@ export class ExtHostApiCommands {
 			description: 'Execute CodeLens provider.',
 			args: [
 				{ name: 'uri', description: 'Uri of a text document', constraint: URI },
-				{ name: 'itemResolveCount', description: '(optional) Number of lenses that should be resolved and returned. Will only retrun resolved lenses, will impact performance)', constraint: (value: any) => value === void 0 || typeof value === 'number' }
+				{ name: 'itemResolveCount', description: '(optional) Number of lenses that should be resolved and returned. Will only retrun resolved lenses, will impact performance)', constraint: (value: any) => value === undefined || typeof value === 'number' }
 			],
 			returns: 'A promise that resolves to an array of CodeLens-instances.'
 		});
@@ -235,8 +235,8 @@ export class ExtHostApiCommands {
 		this._register(OpenFolderAPICommand.ID, adjustHandler(OpenFolderAPICommand.execute), {
 			description: 'Open a folder or workspace in the current window or new window depending on the newWindow argument. Note that opening in the same window will shutdown the current extension host process and start a new one on the given folder/workspace unless the newWindow parameter is set to true.',
 			args: [
-				{ name: 'uri', description: '(optional) Uri of the folder or workspace file to open. If not provided, a native dialog will ask the user for the folder', constraint: (value: any) => value === void 0 || value instanceof URI },
-				{ name: 'newWindow', description: '(optional) Whether to open the folder/workspace in a new window or the same. Defaults to opening in the same window.', constraint: (value: any) => value === void 0 || typeof value === 'boolean' }
+				{ name: 'uri', description: '(optional) Uri of the folder or workspace file to open. If not provided, a native dialog will ask the user for the folder', constraint: (value: any) => value === undefined || value instanceof URI },
+				{ name: 'newWindow', description: '(optional) Whether to open the folder/workspace in a new window or the same. Defaults to opening in the same window.', constraint: (value: any) => value === undefined || typeof value === 'boolean' }
 			]
 		});
 
@@ -245,7 +245,7 @@ export class ExtHostApiCommands {
 			args: [
 				{ name: 'left', description: 'Left-hand side resource of the diff editor', constraint: URI },
 				{ name: 'right', description: 'Right-hand side resource of the diff editor', constraint: URI },
-				{ name: 'title', description: '(optional) Human readable title for the diff editor', constraint: (v: any) => v === void 0 || typeof v === 'string' },
+				{ name: 'title', description: '(optional) Human readable title for the diff editor', constraint: (v: any) => v === undefined || typeof v === 'string' },
 				{ name: 'options', description: '(optional) Editor options, see vscode.TextDocumentShowOptions' }
 			]
 		});
@@ -254,7 +254,7 @@ export class ExtHostApiCommands {
 			description: 'Opens the provided resource in the editor. Can be a text or binary file, or a http(s) url. If you need more control over the options for opening a text file, use vscode.window.showTextDocument instead.',
 			args: [
 				{ name: 'resource', description: 'Resource to open', constraint: URI },
-				{ name: 'columnOrOptions', description: '(optional) Either the column in which to open or editor options, see vscode.TextDocumentShowOptions', constraint: (v: any) => v === void 0 || typeof v === 'number' || typeof v === 'object' }
+				{ name: 'columnOrOptions', description: '(optional) Either the column in which to open or editor options, see vscode.TextDocumentShowOptions', constraint: (v: any) => v === undefined || typeof v === 'number' || typeof v === 'object' }
 			]
 		});
 

src/vs/workbench/api/node/extHostConfiguration.ts

@@ -71,7 +71,7 @@ export class ExtHostConfiguration implements ExtHostConfigurationShape {
 		}
 
 		function parseConfigurationTarget(arg: boolean | ExtHostConfigurationTarget): ConfigurationTarget {
-			if (arg === void 0 || arg === null) {
+			if (arg === undefined || arg === null) {
 				return null;
 			}
 			if (typeof arg === 'boolean') {
@@ -95,9 +95,9 @@ export class ExtHostConfiguration implements ExtHostConfigurationShape {
 				if (typeof result === 'undefined') {
 					result = defaultValue;
 				} else {
-					let clonedConfig = void 0;
+					let clonedConfig = undefined;
 					const cloneOnWriteProxy = (target: any, accessor: string): any => {
-						let clonedTarget = void 0;
+						let clonedTarget = undefined;
 						const cloneTarget = () => {
 							clonedConfig = clonedConfig ? clonedConfig : deepClone(config);
 							clonedTarget = clonedTarget ? clonedTarget : lookUp(clonedConfig, accessor);
@@ -143,7 +143,7 @@ export class ExtHostConfiguration implements ExtHostConfigurationShape {
 			update: (key: string, value: any, arg: ExtHostConfigurationTarget | boolean) => {
 				key = section ? `${section}.${key}` : key;
 				const target = parseConfigurationTarget(arg);
-				if (value !== void 0) {
+				if (value !== undefined) {
 					return this._proxy.$updateConfigurationOption(target, key, value, resource);
 				} else {
 					return this._proxy.$removeConfigurationOption(target, key, resource);
@@ -192,7 +192,7 @@ export class ExtHostConfiguration implements ExtHostConfigurationShape {
 		const scope = OVERRIDE_PROPERTY_PATTERN.test(key) ? ConfigurationScope.RESOURCE : this._configurationScopes[key];
 		const extensionIdText = extensionId ? `[${extensionId.value}] ` : '';
 		if (ConfigurationScope.RESOURCE === scope) {
-			if (resource === void 0) {
+			if (resource === undefined) {
 				console.warn(`${extensionIdText}Accessing a resource scoped configuration without providing a resource is not expected. To get the effective value for '${key}', provide the URI of a resource or 'null' for any resource.`);
 			}
 			return;

src/vs/workbench/api/node/extHostDebugService.ts

@@ -361,7 +361,7 @@ export class ExtHostDebugService implements ExtHostDebugServiceShape {
 				return terminalLauncher.runInTerminal(args, config);
 			}
 		}
-		return void 0;
+		return undefined;
 	}
 
 	public $substituteVariables(folderUri: UriComponents | undefined, config: IConfig): Promise<IConfig> {
@@ -469,7 +469,7 @@ export class ExtHostDebugService implements ExtHostDebugServiceShape {
 		if (da) {
 			da.sendMessage(message);
 		}
-		return void 0;
+		return undefined;
 	}
 
 	public $stopDASession(debugAdapterHandle: number): Promise<void> {
@@ -485,7 +485,7 @@ export class ExtHostDebugService implements ExtHostDebugServiceShape {
 		if (da) {
 			return da.stopSession();
 		} else {
-			return void 0;
+			return undefined;
 		}
 	}
 
@@ -934,27 +934,27 @@ class MultiTracker implements vscode.DebugAdapterTracker {
 	}
 
 	onWillStartSession(): void {
-		this.trackers.forEach(t => t.onWillStartSession ? t.onWillStartSession() : (t.startDebugAdapter ? t.startDebugAdapter() : void 0));
+		this.trackers.forEach(t => t.onWillStartSession ? t.onWillStartSession() : (t.startDebugAdapter ? t.startDebugAdapter() : undefined));
 	}
 
 	onWillReceiveMessage(message: any): void {
-		this.trackers.forEach(t => t.onWillReceiveMessage ? t.onWillReceiveMessage(message) : (t.toDebugAdapter ? t.toDebugAdapter(message) : void 0));
+		this.trackers.forEach(t => t.onWillReceiveMessage ? t.onWillReceiveMessage(message) : (t.toDebugAdapter ? t.toDebugAdapter(message) : undefined));
 	}
 
 	onDidSendMessage(message: any): void {
-		this.trackers.forEach(t => t.onDidSendMessage ? t.onDidSendMessage(message) : (t.fromDebugAdapter ? t.fromDebugAdapter(message) : void 0));
+		this.trackers.forEach(t => t.onDidSendMessage ? t.onDidSendMessage(message) : (t.fromDebugAdapter ? t.fromDebugAdapter(message) : undefined));
 	}
 
 	onWillStopSession(): void {
-		this.trackers.forEach(t => t.onWillStopSession ? t.onWillStopSession() : (t.stopDebugAdapter ? t.stopDebugAdapter() : void 0));
+		this.trackers.forEach(t => t.onWillStopSession ? t.onWillStopSession() : (t.stopDebugAdapter ? t.stopDebugAdapter() : undefined));
 	}
 
 	onError(error: Error): void {
-		this.trackers.forEach(t => t.onError ? t.onError(error) : (t.debugAdapterError ? t.debugAdapterError(error) : void 0));
+		this.trackers.forEach(t => t.onError ? t.onError(error) : (t.debugAdapterError ? t.debugAdapterError(error) : undefined));
 	}
 
 	onExit(code: number, signal: string): void {
-		this.trackers.forEach(t => t.onExit ? t.onExit(code, signal) : (t.debugAdapterExit ? t.debugAdapterExit(code, signal) : void 0));
+		this.trackers.forEach(t => t.onExit ? t.onExit(code, signal) : (t.debugAdapterExit ? t.debugAdapterExit(code, signal) : undefined));
 	}
 }
 
@@ -985,7 +985,7 @@ class DirectDebugAdapter extends AbstractDebugAdapter implements IDapTransport {
 
 	// AbstractDebugAdapter
 	startSession(): Promise<void> {
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	// AbstractDebugAdapter
@@ -997,7 +997,7 @@ class DirectDebugAdapter extends AbstractDebugAdapter implements IDapTransport {
 	// AbstractDebugAdapter
 	stopSession(): Promise<void> {
 		this.stop();
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	// IDapTransport

src/vs/workbench/api/node/extHostDecorations.ts

@@ -48,7 +48,7 @@ export class ExtHostDecorations implements ExtHostDecorationsShape {
 			const { handle, uri, id } = request;
 			if (!this._provider.has(handle)) {
 				// might have been unregistered in the meantime
-				return void 0;
+				return undefined;
 			}
 			const { provider, extensionId } = this._provider.get(handle);
 			return Promise.resolve(provider.provideDecoration(URI.revive(uri), token)).then(data => {

src/vs/workbench/api/node/extHostExtensionActivator.ts

@@ -10,7 +10,7 @@ import { IExtensionDescription } from 'vs/workbench/services/extensions/common/e
 import { ExtensionDescriptionRegistry } from 'vs/workbench/services/extensions/node/extensionDescriptionRegistry';
 import { CanonicalExtensionIdentifier } from 'vs/platform/extensions/common/extensions';
 
-const NO_OP_VOID_PROMISE = Promise.resolve<void>(void 0);
+const NO_OP_VOID_PROMISE = Promise.resolve<void>(undefined);
 
 export interface IExtensionMemento {
 	get<T>(key: string, defaultValue: T): T;
@@ -288,12 +288,12 @@ export class ExtensionsActivator {
 	private _activateExtensions(extensionDescriptions: IExtensionDescription[], reason: ExtensionActivationReason, recursionLevel: number): Promise<void> {
 		// console.log(recursionLevel, '_activateExtensions: ', extensionDescriptions.map(p => p.id));
 		if (extensionDescriptions.length === 0) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		extensionDescriptions = extensionDescriptions.filter((p) => !this._activatedExtensions.has(CanonicalExtensionIdentifier.toKey(p.identifier)));
 		if (extensionDescriptions.length === 0) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		if (recursionLevel > 10) {
@@ -304,7 +304,7 @@ export class ExtensionsActivator {
 				const error = new Error('More than 10 levels of dependencies (most likely a dependency loop)');
 				this._activatedExtensions.set(CanonicalExtensionIdentifier.toKey(extensionDescriptions[i].identifier), new FailedExtension(error));
 			}
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		let greenMap: { [id: string]: IExtensionDescription; } = Object.create(null),
@@ -329,7 +329,7 @@ export class ExtensionsActivator {
 
 		if (red.length === 0) {
 			// Finally reached only leafs!
-			return Promise.all(green.map((p) => this._activateExtension(p, reason))).then(_ => void 0);
+			return Promise.all(green.map((p) => this._activateExtension(p, reason))).then(_ => undefined);
 		}
 
 		return this._activateExtensions(green, reason, recursionLevel + 1).then(_ => {
@@ -341,14 +341,14 @@ export class ExtensionsActivator {
 		const extensionKey = CanonicalExtensionIdentifier.toKey(extensionDescription.identifier);
 
 		if (this._activatedExtensions.has(extensionKey)) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		if (this._activatingExtensions.has(extensionKey)) {
 			return this._activatingExtensions.get(extensionKey);
 		}
 
-		this._activatingExtensions.set(extensionKey, this._host.actualActivateExtension(extensionDescription, reason).then(void 0, (err) => {
+		this._activatingExtensions.set(extensionKey, this._host.actualActivateExtension(extensionDescription, reason).then(undefined, (err) => {
 			this._host.showMessage(Severity.Error, nls.localize('activationError', "Activating extension '{0}' failed: {1}.", extensionDescription.identifier.value, err.message));
 			console.error('Activating extension `' + extensionDescription.identifier.value + '` failed: ', err.message);
 			console.log('Here is the error stack: ', err.stack);

src/vs/workbench/api/node/extHostExtensionService.ts

@@ -270,7 +270,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 				// activation failed => bubble up the error as the promise result
 				return Promise.reject(extension.activationFailedError);
 			}
-			return void 0;
+			return undefined;
 		});
 	}
 
@@ -302,7 +302,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 	}
 
 	private _deactivate(extensionId: CanonicalExtensionIdentifier): Promise<void> {
-		let result = Promise.resolve(void 0);
+		let result = Promise.resolve(undefined);
 
 		if (!this._barrier.isOpen()) {
 			return result;
@@ -320,9 +320,9 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 		// call deactivate if available
 		try {
 			if (typeof extension.module.deactivate === 'function') {
-				result = Promise.resolve(extension.module.deactivate()).then(void 0, (err) => {
+				result = Promise.resolve(extension.module.deactivate()).then(undefined, (err) => {
 					// TODO: Do something with err if this is not the shutdown case
-					return Promise.resolve(void 0);
+					return Promise.resolve(undefined);
 				});
 			}
 		} catch (err) {
@@ -467,7 +467,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 
 	// Handle "eager" activation extensions
 	private _handleEagerExtensions(): Promise<void> {
-		this._activateByEvent('*', true).then(void 0, (err) => {
+		this._activateByEvent('*', true).then(undefined, (err) => {
 			console.error(err);
 		});
 
@@ -476,7 +476,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 
 	private _handleWorkspaceContainsEagerExtensions(workspace: IWorkspaceData): Promise<void> {
 		if (!workspace || workspace.folders.length === 0) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		return Promise.all(
@@ -489,7 +489,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 	private _handleWorkspaceContainsEagerExtension(workspace: IWorkspaceData, desc: IExtensionDescription): Promise<void> {
 		const activationEvents = desc.activationEvents;
 		if (!activationEvents) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		const fileNames: string[] = [];
@@ -507,7 +507,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 		}
 
 		if (fileNames.length === 0 && globPatterns.length === 0) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		const fileNamePromise = Promise.all(fileNames.map((fileName) => this._activateIfFileName(workspace, desc.identifier, fileName))).then(() => { });
@@ -524,7 +524,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 				// the file was found
 				return (
 					this._activateById(extensionId, new ExtensionActivatedByEvent(true, `workspaceContains:${fileName}`))
-						.then(void 0, err => console.error(err))
+						.then(undefined, err => console.error(err))
 				);
 			}
 		}
@@ -536,7 +536,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 		this._extHostLogService.trace(`extensionHostMain#activateIfGlobPatterns: fileSearch, extension: ${extensionId.value}, entryPoint: workspaceContains`);
 
 		if (globPatterns.length === 0) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		const tokenSource = new CancellationTokenSource();
@@ -545,7 +545,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 		const timer = setTimeout(async () => {
 			tokenSource.cancel();
 			this._activateById(extensionId, new ExtensionActivatedByEvent(true, `workspaceContainsTimeout:${globPatterns.join(',')}`))
-				.then(void 0, err => console.error(err));
+				.then(undefined, err => console.error(err));
 		}, ExtHostExtensionService.WORKSPACE_CONTAINS_TIMEOUT);
 
 		let exists: boolean;
@@ -564,16 +564,16 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 			// a file was found matching one of the glob patterns
 			return (
 				this._activateById(extensionId, new ExtensionActivatedByEvent(true, `workspaceContains:${globPatterns.join(',')}`))
-					.then(void 0, err => console.error(err))
+					.then(undefined, err => console.error(err))
 			);
 		}
 
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	private _handleExtensionTests(): Promise<void> {
 		if (!this._initData.environment.extensionTestsPath || !this._initData.environment.extensionDevelopmentLocationURI) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		// Require the test runner via node require from the provided path
@@ -592,7 +592,7 @@ export class ExtHostExtensionService implements ExtHostExtensionServiceShape {
 					if (error) {
 						e(error.toString());
 					} else {
-						c(void 0);
+						c(undefined);
 					}
 
 					// after tests have run, we shutdown the host

src/vs/workbench/api/node/extHostLanguageFeatures.ts

@@ -840,7 +840,7 @@ class FoldingProviderAdapter {
 		const doc = this._documents.getDocumentData(resource).document;
 		return asPromise(() => this._provider.provideFoldingRanges(doc, context, token)).then(ranges => {
 			if (!Array.isArray(ranges)) {
-				return void 0;
+				return undefined;
 			}
 			return ranges.map(typeConvert.FoldingRange.from);
 		});

src/vs/workbench/api/node/extHostQuickOpen.ts

@@ -115,7 +115,7 @@ export class ExtHostQuickOpen implements ExtHostQuickOpenShape {
 					return undefined;
 				});
 			});
-		}).then(void 0, err => {
+		}).then(undefined, err => {
 			if (isPromiseCanceledError(err)) {
 				return undefined;
 			}
@@ -140,7 +140,7 @@ export class ExtHostQuickOpen implements ExtHostQuickOpenShape {
 		this._validateInput = options && options.validateInput;
 
 		return this._proxy.$input(options, typeof this._validateInput === 'function', token)
-			.then(void 0, err => {
+			.then(undefined, err => {
 				if (isPromiseCanceledError(err)) {
 					return undefined;
 				}

src/vs/workbench/api/node/extHostSCM.ts

@@ -284,7 +284,7 @@ class ExtHostSourceControlResourceGroup implements vscode.SourceControlResourceG
 		const command = this._resourceStatesCommandsMap.get(handle);
 
 		if (!command) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		return asPromise(() => this._commands.executeCommand(command.command, ...command.arguments));
@@ -628,11 +628,11 @@ export class ExtHostSCM implements ExtHostSCMShape {
 		const sourceControl = this._sourceControls.get(sourceControlHandle);
 
 		if (!sourceControl) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		sourceControl.inputBox.$onInputBoxValueChange(value);
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	$executeResourceCommand(sourceControlHandle: number, groupHandle: number, handle: number): Promise<void> {
@@ -641,13 +641,13 @@ export class ExtHostSCM implements ExtHostSCMShape {
 		const sourceControl = this._sourceControls.get(sourceControlHandle);
 
 		if (!sourceControl) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		const group = sourceControl.getResourceGroup(groupHandle);
 
 		if (!group) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 
 		return group.$executeResourceCommand(handle);
@@ -709,6 +709,6 @@ export class ExtHostSCM implements ExtHostSCMShape {
 		});
 
 		this._selectedSourceControlHandles = set;
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 }

src/vs/workbench/api/node/extHostStatusBar.ts

@@ -125,7 +125,7 @@ class StatusBarMessage {
 	private _messages: { message: string }[] = [];
 
 	constructor(statusBar: ExtHostStatusBar) {
-		this._item = statusBar.createStatusBarEntry(void 0, ExtHostStatusBarAlignment.Left, Number.MIN_VALUE);
+		this._item = statusBar.createStatusBarEntry(undefined, ExtHostStatusBarAlignment.Left, Number.MIN_VALUE);
 	}
 
 	dispose() {

src/vs/workbench/api/node/extHostTask.ts

@@ -30,13 +30,13 @@ import { CancellationToken } from 'vs/base/common/cancellation';
 
 namespace TaskDefinitionDTO {
 	export function from(value: vscode.TaskDefinition): TaskDefinitionDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
 	}
 	export function to(value: TaskDefinitionDTO): vscode.TaskDefinition {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
@@ -45,13 +45,13 @@ namespace TaskDefinitionDTO {
 
 namespace TaskPresentationOptionsDTO {
 	export function from(value: vscode.TaskPresentationOptions): TaskPresentationOptionsDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
 	}
 	export function to(value: TaskPresentationOptionsDTO): vscode.TaskPresentationOptions {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
@@ -60,13 +60,13 @@ namespace TaskPresentationOptionsDTO {
 
 namespace ProcessExecutionOptionsDTO {
 	export function from(value: vscode.ProcessExecutionOptions): ProcessExecutionOptionsDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
 	}
 	export function to(value: ProcessExecutionOptionsDTO): vscode.ProcessExecutionOptions {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
@@ -79,7 +79,7 @@ namespace ProcessExecutionDTO {
 		return candidate && !!candidate.process;
 	}
 	export function from(value: vscode.ProcessExecution): ProcessExecutionDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		let result: ProcessExecutionDTO = {
@@ -92,7 +92,7 @@ namespace ProcessExecutionDTO {
 		return result;
 	}
 	export function to(value: ProcessExecutionDTO): types.ProcessExecution {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return new types.ProcessExecution(value.process, value.args, value.options);
@@ -101,13 +101,13 @@ namespace ProcessExecutionDTO {
 
 namespace ShellExecutionOptionsDTO {
 	export function from(value: vscode.ShellExecutionOptions): ShellExecutionOptionsDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
 	}
 	export function to(value: ShellExecutionOptionsDTO): vscode.ShellExecutionOptions {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		return value;
@@ -120,12 +120,12 @@ namespace ShellExecutionDTO {
 		return candidate && (!!candidate.commandLine || !!candidate.command);
 	}
 	export function from(value: vscode.ShellExecution): ShellExecutionDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		let result: ShellExecutionDTO = {
 		};
-		if (value.commandLine !== void 0) {
+		if (value.commandLine !== undefined) {
 			result.commandLine = value.commandLine;
 		} else {
 			result.command = value.command;
@@ -137,7 +137,7 @@ namespace ShellExecutionDTO {
 		return result;
 	}
 	export function to(value: ShellExecutionDTO): types.ShellExecution {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		if (value.commandLine) {
@@ -151,7 +151,7 @@ namespace ShellExecutionDTO {
 namespace TaskHandleDTO {
 	export function from(value: types.Task): TaskHandleDTO {
 		let folder: UriComponents;
-		if (value.scope !== void 0 && typeof value.scope !== 'number') {
+		if (value.scope !== undefined && typeof value.scope !== 'number') {
 			folder = value.scope.uri;
 		}
 		return {
@@ -164,7 +164,7 @@ namespace TaskHandleDTO {
 namespace TaskDTO {
 
 	export function fromMany(tasks: vscode.Task[], extension: IExtensionDescription): TaskDTO[] {
-		if (tasks === void 0 || tasks === null) {
+		if (tasks === undefined || tasks === null) {
 			return [];
 		}
 		let result: TaskDTO[] = [];
@@ -178,7 +178,7 @@ namespace TaskDTO {
 	}
 
 	export function from(value: vscode.Task, extension: IExtensionDescription): TaskDTO {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		let execution: ShellExecutionDTO | ProcessExecutionDTO;
@@ -220,7 +220,7 @@ namespace TaskDTO {
 		return result;
 	}
 	export function to(value: TaskDTO, workspace: ExtHostWorkspace): types.Task {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			return undefined;
 		}
 		let execution: types.ShellExecution | types.ProcessExecution;
@@ -232,7 +232,7 @@ namespace TaskDTO {
 		let definition: vscode.TaskDefinition = TaskDefinitionDTO.to(value.definition);
 		let scope: vscode.TaskScope.Global | vscode.TaskScope.Workspace | vscode.WorkspaceFolder;
 		if (value.source) {
-			if (value.source.scope !== void 0) {
+			if (value.source.scope !== undefined) {
 				if (typeof value.source.scope === 'number') {
 					scope = value.source.scope;
 				} else {
@@ -246,10 +246,10 @@ namespace TaskDTO {
 			return undefined;
 		}
 		let result = new types.Task(definition, scope, value.name, value.source.label, execution, value.problemMatchers);
-		if (value.isBackground !== void 0) {
+		if (value.isBackground !== undefined) {
 			result.isBackground = value.isBackground;
 		}
-		if (value.group !== void 0) {
+		if (value.group !== undefined) {
 			result.group = types.TaskGroup.from(value.group);
 		}
 		if (value.presentationOptions) {
@@ -375,11 +375,11 @@ export class ExtHostTask implements ExtHostTaskShape {
 	public executeTask(extension: IExtensionDescription, task: vscode.Task): Promise<vscode.TaskExecution> {
 		let tTask = (task as types.Task);
 		// We have a preserved ID. So the task didn't change.
-		if (tTask._id !== void 0) {
+		if (tTask._id !== undefined) {
 			return this._proxy.$executeTask(TaskHandleDTO.from(tTask)).then(value => this.getTaskExecution(value, task));
 		} else {
 			let dto = TaskDTO.from(task, extension);
-			if (dto === void 0) {
+			if (dto === undefined) {
 				return Promise.reject(new Error('Task is not valid'));
 			}
 			return this._proxy.$executeTask(dto).then(value => this.getTaskExecution(value, task));
@@ -490,9 +490,9 @@ export class ExtHostTask implements ExtHostTaskShape {
 		for (let variable of toResolve.variables) {
 			result.variables[variable] = resolver.resolve(ws, variable);
 		}
-		if (toResolve.process !== void 0) {
+		if (toResolve.process !== undefined) {
 			let paths: string[] | undefined = undefined;
-			if (toResolve.process.path !== void 0) {
+			if (toResolve.process.path !== undefined) {
 				paths = toResolve.process.path.split(path.delimiter);
 				for (let i = 0; i < paths.length; i++) {
 					paths[i] = resolver.resolve(ws, paths[i]);
@@ -500,7 +500,7 @@ export class ExtHostTask implements ExtHostTaskShape {
 			}
 			result.process = win32.findExecutable(
 				resolver.resolve(ws, toResolve.process.name),
-				toResolve.process.cwd !== void 0 ? resolver.resolve(ws, toResolve.process.cwd) : undefined,
+				toResolve.process.cwd !== undefined ? resolver.resolve(ws, toResolve.process.cwd) : undefined,
 				paths
 			);
 		}

src/vs/workbench/api/node/extHostTextEditor.ts

@@ -604,7 +604,7 @@ export class ExtHostTextEditor implements vscode.TextEditor {
 }
 
 function warnOnError(promise: Promise<any>): void {
-	promise.then(void 0, (err) => {
+	promise.then(undefined, (err) => {
 		console.warn(err);
 	});
 }

src/vs/workbench/api/node/extHostTreeViews.ts

@@ -31,15 +31,15 @@ function toTreeItemLabel(label: any, extension: IExtensionDescription): ITreeIte
 		&& typeof label === 'object'
 		&& typeof label.label === 'string') {
 		checkProposedApiEnabled(extension);
-		let highlights: [number, number][] = void 0;
+		let highlights: [number, number][] = undefined;
 		if (Array.isArray(label.highlights)) {
 			highlights = (<[number, number][]>label.highlights).filter((highlight => highlight.length === 2 && typeof highlight[0] === 'number' && typeof highlight[1] === 'number'));
-			highlights = highlights.length ? highlights : void 0;
+			highlights = highlights.length ? highlights : undefined;
 		}
 		return { label: label.label, highlights };
 	}
 
-	return void 0;
+	return undefined;
 }
 
 
@@ -195,7 +195,7 @@ class ExtHostTreeView<T> extends Disposable {
 	}
 
 	getChildren(parentHandle?: TreeItemHandle): Promise<ITreeItem[]> {
-		const parentElement = parentHandle ? this.getExtensionElement(parentHandle) : void 0;
+		const parentElement = parentHandle ? this.getExtensionElement(parentHandle) : undefined;
 		if (parentHandle && !parentElement) {
 			console.error(`No tree item with id \'${parentHandle}\' found.`);
 			return Promise.resolve([]);
@@ -321,7 +321,7 @@ class ExtHostTreeView<T> extends Disposable {
 		// clear children cache
 		this.clearChildren(parentElement);
 
-		const parentNode = parentElement ? this.nodes.get(parentElement) : void 0;
+		const parentNode = parentElement ? this.nodes.get(parentElement) : undefined;
 		return asPromise(() => this.dataProvider.getChildren(parentElement))
 			.then(elements => Promise.all(
 				coalesce(elements || [])
@@ -341,7 +341,7 @@ class ExtHostTreeView<T> extends Disposable {
 				return this.refreshHandles(handlesToRefresh);
 			}
 		}
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	private getHandlesToRefresh(elements: T[]): TreeItemHandle[] {
@@ -415,7 +415,7 @@ class ExtHostTreeView<T> extends Disposable {
 		return {
 			item: this.createTreeItem(element, extensionTreeItem, parent),
 			parent,
-			children: void 0
+			children: undefined
 		};
 	}
 
@@ -425,16 +425,16 @@ class ExtHostTreeView<T> extends Disposable {
 		const icon = this.getLightIconPath(extensionTreeItem);
 		const item = {
 			handle,
-			parentHandle: parent ? parent.item.handle : void 0,
+			parentHandle: parent ? parent.item.handle : undefined,
 			label: toTreeItemLabel(extensionTreeItem.label, this.extension),
 			description: extensionTreeItem.description,
 			resourceUri: extensionTreeItem.resourceUri,
-			tooltip: typeof extensionTreeItem.tooltip === 'string' ? extensionTreeItem.tooltip : void 0,
-			command: extensionTreeItem.command ? this.commands.toInternal(extensionTreeItem.command) : void 0,
+			tooltip: typeof extensionTreeItem.tooltip === 'string' ? extensionTreeItem.tooltip : undefined,
+			command: extensionTreeItem.command ? this.commands.toInternal(extensionTreeItem.command) : undefined,
 			contextValue: extensionTreeItem.contextValue,
 			icon,
 			iconDark: this.getDarkIconPath(extensionTreeItem) || icon,
-			themeIcon: extensionTreeItem.iconPath instanceof ThemeIcon ? { id: extensionTreeItem.iconPath.id } : void 0,
+			themeIcon: extensionTreeItem.iconPath instanceof ThemeIcon ? { id: extensionTreeItem.iconPath.id } : undefined,
 			collapsibleState: isUndefinedOrNull(extensionTreeItem.collapsibleState) ? TreeItemCollapsibleState.None : extensionTreeItem.collapsibleState
 		};
 
@@ -450,7 +450,7 @@ class ExtHostTreeView<T> extends Disposable {
 		const prefix: string = parent ? parent.item.handle : ExtHostTreeView.LABEL_HANDLE_PREFIX;
 		let elementId = treeItemLabel ? treeItemLabel.label : resourceUri ? basename(resourceUri.path) : '';
 		elementId = elementId.indexOf('/') !== -1 ? elementId.replace('/', '//') : elementId;
-		const existingHandle = this.nodes.has(element) ? this.nodes.get(element).item.handle : void 0;
+		const existingHandle = this.nodes.has(element) ? this.nodes.get(element).item.handle : undefined;
 		const childrenNodes = (this.getChildrenNodes(parent) || []);
 
 		let handle: TreeItemHandle;
@@ -477,14 +477,14 @@ class ExtHostTreeView<T> extends Disposable {
 			}
 			return this.getIconPath(extensionTreeItem.iconPath['light']);
 		}
-		return void 0;
+		return undefined;
 	}
 
 	private getDarkIconPath(extensionTreeItem: vscode.TreeItem): URI {
 		if (extensionTreeItem.iconPath && !(extensionTreeItem.iconPath instanceof ThemeIcon) && extensionTreeItem.iconPath['dark']) {
 			return this.getIconPath(extensionTreeItem.iconPath['dark']);
 		}
-		return void 0;
+		return undefined;
 	}
 
 	private getIconPath(iconPath: string | URI): URI {
@@ -543,7 +543,7 @@ class ExtHostTreeView<T> extends Disposable {
 					}
 				}
 			}
-			node.children = void 0;
+			node.children = undefined;
 		} else {
 			this.clearAll();
 		}

src/vs/workbench/api/node/extHostTypeConverters.ts

@@ -111,7 +111,7 @@ export namespace Diagnostic {
 			...Range.from(value.range),
 			message: value.message,
 			source: value.source,
-			code: isString(value.code) || isNumber(value.code) ? String(value.code) : void 0,
+			code: isString(value.code) || isNumber(value.code) ? String(value.code) : undefined,
 			severity: DiagnosticSeverity.from(value.severity),
 			relatedInformation: value.relatedInformation && value.relatedInformation.map(DiagnosticRelatedInformation.from),
 			tags: Array.isArray(value.tags) ? value.tags.map(DiagnosticTag.from) : undefined,
@@ -929,7 +929,7 @@ export namespace FoldingRangeKind {
 					return modes.FoldingRangeKind.Region;
 			}
 		}
-		return void 0;
+		return undefined;
 	}
 }
 

src/vs/workbench/api/node/extHostTypes.ts

@@ -1466,7 +1466,7 @@ export class ProcessExecution implements vscode.ProcessExecution {
 			throw illegalArgument('process');
 		}
 		this._process = process;
-		if (varg1 !== void 0) {
+		if (varg1 !== undefined) {
 			if (Array.isArray(varg1)) {
 				this._args = varg1;
 				this._options = varg2;
@@ -1474,7 +1474,7 @@ export class ProcessExecution implements vscode.ProcessExecution {
 				this._options = varg1;
 			}
 		}
-		if (this._args === void 0) {
+		if (this._args === undefined) {
 			this._args = [];
 		}
 	}
@@ -1513,7 +1513,7 @@ export class ProcessExecution implements vscode.ProcessExecution {
 	public computeId(): string {
 		const hash = crypto.createHash('md5');
 		hash.update('process');
-		if (this._process !== void 0) {
+		if (this._process !== undefined) {
 			hash.update(this._process);
 		}
 		if (this._args && this._args.length > 0) {
@@ -1595,10 +1595,10 @@ export class ShellExecution implements vscode.ShellExecution {
 	public computeId(): string {
 		const hash = crypto.createHash('md5');
 		hash.update('shell');
-		if (this._commandLine !== void 0) {
+		if (this._commandLine !== undefined) {
 			hash.update(this._commandLine);
 		}
-		if (this._command !== void 0) {
+		if (this._command !== undefined) {
 			hash.update(typeof this._command === 'string' ? this._command : this._command.value);
 		}
 		if (this._args && this._args.length > 0) {
@@ -1688,7 +1688,7 @@ export class Task implements vscode.Task {
 	}
 
 	private clear(): void {
-		if (this.__id === void 0) {
+		if (this.__id === undefined) {
 			return;
 		}
 		this.__id = undefined;
@@ -1720,7 +1720,7 @@ export class Task implements vscode.Task {
 	}
 
 	set definition(value: vscode.TaskDefinition) {
-		if (value === void 0 || value === null) {
+		if (value === undefined || value === null) {
 			throw illegalArgument('Kind can\'t be undefined or null');
 		}
 		this.clear();

src/vs/workbench/api/node/extHostUrls.ts

@@ -45,7 +45,7 @@ export class ExtHostUrls implements ExtHostUrlsShape {
 		const handler = this.handlers.get(handle);
 
 		if (!handler) {
-			return Promise.resolve(void 0);
+			return Promise.resolve(undefined);
 		}
 		try {
 			handler.handleUri(URI.revive(uri));
@@ -53,6 +53,6 @@ export class ExtHostUrls implements ExtHostUrlsShape {
 			onUnexpectedError(err);
 		}
 
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 }

src/vs/workbench/api/node/extHostWebview.ts

@@ -311,7 +311,7 @@ export class ExtHostWebviews implements ExtHostWebviewsShape {
 			panel.dispose();
 			this._webviewPanels.delete(handle);
 		}
-		return Promise.resolve(void 0);
+		return Promise.resolve(undefined);
 	}
 
 	$deserializeWebviewPanel(

src/vs/workbench/api/node/extHostWorkspace.ts

@@ -233,7 +233,7 @@ export class ExtHostWorkspace implements ExtHostWorkspaceShape {
 		// Trigger on main side
 		if (this._proxy) {
 			const extName = extension.displayName || extension.name;
-			this._proxy.$updateWorkspaceFolders(extName, index, deleteCount, validatedDistinctWorkspaceFoldersToAdd).then(void 0, error => {
+			this._proxy.$updateWorkspaceFolders(extName, index, deleteCount, validatedDistinctWorkspaceFoldersToAdd).then(undefined, error => {
 
 				// in case of an error, make sure to clear out the unconfirmed workspace
 				// because we cannot expect the acknowledgement from the main side for this