From e71be1292b92b244d065873fae5a17d5e1f0a16c Mon Sep 17 00:00:00 2001 From: =?utf8?q?P=C3=A1draig=20Brady?= Date: Wed, 21 Oct 2015 13:57:41 +0100 Subject: [PATCH] ls: fix off by one error when determining max display columns * src/ls.c (main): Account for the first column not including a separator when calculating max_idx. * tests/ls/w-option.sh: Add a test case. * NEWS: Mention the bug fix. --- NEWS | 3 +++ src/ls.c | 6 +++++- tests/ls/w-option.sh | 4 ++++ 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 07b88b09cb..e771585625 100644 --- a/NEWS +++ b/NEWS @@ -4,6 +4,9 @@ GNU coreutils NEWS -*- outline -*- ** Bug fixes + ls no longer prematurely wraps lines when printing short file names. + [bug introduced in 5.1.0] + shred again uses defined patterns for all iteration counts. [bug introduced in coreutils-5.93] diff --git a/src/ls.c b/src/ls.c index 0c9dc78db4..ef372553f1 100644 --- a/src/ls.c +++ b/src/ls.c @@ -1979,7 +1979,11 @@ decode_switches (int argc, char **argv) } } - max_idx = MAX (1, line_length / MIN_COLUMN_WIDTH); + /* Determine the max possible number of display columns. */ + max_idx = line_length / MIN_COLUMN_WIDTH; + /* Account for first display column not having a separator, + or line_lengths shorter than MIN_COLUMN_WIDTH. */ + max_idx += line_length % MIN_COLUMN_WIDTH != 0; filename_quoting_options = clone_quoting_options (NULL); if (get_quoting_style (filename_quoting_options) == escape_quoting_style) diff --git a/tests/ls/w-option.sh b/tests/ls/w-option.sh index f49c02815d..6361aaf597 100755 --- a/tests/ls/w-option.sh +++ b/tests/ls/w-option.sh @@ -41,4 +41,8 @@ compare exp out || fail=1 # Ensure that 0 line length doesn't cause division by zero TERM=xterm ls -w0 -x --color=always || fail=1 +# coreutils <= 8.24 could display 1 column too few +ls -w4 -x -T0 a b > out || fail=1 +compare exp out || fail=1 + Exit $fail -- 2.47.2