chore(lib/vscode): clean up comments

This commit is contained in:
Akash Satheesan 2021-04-16 00:39:02 +05:30
parent bdf555a06e
commit 578b2de131
No known key found for this signature in database
GPG Key ID: 93CBAADFEA703F8E
2 changed files with 13 additions and 10 deletions

View File

@ -192,12 +192,12 @@ export abstract class AbstractNativeEnvironmentService implements INativeEnviron
/** /**
* NOTE@coder: add extraExtensionPaths and extraBuiltinExtensionPaths * NOTE@coder: add extraExtensionPaths and extraBuiltinExtensionPaths
* Code location changed after 1.54 (was earlier directly in NativeEnvironmentService).
*/ */
@memoize @memoize
get extraExtensionPaths(): string[] { get extraExtensionPaths(): string[] {
return (this._args['extra-extensions-dir'] || []).map((p) => <string>parsePathArg(p, process)); return (this._args['extra-extensions-dir'] || []).map((p) => <string>parsePathArg(p, process));
} }
@memoize @memoize
get extraBuiltinExtensionPaths(): string[] { get extraBuiltinExtensionPaths(): string[] {
return (this._args['extra-builtin-extensions-dir'] || []).map((p) => <string>parsePathArg(p, process)); return (this._args['extra-builtin-extensions-dir'] || []).map((p) => <string>parsePathArg(p, process));

View File

@ -481,22 +481,25 @@ export class TerminalProviderChannel implements IServerChannel<RemoteAgentConnec
public listen(_: RemoteAgentConnectionContext, event: string, args: any): Event<any> { public listen(_: RemoteAgentConnectionContext, event: string, args: any): Event<any> {
logger.trace('TerminalProviderChannel:listen', field('event', event), field('args', args)); logger.trace('TerminalProviderChannel:listen', field('event', event), field('args', args));
// TODO@oxy/code-asher: implement these events (currently Event.None) as needed
// Right now, most functionality tested works;
// but VSCode might rely on the other events in the future.
switch (event) { switch (event) {
case '$onPtyHostExitEvent': return Event.None; // TODO case '$onPtyHostExitEvent': return Event.None;
case '$onPtyHostStartEvent': return Event.None; // TODO case '$onPtyHostStartEvent': return Event.None;
case '$onPtyHostUnresponsiveEvent': return Event.None; // TODO case '$onPtyHostUnresponsiveEvent': return Event.None;
case '$onPtyHostResponsiveEvent': return Event.None; // TODO case '$onPtyHostResponsiveEvent': return Event.None;
case '$onProcessDataEvent': return this._onProcessData.event; case '$onProcessDataEvent': return this._onProcessData.event;
case '$onProcessExitEvent': return this._onProcessExit.event; case '$onProcessExitEvent': return this._onProcessExit.event;
case '$onProcessReadyEvent': return this._onProcessReady.event; case '$onProcessReadyEvent': return this._onProcessReady.event;
case '$onProcessReplayEvent': return this._onProcessReplay.event; case '$onProcessReplayEvent': return this._onProcessReplay.event;
case '$onProcessTitleChangedEvent': return this._onProcessTitleChanged.event; case '$onProcessTitleChangedEvent': return this._onProcessTitleChanged.event;
case '$onProcessShellTypeChangedEvent': return Event.None; // TODO; case '$onProcessShellTypeChangedEvent': return Event.None;
case '$onProcessOverrideDimensionsEvent': return Event.None; // TODO; case '$onProcessOverrideDimensionsEvent': return Event.None;
case '$onProcessResolvedShellLaunchConfigEvent': return Event.None; // TODO; case '$onProcessResolvedShellLaunchConfigEvent': return Event.None;
case '$onProcessOrphanQuestion': return Event.None; // TODO case '$onProcessOrphanQuestion': return Event.None;
// TODO: I think this must have something to do with running commands on // NOTE@code-asher: I think this must have something to do with running commands on
// the terminal that will do things in VS Code but we already have that // the terminal that will do things in VS Code but we already have that
// functionality via a socket so I'm not sure what this is for. // functionality via a socket so I'm not sure what this is for.
case '$onExecuteCommand': return Event.None; case '$onExecuteCommand': return Event.None;