Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Commit

Permalink
use better name 'resolveConfigAsync'
Browse files Browse the repository at this point in the history
  • Loading branch information
weinand committed Mar 16, 2018
1 parent 5496786 commit 8721a2c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/node/extension/configurationProvider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,15 @@ export class NodeConfigurationProvider implements vscode.DebugConfigurationProvi
* Try to add all missing attributes to the debug configuration being launched.
*/
resolveDebugConfiguration(folder: vscode.WorkspaceFolder | undefined, config: vscode.DebugConfiguration, token?: vscode.CancellationToken): vscode.ProviderResult<vscode.DebugConfiguration> {
return this.resolveDebugConfiguration2(folder, config).catch(err => {
return this.resolveConfigAsync(folder, config).catch(err => {
return vscode.window.showErrorMessage(err.message, { modal: true }).then(_ => undefined); // abort launch
});
}

/**
* Try to add all missing attributes to the debug configuration being launched.
*/
private async resolveDebugConfiguration2(folder: vscode.WorkspaceFolder | undefined, config: vscode.DebugConfiguration, token?: vscode.CancellationToken): Promise<vscode.DebugConfiguration | undefined> {
private async resolveConfigAsync(folder: vscode.WorkspaceFolder | undefined, config: vscode.DebugConfiguration, token?: vscode.CancellationToken): Promise<vscode.DebugConfiguration | undefined> {

// if launch.json is missing or empty
if (!config.type && !config.request && !config.name) {
Expand Down

0 comments on commit 8721a2c

Please sign in to comment.