Skip to content

Commit 0522232

Browse files
DonJayamanneDonJayamanne
committed
removed unnecessary comments
1 parent 8051124 commit 0522232

13 files changed

Lines changed: 75 additions & 131 deletions

‎src/client/formatters/autoPep8Formatter.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';

‎src/client/formatters/baseFormatter.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';

‎src/client/formatters/yapfFormatter.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';

‎src/client/linters/pylint.ts‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,8 @@
33
import * as path from 'path';
44
import * as baseLinter from './baseLinter';
55
import {OutputChannel, workspace} from 'vscode';
6-
76
const PYLINT_COMMANDLINE = " --msg-template='{line},{column},{category},{msg_id}:{msg}' --reports=n --output-format=text";
87

9-
108
export class Linter extends baseLinter.BaseLinter {
119
constructor(outputChannel: OutputChannel) {
1210
super("pylint", outputChannel);

‎src/client/providers/completionProvider.ts‎

Lines changed: 11 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,24 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';
84
import * as proxy from './jediProxy';
95

10-
function parseData(data: proxy.ICompletionResult): vscode.CompletionItem[] {
11-
if (data && data.items.length > 0) {
12-
return data.items.map(item=> {
13-
var completionItem = new vscode.CompletionItem(item.text);
14-
completionItem.documentation = item.description;
15-
return completionItem;
16-
});
17-
}
18-
return [];
19-
}
20-
216
export class PythonCompletionItemProvider implements vscode.CompletionItemProvider {
227
private jediProxyHandler: proxy.JediProxyHandler<proxy.ICompletionResult, vscode.CompletionItem[]>;
238

249
public constructor(context: vscode.ExtensionContext) {
25-
this.jediProxyHandler = new proxy.JediProxyHandler(context, [], parseData);
10+
this.jediProxyHandler = new proxy.JediProxyHandler(context, [], PythonCompletionItemProvider.parseData);
11+
}
12+
private static parseData(data: proxy.ICompletionResult): vscode.CompletionItem[] {
13+
if (data && data.items.length > 0) {
14+
return data.items.map(item => {
15+
var completionItem = new vscode.CompletionItem(item.text);
16+
completionItem.documentation = item.description;
17+
return completionItem;
18+
});
19+
}
20+
return [];
2621
}
27-
2822
public provideCompletionItems(document: vscode.TextDocument, position: vscode.Position, token: vscode.CancellationToken): Thenable<vscode.CompletionItem[]> {
2923
return new Promise<vscode.CompletionItem[]>((resolve, reject) => {
3024
var filename = document.fileName;
@@ -51,5 +45,4 @@ export class PythonCompletionItemProvider implements vscode.CompletionItemProvid
5145
this.jediProxyHandler.sendCommand(cmd, resolve, token);
5246
});
5347
}
54-
5548
}

‎src/client/providers/definitionProvider.ts‎

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,24 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';
84
import * as proxy from './jediProxy';
95
import * as fs from 'fs';
106

11-
function parseData(data: proxy.IDefinitionResult): vscode.Definition {
12-
if (data) {
13-
var definitionResource = vscode.Uri.file(data.definition.fileName);
14-
var range = new vscode.Range(data.definition.lineIndex, data.definition.columnIndex, data.definition.lineIndex, data.definition.columnIndex);
15-
16-
return new vscode.Location(definitionResource, range);
17-
}
18-
return null;
19-
}
20-
217
export class PythonDefinitionProvider implements vscode.DefinitionProvider {
228
private jediProxyHandler: proxy.JediProxyHandler<proxy.IDefinitionResult, vscode.Definition>;
239

2410
public constructor(context: vscode.ExtensionContext) {
25-
this.jediProxyHandler = new proxy.JediProxyHandler(context, null, parseData);
11+
this.jediProxyHandler = new proxy.JediProxyHandler(context, null, PythonDefinitionProvider.parseData);
12+
}
13+
private static parseData(data: proxy.IDefinitionResult): vscode.Definition {
14+
if (data) {
15+
var definitionResource = vscode.Uri.file(data.definition.fileName);
16+
var range = new vscode.Range(data.definition.lineIndex, data.definition.columnIndex, data.definition.lineIndex, data.definition.columnIndex);
17+
18+
return new vscode.Location(definitionResource, range);
19+
}
20+
return null;
2621
}
27-
2822
public provideDefinition(document: vscode.TextDocument, position: vscode.Position, token: vscode.CancellationToken): Thenable<vscode.Definition> {
2923
return new Promise<vscode.Definition>((resolve, reject) => {
3024
var filename = document.fileName;

‎src/client/providers/formatProvider.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';

‎src/client/providers/hoverProvider.ts‎

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,24 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import * as vscode from 'vscode';
84
import * as proxy from './jediProxy';
95

10-
function parseData(data: proxy.ICompletionResult) {
11-
if (data && data.items.length > 0) {
12-
var definition = data.items[0];
13-
14-
var txt = definition.description || definition.text;
15-
return new vscode.Hover({ language: "python", value: txt });
16-
}
17-
return null;
18-
}
196

207
export class PythonHoverProvider implements vscode.HoverProvider {
218
private jediProxyHandler: proxy.JediProxyHandler<proxy.ICompletionResult, vscode.Hover>;
229

2310
public constructor(context: vscode.ExtensionContext) {
24-
this.jediProxyHandler = new proxy.JediProxyHandler(context, null, parseData);
11+
this.jediProxyHandler = new proxy.JediProxyHandler(context, null, PythonHoverProvider.parseData);
12+
}
13+
private static parseData(data: proxy.ICompletionResult) {
14+
if (data && data.items.length > 0) {
15+
var definition = data.items[0];
16+
17+
var txt = definition.description || definition.text;
18+
return new vscode.Hover({ language: "python", value: txt });
19+
}
20+
return null;
2521
}
26-
2722
public provideHover(document: vscode.TextDocument, position: vscode.Position, token: vscode.CancellationToken): Thenable<vscode.Hover> {
2823
return new Promise<vscode.Hover>((resolve, reject) => {
2924
var filename = document.fileName;

‎src/client/providers/importSortProvider.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
/*---------------------------------------------------------
2-
* Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
4-
51
'use strict';
62

73
import vscode = require('vscode');

‎src/client/providers/lintProvider.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
/*---------------------------------------------------------
2-
** Copyright (C) Microsoft Corporation. All rights reserved.
3-
*--------------------------------------------------------*/
41
'use strict';
52

63
import * as vscode from 'vscode';

0 commit comments

Comments
 (0)