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

Commit

Permalink
Fix microsoft/vscode#53785 - setBreakpoints override no longer needed…
Browse files Browse the repository at this point in the history
…, it will map remote paths twice
  • Loading branch information
roblourens committed Jul 9, 2018
1 parent 567f554 commit 9f0d734
Showing 1 changed file with 2 additions and 11 deletions.
13 changes: 2 additions & 11 deletions src/transformers/remotePathTransformer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ import * as fs from 'fs';
import * as path from 'path';
import { logger } from 'vscode-debugadapter';
import { DebugProtocol } from 'vscode-debugprotocol';
import { UrlPathTransformer } from '../transformers/urlPathTransformer';
import { IAttachRequestArgs, ICommonRequestArgs, ILaunchRequestArgs, ISetBreakpointsArgs, IStackTraceResponseBody } from '../debugAdapterInterfaces';
import { IAttachRequestArgs, ICommonRequestArgs, ILaunchRequestArgs, IStackTraceResponseBody } from '../debugAdapterInterfaces';
import * as errors from '../errors';
import { UrlPathTransformer } from '../transformers/urlPathTransformer';
import * as utils from '../utils';

/**
Expand Down Expand Up @@ -55,15 +55,6 @@ export class RemotePathTransformer extends UrlPathTransformer {
return localRootP;
}

public setBreakpoints(args: ISetBreakpointsArgs): ISetBreakpointsArgs {
args = super.setBreakpoints(args);
if (args.source.path) {
args.source.path = this.getTargetPathFromClientPath(args.source.path) || args.source.path;
}

return args;
}

public async scriptParsed(scriptPath: string): Promise<string> {
scriptPath = await super.scriptParsed(scriptPath);
scriptPath = this.getClientPathFromTargetPath(scriptPath) || scriptPath;
Expand Down

0 comments on commit 9f0d734

Please sign in to comment.