From: Kristaps Dzonsons <kristaps@bsd.lv>
To: tech@mdocml.bsd.lv
Cc: Ingo Schwarze <schwarze@usta.de>
Subject: Re: [PATCH] multiple tbl improvements
Date: Tue, 25 Jan 2011 00:08:26 +0100 [thread overview]
Message-ID: <4D3E066A.1060000@bsd.lv> (raw)
In-Reply-To: <20110124224333.GE16282@iris.usta.de>
> here is a cumulative patch to fix various issues in tbl.
> I'm planning to commit that in pieces, but i hate it when
> people send patchsets and find one large patch much easier
> to read - or do you feel the other way round?
Ingo,
This is all pretty easy to understand, so I've no problem with it.
> * out.c, tbl_term.c:
> Correct horizontal data spacing.
> Correct centered data alignment.
> Adjust the hrule width to the corrected spacing.
>
> * tbl_data.c:
> Do not skip data after horizontal lines in the layout.
> This causes one input line to produce two spans.
>
> * libroff.h, tbl.c, main.c:
> Actually use the additional span.
Out of curiosity---where do you see this last one being an issue? It
makes perfect sense.
>
> * man_term.c:
> avoid double blank line before a table
(man_html.c...?)
>
> OK?
Ok kristaps.
> I'm now going to start committing patches to OpenBSD even without OKs
> because my tree is getting too cluttered. We can fix the stuff in tree
> in case problems show up. Of course, i'm not committing those patches
> that faced opposition.
>
> Expect some activity during this week, i'm off my day job for a week,
> and even though a few other things need to be done (among others,
> i will once again be helping at http://www.usta.de/unifest ,
> this time running the box office), there will also be some time for
> mandoc.
I've a bit of time too, so feel free to bounce patches off of me. I'm
only aware of the bullet patch causing problems, and those are "hard"
problems in terms of -Tps segfaulting.
The festival looks like fun, though...
Thanks!
Kristaps
--
To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv
prev parent reply other threads:[~2011-01-24 23:08 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-24 22:43 Ingo Schwarze
2011-01-24 23:08 ` Kristaps Dzonsons [this message]
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=4D3E066A.1060000@bsd.lv \
--to=kristaps@bsd.lv \
--cc=schwarze@usta.de \
--cc=tech@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).