pr: fix a bug whereby --indent=N (-o) did not indent header lines
authorJim Meyering <meyering@redhat.com>
Fri, 6 Mar 2009 19:03:26 +0000 (20:03 +0100)
committerJim Meyering <meyering@redhat.com>
Sat, 7 Mar 2009 22:52:08 +0000 (23:52 +0100)
* src/pr.c (print_header): Honor chars_per_margin also for the header.
Inspired by a patch from C de-Avillez.
This change also adjusts two other %*s directives to be %*.*s instead,
(for the padding on either side of "file_text") to correct a bug that
could make pr print one space of padding instead of none.
* NEWS (Bug fixes): Mention it.
* tests/pr/o3Jml24f-lm-lo: Adjust for new expected output.
* tests/pr/o3a3Sl24f-tn: Likewise.
* tests/pr/o3a3Snl24f-tn: Likewise.
* tests/pr/o3a3l24f-tn: Likewise.
* tests/pr/o3b3Sl24f-tn: Likewise.
* tests/pr/o3b3Snl24f-tn: Likewise.
* tests/pr/o3b3l24f-tn: Likewise.
* tests/pr/o3mSl24f-bl-tn: Likewise.
* tests/pr/o3mSnl24fbltn: Likewise.
* tests/pr/o3ml24f-bl-tn: Likewise.

14 files changed:
NEWS
THANKS
src/pr.c
tests/pr/W20l24f-ll
tests/pr/o3Jml24f-lm-lo
tests/pr/o3a3Sl24f-tn
tests/pr/o3a3Snl24f-tn
tests/pr/o3a3l24f-tn
tests/pr/o3b3Sl24f-tn
tests/pr/o3b3Snl24f-tn
tests/pr/o3b3l24f-tn
tests/pr/o3mSl24f-bl-tn
tests/pr/o3mSnl24fbltn
tests/pr/o3ml24f-bl-tn

diff --git a/NEWS b/NEWS
index 87589c4..0f6e853 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -9,6 +9,9 @@ GNU coreutils NEWS                                    -*- outline -*-
   The bug strikes only with both --recursive (-r, -R) and --link (-l).
   [bug introduced in coreutils-7.1]
 
+  pr: fix the bug whereby --indent=N (-o) did not indent header lines
+  [bug introduced in coreutils-6.9.90]
+
   sort now handles specified key ends correctly.
   Previously -k1,1b would have caused leading space from field 2 to be
   included in the sort while -k2,3.0 would have not included field 3.
diff --git a/THANKS b/THANKS
index c4e900b..e40d2b0 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -89,6 +89,7 @@ Brian Youmans                       3diff@gnu.org
 Bruce Korb                          bkorb@veritas.com
 Bruce Robertson                     brucer@theodolite.dyndns.org
 Bruno Haible                        haible@clisp.cons.org
+C de-Avillez                        hggdh2@gmail.com
 Carl Johnson                        carlj@cjlinux.home.org
 Carl Lowenstein                     cdl@mpl.UCSD.EDU
 Carl Roth                           roth@urs.us
index 0c02430..3b6e801 100644 (file)
--- a/src/pr.c
+++ b/src/pr.c
@@ -2402,8 +2402,10 @@ print_header (void)
   lhs_spaces = available_width >> 1;
   rhs_spaces = available_width - lhs_spaces;
 
-  printf ("\n\n%s%*s%s%*s%s\n\n\n",
-         date_text, lhs_spaces, " ", file_text, rhs_spaces, " ", page_text);
+  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);
 
   print_a_header = false;
   output_position = 0;
index ecc9a7a..92d08af 100644 (file)
@@ -1,6 +1,6 @@
 
 
--- Date/Time -- x Page 1
+-- Date/Time --xPage 1
 
 
 1<<<  -Test: FF's in
 14<<<  123456789 123
 \f
 
--- Date/Time -- x Page 2
+-- Date/Time --xPage 2
 
 
 
 \f
 
