Skip to content

Commit

Permalink
Auto merge of #8892 - nnethercote:shrink-progress-bar, r=alexcrichton
Browse files Browse the repository at this point in the history
Shrink the progress bar, to give more space after it.

Because:
- A big progress bar isn't necessary. Making it smaller loses some
  precision, but a big bar isn't that precise in the first place. If you
  precision you can look at the percentage or ratio figure after the
  progress bar anyway.
- A big bar sometimes obscures important info. For the "Building"
  step the crate names after the progress bar are important -- they make
  it clear how much parallelism is in the build. If you don't see that
  you're getting a much worse understanding of the build process.

On an 80-char wide terminal, the change looks like this. Before:
```
    Building [========>                                               ] 18/105
```
After:
```
    Building [=======================>  ] 97/105: goblin, pdb, symbolic-dem...
```

On a 100-char wide terminal, it looks like this. Before:
```
    Building [============================>                           ] 56/105: cpp_demangle(bu...
```
After:
```
    Building [============>             ] 56/105: byteorder, proc-macro2, pest, gimli, cpp_dema...
```
  • Loading branch information
bors committed Nov 24, 2020
2 parents 02a3a02 + 7e780a9 commit bfca1cd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/cargo/util/progress.rs
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,9 @@ impl<'cfg> Progress<'cfg> {
format: Format {
style,
max_width: n,
max_print: 80,
// 50 gives some space for text after the progress bar,
// even on narrow (e.g. 80 char) terminals.
max_print: 50,
},
name: name.to_string(),
done: false,
Expand Down

0 comments on commit bfca1cd

Please sign in to comment.