From: Yuri Pankov <yuri.pankov@gmail.com>
To: discuss@mdocml.bsd.lv
Subject: Re: tbl issues with 1.11.1
Date: Sun, 18 Sep 2011 21:25:05 +0400 [thread overview]
Message-ID: <20110918172505.GA1275@procyon.xvoid.org> (raw)
In-Reply-To: <20110912201130.GB1331@procyon.xvoid.org>
On Tue, Sep 13, 2011 at 12:11:30AM +0400, Yuri Pankov wrote:
> On Sat, May 14, 2011 at 06:12:28AM +0200, Ingo Schwarze wrote:
> > Hi Kristaps and Joerg,
> >
> > >>> Ingo, that's your change from January. Care to check the attached patch?
> > >> Anyone?
> > > Ingo? I remember you checking these in... care to take another look?
> >
> > Er, jaja, i was a bit swamped...
> >
> > Unfortunately, Joergs patch is not right, i breaks the horizontal
> > spacing in tables without vertical rulers almost completely, see
> > for example captoinfo(1) and compare the groff and mandoc output.
> >
> > Getting this right in both cases - with and without vertical rulers -
> > requires rather large changes. The logic, as far as i understand it
> > so far, goes like this:
> >
> > - box produces "|"
> > - no spacing at this point
> > - first cell, width is max(lengths)
> > - one space
> > - vertical ruler, if none, another space
> > - another space, unless there was a double ruler (!!)
> > - second cell
> > - and so on.
> >
> > That is vastly different from what we have now, both regarding the
> > width calculations - which apparently must not include padding - and
> > regarding the output itself.
> >
> > The work-in-progress patch below comes closer. However, it still
> > has at least two deficiencies, so it is not ready for commit:
> > - There still is a bogus space after double rulers, see (!!).
> > - The handling of explicit spacing options got sweeped away
> > by the reorg i was forced to do. Probably, i need to put
> > it back in at the right places. It was in the calc routine
> > but probably belongs in the output routines themselves.
> >
> > Just sending this such that you finally get some feedback.
> > In case we need more discussion, we should probably move to tech@.
>
> Sorry for bringing this up again, but I'm still seeing the same
> behaviour in 1.11.6. Is there anything wrong with proposed patch?
And sorry for sending this again, but I'm not sure it got through due do
the site downtime...
Yuri
--
To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv
next prev parent reply other threads:[~2011-09-18 17:25 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-14 1:45 Yuri Pankov
2011-04-16 21:34 ` Ingo Schwarze
2011-04-26 20:10 ` Joerg Sonnenberger
2011-05-12 22:02 ` Joerg Sonnenberger
2011-05-13 21:03 ` Kristaps Dzonsons
2011-05-14 4:12 ` Ingo Schwarze
2011-09-12 20:11 ` Yuri Pankov
2011-09-18 17:25 ` Yuri Pankov [this message]
2011-04-17 0:12 ` Joerg Sonnenberger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110918172505.GA1275@procyon.xvoid.org \
--to=yuri.pankov@gmail.com \
--cc=discuss@mdocml.bsd.lv \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).