--- Date/Time -- x Page 3
+-- Date/Time --xPage 3
 
 
 15<<<  xyzxyzxyz XYZ
 28<<<  trunc
 \f
 
--- Date/Time -- x Page 4
+-- Date/Time --xPage 4
 
 
 
 \f
 
--- Date/Time -- x Page 5
+-- Date/Time --xPage 5
 
 
 29<<<xyzxyzxyz XYZXY
 42<<<  123456789 abc
 \f
 
--- Date/Time -- x Page 6
+-- Date/Time --xPage 6
 
 
 
 \f
 
--- Date/Time -- x Page 7
+-- Date/Time --xPage 7
 
 
 
 \f
 
--- Date/Time -- x Page 8
+-- Date/Time --xPage 8
 
 
 43<<<  xyzxyzxyz XYZ
 56<<<  123456789 abc
 \f
 
--- Date/Time -- x Page 9
+-- Date/Time --xPage 9
 
 
 57<<<  xyzxyzxyz XYZ
index a141835..e0aa4a4 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1<<<         -Test: FF's in Text                    >>>     1:::  Test-INPUT: "Without FF set by Hand"        :::
@@ -19,7 +19,7 @@
    14<<<  123456789 123456789 123456789         >>>    14:::  89 123456789 123456789 123456789 123456789 123456789 123456789 1234567890  :::
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
        15:::  xyzxyzxyz XYZXYZXYZ abcabcab      :::
@@ -38,7 +38,7 @@
        28:::  89 123456789 123456789 123456789 123456789 123456789 123456789 1234567890  :::
 \f   
 
--- Date/Time --                         x                         Page 3
+   -- Date/Time --                         x                         Page 3
 
 
    15<<<  xyzxyzxyz XYZXYZXYZ abcabcab >>>     29:::  xyzxyzxyz XYZXYZXYZ abcabcab      :::
@@ -57,7 +57,7 @@
    28<<<  trunc        42:::  89 123456789 123456789 123456789 123456789 123456789 123456789 1234567890  :::
 \f   
 
--- Date/Time --                         x                         Page 4
+   -- Date/Time --                         x                         Page 4
 
 
        43:::  xyzxyzxyz XYZXYZXYZ abcabcab      :::
@@ -76,7 +76,7 @@
        56:::  89 123456789 123456789 123456789 123456789 123456789 123456789 1234567890  :::
 \f   
 
--- Date/Time --                         x                         Page 5
+   -- Date/Time --                         x                         Page 5
 
 
    29<<<xyzxyzxyz XYZXYZXYZ abcabcab  >>>      57:::    xyzxyzxyz XYZXYZXYZ abcabcab  :::
    42<<<  123456789 abcdefghi ABCDEDFHI         >>>
 \f   
 
--- Date/Time --                         x                         Page 6
+   -- Date/Time --                         x                         Page 6
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 7
+   -- Date/Time --                         x                         Page 7
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 8
+   -- Date/Time --                         x                         Page 8
 
 
    43<<<  xyzxyzxyz XYZXYZXYZ abcabcab >>>     
    56<<<  123456789 abcdefghi ABCDEDFHI         >>>
 \f   
 
--- Date/Time --                         x                         Page 9
+   -- Date/Time --                         x                         Page 9
 
 
    57<<<  xyzxyzxyz XYZXYZXYZ abcabcab >>>     
index 8986aea..0064f2f 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   Test-INPUT: "With:--:2  Options -b -3 [+2:--:3   Options -a -3 [+2
@@ -19,7 +19,7 @@
    40 DEFGHI 123456789 :--:41 yzxyzxyz XYZXYZXYZ:--:42 456789 123456789 a
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
    43 xyzxyzxyz XYZXYZXY:--:44 456789 123456789 x:--:5
index 2560cec..5a8f263 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
        1.1   Test-INPUT::--:   2.2   Options -b :--:    3.3   Options -a
@@ -19,7 +19,7 @@
       40.40 DEFGHI 12345:--:   41.41 yzxyzxyz XYZ:--:  42.42 456789 12345
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
       43.43 xyzxyzxyz XY:--:   44.44 456789 12345:--:  45.5
