Skip to content

Commit

Permalink
Specify that we need the width of stdout
Browse files Browse the repository at this point in the history
The term_size crate introduced in #237 did things *slightly* differently than exa: it tried to get the terminal width of stdout, stderr, and stdin. This broke some tests that only redirected stdout.
  • Loading branch information
ogham committed Aug 5, 2017
1 parent d701547 commit 4289f4d
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/options/view.rs
Original file line number Diff line number Diff line change
Expand Up @@ -392,7 +392,10 @@ impl Classify {
// so it’s easier to just cache it the first time it runs.
lazy_static! {
static ref TERM_WIDTH: Option<usize> = {
use term_size::dimensions;
dimensions().map(|t| t.0)
// All of stdin, stdout, and stderr could not be connected to a
// terminal, but we’re only interested in stdout because it’s
// where the output goes.
use term_size::dimensions_stdout;
dimensions_stdout().map(|t| t.0)
};
}

0 comments on commit 4289f4d

Please sign in to comment.