Fix code after "apply suggestions"

This commit is contained in:
Aleksei Sidorov 2019-06-24 13:50:34 +03:00
parent c40ee089f2
commit 28e9e8d4cf
4 changed files with 21 additions and 15 deletions

View File

@ -201,7 +201,7 @@
],
"description": "Whether to run `cargo watch` on startup"
},
"rust-analyzer.cargo-watch.command-arguments": {
"rust-analyzer.cargo-watch.arguments": {
"type": "string",
"description": "`cargo-watch` arguments. (e.g: `--features=\"shumway,pdf\"` will run as `cargo watch -x \"check --features=\"shumway,pdf\"\"` )",
"default": ""
@ -209,7 +209,7 @@
"rust-analyzer.cargo-watch.command": {
"type": "string",
"description": "`cargo-watch` command. (e.g: `clippy` will run as `cargo watch -x clippy` )",
"default": "check"
"default": "check"
},
"rust-analyzer.trace.server": {
"type": "string",

View File

@ -43,7 +43,9 @@ export class CargoWatchProvider implements vscode.Disposable {
this.diagnosticCollection = vscode.languages.createDiagnosticCollection(
'rustc'
);
this.statusDisplay = new StatusDisplay(Server.config.cargoWatchOptions.checkCommand);
this.statusDisplay = new StatusDisplay(
Server.config.cargoWatchOptions.command
);
this.outputChannel = vscode.window.createOutputChannel(
'Cargo Watch Trace'
);
@ -57,10 +59,12 @@ export class CargoWatchProvider implements vscode.Disposable {
return;
}
let args = Server.config.cargoWatchOptions.checkCommand + ' --all-targets --message-format json';
if (Server.config.cargoWatchOptions.checkArguments.length > 0) {
let args =
Server.config.cargoWatchOptions.command +
' --all-targets --message-format json';
if (Server.config.cargoWatchOptions.command.length > 0) {
// Excape the double quote string:
args += ' ' + Server.config.cargoWatchOptions.checkArguments;
args += ' ' + Server.config.cargoWatchOptions.arguments;
}
// Windows handles arguments differently than the unix-likes, so we need to wrap the args in double quotes
if (process.platform === 'win32') {

View File

@ -30,7 +30,9 @@ export class StatusDisplay implements vscode.Disposable {
this.packageName
}] ${this.frame()}`;
} else {
this.statusBarItem!.text = `cargo ${this.command} ${this.frame()}`;
this.statusBarItem!.text = `cargo ${
this.command
} ${this.frame()}`;
}
}, 300);

View File

@ -25,8 +25,8 @@ export class Config {
public cargoWatchOptions: CargoWatchOptions = {
enableOnStartup: 'ask',
trace: 'off',
checkArguments: '',
checkCommand: ''
arguments: '',
command: ''
};
private prevEnhancedTyping: null | boolean = null;
@ -107,16 +107,16 @@ export class Config {
);
}
if (config.has('cargo-watch.check-arguments')) {
this.cargoWatchOptions.checkArguments = config.get<string>(
'cargo-watch.check-arguments',
if (config.has('cargo-watch.arguments')) {
this.cargoWatchOptions.arguments = config.get<string>(
'cargo-watch.arguments',
''
);
}
if (config.has('cargo-watch.check-command')) {
this.cargoWatchOptions.checkCommand = config.get<string>(
'cargo-watch.check-command',
if (config.has('cargo-watch.command')) {
this.cargoWatchOptions.command = config.get<string>(
'cargo-watch.command',
''
);
}