index b2d8a42..9adbe67 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   Test-INPUT: "Withou 2   Options -b -3 [+2|+ 3   Options -a -3 [+2|+
@@ -19,7 +19,7 @@
    40 DEFGHI 123456789    41 yzxyzxyz XYZXYZXYZ a 42 456789 123456789 abc
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
    43 xyzxyzxyz XYZXYZXYZ  44 456789 123456789 xyz 5
index 6294d91..2e11983 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   Test-INPUT: "With:--:15 xyzxyzxyz XYZXYZXY:--:29 xyzxyzxyz XYZXYZXY
@@ -19,7 +19,7 @@
    14 456789 123456789 1:--:28 no trunc                 :--:42 456789 123456789 a
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
    43 xyzxyzxyz XYZXYZXY:--:50                  :--:57 xyzxyzxyz XYZXYZXY
index 200b780..1b945fe 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
        1.1   Test-INPUT::--:   15.15 xyzxyzxyz XY:--:  29.29 xyzxyzxyz XY
@@ -19,7 +19,7 @@
       14.14 456789 12345:--:   28.28 no trunc   :--:   42.42 456789 12345
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
       43.43 xyzxyzxyz XY:--:   50.50            :--:   57.57 xyzxyzxyz XY
index 73309b6..0d237bb 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   Test-INPUT: "Withou 15 xyzxyzxyz XYZXYZXYZ  29 xyzxyzxyz XYZXYZXYZ
@@ -19,7 +19,7 @@
    14 456789 123456789 123 28 no trunc            42 456789 123456789 abc
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
    43 xyzxyzxyz XYZXYZXYZ  50                     57 xyzxyzxyz XYZXYZXYZ
index adac88e..c90a2b4 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   FF-Test: FF's in Text        :--:1   Test-INPUT: "Without FF set by
@@ -19,7 +19,7 @@
    14 456789 123456789 123456789 1234:--:14 456789 123456789 123456789 1234
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
                                     :--:15 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -38,7 +38,7 @@
                                     :--:28 no trunc
 \f   
 
--- Date/Time --                         x                         Page 3
+   -- Date/Time --                         x                         Page 3
 
 
    15 xyzxyzxyz XYZXYZXYZ abcabcab   :--:29 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -57,7 +57,7 @@
                                     :--:42 456789 123456789 abcdefghi ABCD
 \f   
 
--- Date/Time --                         x                         Page 4
+   -- Date/Time --                         x                         Page 4
 
 
                                     :--:43 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -76,7 +76,7 @@
                                     :--:56 456789 123456789 abcdefghi ABCD
 \f   
 
--- Date/Time --                         x                         Page 5
+   -- Date/Time --                         x                         Page 5
 
 
    29 xyzxyzxyz XYZXYZXYZ abcabcab   :--:57 xyzxyzxyz XYZXYZXYZ abcabcab
    42 456789 123456789 abcdefghi ABCD
 \f   
 
--- Date/Time --                         x                         Page 6
+   -- Date/Time --                         x                         Page 6
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 7
+   -- Date/Time --                         x                         Page 7
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 8
+   -- Date/Time --                         x                         Page 8
 
 
    43 xyzxyzxyz XYZXYZXYZ abcabcab   :--:
    56 456789 123456789 abcdefghi ABCD
 \f   
 
--- Date/Time --                         x                         Page 9
+   -- Date/Time --                         x                         Page 9
 
 
                                     :--:
    70  456789 123456789 abcdefghi ABC:--:
 \f   
 
--- Date/Time --                        x                         Page 10
+   -- Date/Time --                        x                         Page 10
 
 
    1                                :--:
index 7243641..655b56f 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
        1.1   FF-Test: FF's in Text     :--:1   Test-INPUT: "Without FF set
@@ -19,7 +19,7 @@
       14.14 456789 123456789 123456789 1:--:14 456789 123456789 123456789 1
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
       15.                              :--:15 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -38,7 +38,7 @@
       28.                              :--:28 no trunc
 \f   
 
