From a7fbcb8156747c64932432d929d7bcfcfe810cfb Mon Sep 17 00:00:00 2001 From: "andy.boot" Date: Thu, 18 Aug 2022 15:03:37 +0100 Subject: [PATCH] Refactor: Simplify get_pretty_size Remove the constant '5' used in format. Now all paths use num_chars_needed_on_left_most instead. --- src/display.rs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/display.rs b/src/display.rs index 7ac7a8a..ab05f3c 100644 --- a/src/display.rs +++ b/src/display.rs @@ -344,13 +344,12 @@ fn get_name_percent( fn get_pretty_size(node: &DisplayNode, is_biggest: bool, display_data: &DisplayData) -> String { let output = if display_data.by_filecount { - let size_as_str = node.size.separate_with_commas(); - let spaces_to_add = - display_data.num_chars_needed_on_left_most - size_as_str.chars().count(); - size_as_str + " ".repeat(spaces_to_add).as_str() + node.size.separate_with_commas() } else { - format!("{:>5}", human_readable_number(node.size, display_data.iso)) + human_readable_number(node.size, display_data.iso) }; + let spaces_to_add = display_data.num_chars_needed_on_left_most - output.chars().count(); + let output = output + " ".repeat(spaces_to_add).as_str(); if is_biggest && display_data.colors_on { format!("{}", Red.paint(output))