grotty: One vertical line too mutch in a table

[Bjarni Ingi Gislason]

GROFF VERSION:
groff-current 1.21 (from 10th March 2012)

MACHINE:
AMD-K6

OS:
Debian GNU/Linux 6.0.5

COMPILER:
g++ (Debian 4.4.5-8) 4.4.5

INPUT FILES:
'\" t
.\"
.\" Make page length short to reduce size of input file.
.pl 10v
.TH SUFFIXES 7  2000-11-16 "Linux" "Linux Programmer's Manual"
.SH NAME
suffixes \- list of file suffixes
.SH DESCRIPTION
The
.BR make (1)
utility is driven by rules based on file suffix.
.PP
Following is a list of suffixes which are likely to be found on a
Linux system.
.PP
.TS
l | l
_ | _
lI |  l .
Suffix  File type
 ,v     files for RCS (Revision Control System)
 -      backup file
 .C     C++ source code, equivalent to \fI.cc\fP
 .F     Fortran source with \fBcpp\fP(1) directives
        or file compressed using freeze
 .S     assembler source with \fBcpp\fP(1) directives
 .Y     file compressed using yabba
 .Z     file compressed using \fBcompress\fP(1)
 .[0-9]+gf      TeX generic font files
 .[0-9]+pk      TeX packed font files
 .[1-9] manual page for the corresponding section
 .[1-9][a-z]    manual page for section plus subsection
 .a     static object code library
 .ad    X application default resource file
 .ada   Ada source (may be body, spec, or combination)
 .adb   Ada body source
 .ads   Ada spec source
 .afm   PostScript font metrics
.TE

COMMAND LINE:
./test-groff -b -ww -t -Tascii -mandoc <input file>

DESCRIPTION OF INCORRECT BEHAVIOUR:
  The first vertical character starts _on_ the baseline (start
of vertical line down the page) instead of on the next line
under it.  Thus there is one too many of them.

  The output shows the first vertical line one line higher than
expected.

  Warnings from "grotty":

grotty:<standard input> (prof.gro):387: character above first line discarded
grotty:<standard input> (prof.gro):470: character above first line discarded


SUGGESTED FIX [optional]:
  The line lies between the marked lines (marked vertical
postition).  The graphic characters must thus start under the
higer mark and end on the lower mark.

  The (minor) drawback I have seen with this, is in a "box"-table
mode.  Then there is no "+" in the upper corners.

  Patch that works for the input file (part of manual
"suffix(7)").

--- tty.cpp     2011-01-20 07:13:57.000000000 +0000
+++ tty.cpp.new 2012-05-18 20:18:24.913374942 +0000
@@ -553,6 +553,8 @@
       add_char(vline_char, font::hor, hpos, v, col, fill,
               VDRAW_MODE|START_LINE|END_LINE);
     else {
+      len -= font::vert;
+      v += font::vert;
       add_char(vline_char, font::hor, hpos, v, col, fill,
               VDRAW_MODE|START_LINE);
       len -= font::vert;


  There is still the possibility that the case "len=0" is wrong.
Should the character be placed on or under the vertical
postition?  Dependent on where it is on the page?