[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Groff] Suggested patch to post-html.cpp
From: |
Bertrand Garrigues |
Subject: |
Re: [Groff] Suggested patch to post-html.cpp |
Date: |
Sun, 21 Sep 2014 23:25:07 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Hi Robert,
On Thu, Sep 18 2014 at 05:25:40 AM, Robert Bocchino <address@hidden> wrote:
> Attached for your review is a suggested patch to post-html.cpp. The
> motivation is that grohtml occasionally bungles the formatting of
> indented lists. Bug 30043 seems related, although that one is focused
> mainly on interactions with -mdoc. -ms is what I use for my
> day-to-day formatting.
[...]
> Please note that I have in no way adequately tested this. I've just
> tried out a few examples, and they seem to work. I realize that
> grohtml has to work with lots of macro packages, is doing a lot of
> complicated stuff only some of which I've been able to figure out so
> far, etc. Any suggestions about how to do proper regression testing
> on this or other patches would be most welcome! Also suggestions on
> the patch itself are welcome. As I mentioned before, my goal is to
> eventually learn enough about how grohtml works that I can do active
> development on it.
>
> Anyway, let me know what you think.
I am not qualified to review your patch because I haven't studied
grohtml's code, but I've tested it on the few .html files that are built
by default in the 'doc' directory from pic.ms and webpage.ms, there are
a few .html pages impacted but the result is unchanged, I haven't seen
any regression.
Regards,
--
Bertrand Garrigues