diff options
author | Eric Blake <ebb9@byu.net> | 2010-01-06 20:57:40 -0700 |
---|---|---|
committer | Eric Blake <ebb9@byu.net> | 2010-01-06 21:07:23 -0700 |
commit | 67d3e263504115144380dfaffd41c9cef3cf68a4 (patch) | |
tree | 5cf755723247f15a892bdaf54227e3833c9ebfbd /src | |
parent | 72a0a8264d1eeacfc92231e71d75d2eb7ee3e08f (diff) | |
download | coreutils-67d3e263504115144380dfaffd41c9cef3cf68a4.tar.xz |
pr: ensure the page header line is of the required format
Before this change, with too long a file name, the name would
abut the date field on the left and possibly also the "Page N"
field on the right, rather than leaving a one-space separator
in each case. Fixes a regression introduced on Mar 6 2009,
by commit a4053c5291d5797734b3e4f042f9e1adf3944fd6
* src/pr.c (print_header): Ensure that there is at least one
space before and after the file name part of the header line.
* NEWS: Mention it.
* tests/pr/W20l24f-ll: s/xPage/ x Page/.
* THANKS: Update.
Reported by Denis McKeon, in https://savannah.gnu.org/bugs/?28492.
Diffstat (limited to 'src')
-rw-r--r-- | src/pr.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -2403,10 +2403,10 @@ print_header (void) lhs_spaces = available_width >> 1; rhs_spaces = available_width - lhs_spaces; - printf ("\n\n%*.*s%s%*.*s%s%*.*s%s\n\n\n", - chars_per_margin, chars_per_margin, " ", - date_text, lhs_spaces, lhs_spaces, " ", - file_text, rhs_spaces, rhs_spaces, " ", page_text); + printf ("\n\n%*s%s%*s%s%*s%s\n\n\n", + chars_per_margin, "", + date_text, lhs_spaces, " ", + file_text, rhs_spaces, " ", page_text); print_a_header = false; output_position = 0; |