--- Date/Time --                         x                         Page 3
+   -- Date/Time --                         x                         Page 3
 
 
       29.15 xyzxyzxyz XYZXYZXYZ abcabcab:--:29 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -57,7 +57,7 @@
       42.                              :--:42 456789 123456789 abcdefghi A
 \f   
 
--- Date/Time --                         x                         Page 4
+   -- Date/Time --                         x                         Page 4
 
 
       43.                              :--:43 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -76,7 +76,7 @@
       56.                              :--:56 456789 123456789 abcdefghi A
 \f   
 
--- Date/Time --                         x                         Page 5
+   -- Date/Time --                         x                         Page 5
 
 
       57.29 xyzxyzxyz XYZXYZXYZ abcabcab:--:57 xyzxyzxyz XYZXYZXYZ abcabcab
       70.42 456789 123456789 abcdefghi A
 \f   
 
--- Date/Time --                         x                         Page 6
+   -- Date/Time --                         x                         Page 6
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 7
+   -- Date/Time --                         x                         Page 7
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 8
+   -- Date/Time --                         x                         Page 8
 
 
       71.43 xyzxyzxyz XYZXYZXYZ abcabcab:--:
       84.56 456789 123456789 abcdefghi A
 \f   
 
--- Date/Time --                         x                         Page 9
+   -- Date/Time --                         x                         Page 9
 
 
       85.                              :--:
       98.70  456789 123456789 abcdefghi :--:
 \f   
 
--- Date/Time --                        x                         Page 10
+   -- Date/Time --                        x                         Page 10
 
 
       99.1                             :--:
index a2b7be5..5109be0 100644 (file)
@@ -1,6 +1,6 @@
    
 
--- Date/Time --                         x                         Page 1
+   -- Date/Time --                         x                         Page 1
 
 
    1   FF-Test: FF's in Text          1   Test-INPUT: "Without FF set by
@@ -19,7 +19,7 @@
    14 456789 123456789 123456789 12345 14 456789 123456789 123456789 12345
 \f   
 
--- Date/Time --                         x                         Page 2
+   -- Date/Time --                         x                         Page 2
 
 
                                       15 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -38,7 +38,7 @@
                                       28 no trunc
 \f   
 
--- Date/Time --                         x                         Page 3
+   -- Date/Time --                         x                         Page 3
 
 
    15 xyzxyzxyz XYZXYZXYZ abcabcab     29 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -57,7 +57,7 @@
                                       42 456789 123456789 abcdefghi ABCDE
 \f   
 
--- Date/Time --                         x                         Page 4
+   -- Date/Time --                         x                         Page 4
 
 
                                       43 xyzxyzxyz XYZXYZXYZ abcabcab
@@ -76,7 +76,7 @@
                                       56 456789 123456789 abcdefghi ABCDE
 \f   
 
--- Date/Time --                         x                         Page 5
+   -- Date/Time --                         x                         Page 5
 
 
    29 xyzxyzxyz XYZXYZXYZ abcabcab     57 xyzxyzxyz XYZXYZXYZ abcabcab
    42 456789 123456789 abcdefghi ABCDE
 \f   
 
--- Date/Time --                         x                         Page 6
+   -- Date/Time --                         x                         Page 6
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 7
+   -- Date/Time --                         x                         Page 7
 
 
 
 \f   
 
--- Date/Time --                         x                         Page 8
+   -- Date/Time --                         x                         Page 8
 
 
    43 xyzxyzxyz XYZXYZXYZ abcabcab     
    56 456789 123456789 abcdefghi ABCDE
 \f   
 
--- Date/Time --                         x                         Page 9
+   -- Date/Time --                         x                         Page 9
 
 
                                       
    70  456789 123456789 abcdefghi ABCD 
 \f   
 
--- Date/Time --                        x                         Page 10
+   -- Date/Time --                        x                         Page 10
 
 
    1