Merge pull request DonJayamanne#39 from jxem/fixFormatCommands

DonJayamanne · DonJayamanne · commit 443071869d2e · 2016-03-11T20:17:57.000+11:00
Added quotes around format-commands to prevent space errors.
diff --git a/src/client/formatters/autoPep8Formatter.ts b/src/client/formatters/autoPep8Formatter.ts
@@ -19,6 +19,6 @@ export class AutoPep8Formatter extends BaseFormatter {
     public formatDocument(document: vscode.TextDocument, options: vscode.FormattingOptions, token: vscode.CancellationToken): Thenable<vscode.TextEdit[]> {
         var autopep8Path = this.pythonSettings.formatting.autopep8Path;
         var fileDir = path.dirname(document.uri.fsPath);
-        return super.provideDocumentFormattingEdits(document, options, token, `${autopep8Path} ${document.uri.fsPath}`);
+        return super.provideDocumentFormattingEdits(document, options, token, `${autopep8Path} "${document.uri.fsPath}"`);
     }
 }
diff --git a/src/client/formatters/yapfFormatter.ts b/src/client/formatters/yapfFormatter.ts
@@ -19,6 +19,6 @@ export class YapfFormatter extends BaseFormatter {
     public formatDocument(document: vscode.TextDocument, options: vscode.FormattingOptions, token: vscode.CancellationToken): Thenable<vscode.TextEdit[]> {
         var yapfPath = this.pythonSettings.formatting.yapfPath;
         var fileDir = path.dirname(document.uri.fsPath);
-        return super.provideDocumentFormattingEdits(document, options, token, `${yapfPath} ${document.uri.fsPath}`);
+        return super.provideDocumentFormattingEdits(document, options, token, `${yapfPath} "${document.uri.fsPath}"`);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,6 @@ export class AutoPep8Formatter extends BaseFormatter {`
`19`	`19`	`public formatDocument(document: vscode.TextDocument, options: vscode.FormattingOptions, token: vscode.CancellationToken): Thenable<vscode.TextEdit[]> {`
`20`	`20`	`var autopep8Path = this.pythonSettings.formatting.autopep8Path;`
`21`	`21`	`var fileDir = path.dirname(document.uri.fsPath);`
`22`		- return super.provideDocumentFormattingEdits(document, options, token, `${autopep8Path} ${document.uri.fsPath}`);
	`22`	+ return super.provideDocumentFormattingEdits(document, options, token, `${autopep8Path} "${document.uri.fsPath}"`);
`23`	`23`	`}`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,6 @@ export class YapfFormatter extends BaseFormatter {`
`19`	`19`	`public formatDocument(document: vscode.TextDocument, options: vscode.FormattingOptions, token: vscode.CancellationToken): Thenable<vscode.TextEdit[]> {`
`20`	`20`	`var yapfPath = this.pythonSettings.formatting.yapfPath;`
`21`	`21`	`var fileDir = path.dirname(document.uri.fsPath);`
`22`		- return super.provideDocumentFormattingEdits(document, options, token, `${yapfPath} ${document.uri.fsPath}`);
	`22`	+ return super.provideDocumentFormattingEdits(document, options, token, `${yapfPath} "${document.uri.fsPath}"`);
`23`	`23`	`}`
`24`	`24`	`}`