blasthavers/minicrossterm
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid double evaluation of expression in handle_command! (#349)

Browse Source
This commit is contained in:
Canop 2019-12-16 17:33:50 +01:00 committed by Timon
parent b3cf303295
commit aad7aaced8
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/macros.rs
View File

@ -32,14 +32,14 @@ macro_rules! handle_command {
#[cfg(windows)] #[cfg(windows)]
{ {
let command = $command;
// ansi code is not always a string, however it does implement `Display` and `Write`. // ansi code is not always a string, however it does implement `Display` and `Write`.
// In order to check if the code is supported we have to make it a string. // In order to check if the code is supported we have to make it a string.
let ansi_code = format!("{}", $command.ansi_code()); let ansi_code = format!("{}", command.ansi_code());
if command.is_ansi_code_supported() {
if $command.is_ansi_code_supported() {
write_ansi_code!($writer, &ansi_code) write_ansi_code!($writer, &ansi_code)
} else { } else {
$command.execute_winapi().map_err($crate::ErrorKind::from) command.execute_winapi().map_err($crate::ErrorKind::from)
} }
} }
#[cfg(unix)] #[cfg(unix)]