RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-29 03:57:44 +00:00
Code Issues Activity

df: remove trailing spaces in rightmost column (#3882)

Browse source 
Co-authored-by: Daniel Hofstetter <daniel.hofstetter@42dh.com>
This commit is contained in:
jfinkels 2022-09-22 02:37:20 -04:00 • committed by GitHub
parent 1e268b1a8f
commit dd0daa5e06
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 15 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/uu/df/src/table.rs
View file

@ -432,12 +432,21 @@ impl fmt::Display for Table {
while let Some(row) = row_iter.next() { while let Some(row) = row_iter.next() {
let mut col_iter = row.iter().enumerate().peekable(); let mut col_iter = row.iter().enumerate().peekable();
while let Some((i, elem)) = col_iter.next() { while let Some((i, elem)) = col_iter.next() {
let is_last_col = col_iter.peek().is_none();
match self.alignments[i] { match self.alignments[i] {
Alignment::Left => write!(f, "{:<width$}", elem, width = self.widths[i])?, Alignment::Left => {
if is_last_col {
// no trailing spaces in last column
write!(f, "{}", elem)?;
} else {
write!(f, "{:<width$}", elem, width = self.widths[i])?;
}
}
Alignment::Right => write!(f, "{:>width$}", elem, width = self.widths[i])?, Alignment::Right => write!(f, "{:>width$}", elem, width = self.widths[i])?,
} }
if col_iter.peek().is_some() { if !is_last_col {
// column separator // column separator
write!(f, " ")?; write!(f, " ")?;
} }

2
tests/by-util/test_df.rs
View file

@ -742,7 +742,7 @@ fn test_output_selects_columns() {
.args(&["--output=source"]) .args(&["--output=source"])
.succeeds() .succeeds()
.stdout_move_str(); .stdout_move_str();
assert_eq!(output.lines().next().unwrap().trim_end(), "Filesystem"); assert_eq!(output.lines().next().unwrap(), "Filesystem");
let output = new_ucmd!() let output = new_ucmd!()
.args(&["--output=source,target"]) .args(&["--output=source,target"])