ok/jj
1
0
Fork 0
forked from mirrors/jj
Code Pull requests Activity

cli: reuse number_padding instead of calculating twice in diff stat

Browse source
This commit is contained in:
Martin von Zweigbergk 2023-08-30 17:10:53 -07:00 committed by Martin von Zweigbergk
parent 5ce4ed45de
commit 5a054fcd20
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
cli/src/diff_util.rs
View file

@ -792,14 +792,14 @@ pub fn show_diff_stat(
}
let display_width = usize::from(ui.term_width().unwrap_or(80)) - 4; // padding
let number_padding = max_diffs.to_string().len();
let max_bar_length =
display_width.saturating_sub(max_path_length + " | ".len() + max_diffs.to_string().len());
display_width.saturating_sub(max_path_length + " | ".len() + number_padding);
let factor = if max_diffs < max_bar_length {
1.0
} else {
max_bar_length as f64 / max_diffs as f64
};
let number_padding = max_diffs.to_string().len();
formatter.with_label("diff", |formatter| {
let mut total_added = 0;