Bug#871310: gcc.1: Some point size changes are wrong in the manual

2017-08-22 Thread Guo Yixuan 
Hello,

On Mon, Aug 7, 2017 at 11:20 AM, Bjarni Ingi Gislason 
wrote:
>
> Package: gcc-6-doc
> Version: 6.3.0-1
> Severity: normal
> Tags: patch
>
> Dear Maintainer,
>
>* What led up to the situation?
>
> "man gcc" with
>
> MAN_KEEP_STDERR=yes
> MANOPT=--encoding=latin1 --no-hyphenation --warnings=w --no-justification
> MANWIDTH=80
>
>* What was the outcome of this action?
>
> :3310: warning: \s escape results in non-positive point
size; set to 1
> :3311: warning: \s escape results in non-positive point
size; set to 1
> :25: backtrace: string `C+'
> :3311: warning: \s escape results in non-positive point
size; set to 1
> :3320: warning: \s escape results in non-positive point
size; set to 1
> :3323: warning: \s escape results in non-positive point
size; set to 1
> :3326: warning: \s escape results in non-positive point
size; set to 1
> :3331: warning: \s escape results in non-positive point
size; set to 1
> :: warning: \s escape results in non-positive point
size; set to 1
> :: warning: \s escape results in non-positive point
size; set to 1
> :3336: warning: \s escape results in non-positive point
size; set to 1
> :3337: warning: \s escape results in non-positive point
size; set to 1
> :4668: warning: \s escape results in non-positive point
size; set to 1
> :25: backtrace: string `C+'
> :4668: warning: \s escape results in non-positive point
size; set to 1
> :4670: backtrace
> /home/bg/git/groff/tmac/an-old.tmac:493: backtrace: macro `IP'
> :4670: warning: \s escape results in non-positive point
size; set to 1
> :4670: backtrace
> /home/bg/git/groff/tmac/an-old.tmac:493: backtrace: macro `IP'
> :4670: warning: \s escape results in non-positive point
size; set to 1
> :9001: warning [p 93, 4.5i]: can't break line
> :9023: warning [p 93, 8.2i]: can't break line
> :9234: warning [p 95, 7.3i]: can't break line
>
>* What outcome did you expect instead?
>
>   No warnings
>
>   The escape '\s0' does not get the right former size if another '\s0'
comes
> after it (like '\s-1...\s-2...\s0...\s0').
>
>   The string '\*(C+' gets resized in the manual for "gcc" in some lines.
>   Each use of this string gets the previous size smaller, the warning is
not
> issued until the size is down to less than 1 point.
>
> Patch:
>
> --- gcc.1   2017-08-07 14:54:44.0 +
> +++ gcc.1.new   2017-08-07 14:55:41.0 +
> @@ -22,7 +22,7 @@
>  .\" therefore won't be available.  \*(C` and \*(C' expand to `' in nroff,
>  .\" nothing in troff, for use with C<>.
>  .tr \(*W-
> -.ds C+ C\v'-.1v'\h'-1p'\s-2+\h'-1p'+\s0\v'.1v'\h'-1p'
> +.ds C+ C\v'-.1v'\h'-1p'\s-2+\h'-1p'+\s+2\v'.1v'\h'-1p'
>  .ie n \{\
>  .ds -- \(*W-
>  .ds PI pi
>
>   And then change all the '\s0' into the correct escape (\s-1 ... \s0)
>
> sed -e 's/\\s0/\\s+1/g'  > 

Thank you for the report. The gcc-6.1.gz manpage is generated automatically
from its texinfo source, by texi2pod.pl and then pod2man. If there's any
bug in
the manpage, the problem is more likely to be in pod2man, so it's better to
try to report to the perl package (which contains /usr/bin/pod2man).

Thanks,
Yixuan


>
>
> -- System Information:
> Debian Release: buster/sid
>   APT prefers proposed-updates
>   APT policy: (500, 'proposed-updates'), (500, 'testing'), (500, 'stable')
> Architecture: amd64 (x86_64)
>
> Kernel: Linux 4.9.30-2u2 (SMP w/2 CPU cores)
> Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591
(charmap=ISO-8859-1), LANGUAGE=is_IS.iso88591 (charmap=ISO-8859-1)
> Shell: /bin/sh linked to /bin/dash
> Init: sysvinit (via /sbin/init)
>
> Versions of packages gcc-6-doc depends on:
> ii  gcc-doc-base  6.3.0-1
>
> gcc-6-doc recommends no packages.
>
> Versions of packages gcc-6-doc suggests:
> pn  doc-base  
>
> -- no debconf information
>
> --
> Bjarni I. Gislason




--
GUO Yixuan

Bug#871310: gcc.1: Some point size changes are wrong in the manual

2017-08-07 Thread Bjarni Ingi Gislason 
Package: gcc-6-doc
Version: 6.3.0-1
Severity: normal
Tags: patch

Dear Maintainer,

   * What led up to the situation?

"man gcc" with

MAN_KEEP_STDERR=yes
MANOPT=--encoding=latin1 --no-hyphenation --warnings=w --no-justification
MANWIDTH=80

   * What was the outcome of this action?

:3310: warning: \s escape results in non-positive point size; 
set to 1
:3311: warning: \s escape results in non-positive point size; 
set to 1
:25: backtrace: string `C+'
:3311: warning: \s escape results in non-positive point size; 
set to 1
:3320: warning: \s escape results in non-positive point size; 
set to 1
:3323: warning: \s escape results in non-positive point size; 
set to 1
:3326: warning: \s escape results in non-positive point size; 
set to 1
:3331: warning: \s escape results in non-positive point size; 
set to 1
:: warning: \s escape results in non-positive point size; 
set to 1
:: warning: \s escape results in non-positive point size; 
set to 1
:3336: warning: \s escape results in non-positive point size; 
set to 1
:3337: warning: \s escape results in non-positive point size; 
set to 1
:4668: warning: \s escape results in non-positive point size; 
set to 1
:25: backtrace: string `C+'
:4668: warning: \s escape results in non-positive point size; 
set to 1
:4670: backtrace
/home/bg/git/groff/tmac/an-old.tmac:493: backtrace: macro `IP'
:4670: warning: \s escape results in non-positive point size; 
set to 1
:4670: backtrace
/home/bg/git/groff/tmac/an-old.tmac:493: backtrace: macro `IP'
:4670: warning: \s escape results in non-positive point size; 
set to 1
:9001: warning [p 93, 4.5i]: can't break line
:9023: warning [p 93, 8.2i]: can't break line
:9234: warning [p 95, 7.3i]: can't break line

   * What outcome did you expect instead?

  No warnings

  The escape '\s0' does not get the right former size if another '\s0' comes
after it (like '\s-1...\s-2...\s0...\s0').

  The string '\*(C+' gets resized in the manual for "gcc" in some lines.
  Each use of this string gets the previous size smaller, the warning is not
issued until the size is down to less than 1 point.

Patch:

--- gcc.1   2017-08-07 14:54:44.0 +
+++ gcc.1.new   2017-08-07 14:55:41.0 +
@@ -22,7 +22,7 @@
 .\" therefore won't be available.  \*(C` and \*(C' expand to `' in nroff,
 .\" nothing in troff, for use with C<>.
 .tr \(*W-
-.ds C+ C\v'-.1v'\h'-1p'\s-2+\h'-1p'+\s0\v'.1v'\h'-1p'
+.ds C+ C\v'-.1v'\h'-1p'\s-2+\h'-1p'+\s+2\v'.1v'\h'-1p'
 .ie n \{\
 .ds -- \(*W-
 .ds PI pi

  And then change all the '\s0' into the correct escape (\s-1 ... \s0)

sed -e 's/\\s0/\\s+1/g'  > 


-- System Information:
Debian Release: buster/sid
  APT prefers proposed-updates
  APT policy: (500, 'proposed-updates'), (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.30-2u2 (SMP w/2 CPU cores)
Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), 
LANGUAGE=is_IS.iso88591 (charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages gcc-6-doc depends on:
ii  gcc-doc-base  6.3.0-1

gcc-6-doc recommends no packages.

Versions of packages gcc-6-doc suggests:
pn  doc-base  

-- no debconf information

-- 
Bjarni I. Gislason