From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-1.sys.kth.se (smtp-1.sys.kth.se [130.237.32.175]) by krisdoz.my.domain (8.14.3/8.14.3) with ESMTP id p65E6wbQ002074 for ; Tue, 5 Jul 2011 10:07:01 -0400 (EDT) Received: from mailscan-1.sys.kth.se (mailscan-1.sys.kth.se [130.237.32.91]) by smtp-1.sys.kth.se (Postfix) with ESMTP id 065FC156419; Tue, 5 Jul 2011 16:06:53 +0200 (CEST) X-Virus-Scanned: by amavisd-new at kth.se Received: from smtp-1.sys.kth.se ([130.237.32.175]) by mailscan-1.sys.kth.se (mailscan-1.sys.kth.se [130.237.32.91]) (amavisd-new, port 10024) with LMTP id O9ORpqQ2wuUa; Tue, 5 Jul 2011 16:06:48 +0200 (CEST) X-KTH-Auth: kristaps [89.158.117.88] X-KTH-mail-from: kristaps@bsd.lv Received: from macky.local (89-158-117-88.rev.dartybox.com [89.158.117.88]) by smtp-1.sys.kth.se (Postfix) with ESMTP id D5DAD156432; Tue, 5 Jul 2011 16:06:45 +0200 (CEST) Message-ID: <4E131A74.3050900@bsd.lv> Date: Tue, 05 Jul 2011 16:06:44 +0200 From: Kristaps Dzonsons User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11 X-Mailinglist: mdocml-tech Reply-To: tech@mdocml.bsd.lv MIME-Version: 1.0 To: tech@mdocml.bsd.lv CC: Ingo Schwarze Subject: Re: sync results References: <20110704224519.GC28684@iris.usta.de> <4E1306FA.3070801@bsd.lv> In-Reply-To: <4E1306FA.3070801@bsd.lv> Content-Type: multipart/mixed; boundary="------------070704080805030604050206" This is a multi-part message in MIME format. --------------070704080805030604050206 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit >> nsd-xfer(8) contains this code: >> >> .SH "OPTIONS" >> .LP >> .TP >> .B \-4 >> Only send to IPv4 addresses. >> >> In groff and old mandoc, there is no blank line after OPTIONS, >> now there is one. > > Ouch, this is indeed a bug: the syntax tree appears to be messed up by > the removal of the `LP'. I'll look into it... Ingo, Please check if the enclosed patch fixes this problem without fallout. Basically, the "next node append type" for -man was automatically being set to NEXT, clobbering the value set during node deletion in the post-validator. This was causing the `LP' to be removed and the next node, the `TP', being assigned under the block of `SH'. Ew. I eye-balled it over a selection of -man and ran it over my regular tests for failure. This is quite an old bug, it turns out. Thanks, Kristaps --------------070704080805030604050206 Content-Type: text/plain; name="patch.txt" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="patch.txt" Index: man_macro.c =================================================================== RCS file: /usr/vhosts/mdocml.bsd.lv/cvs/mdocml/man_macro.c,v retrieving revision 1.63 diff -u -r1.63 man_macro.c --- man_macro.c 18 Jun 2011 17:36:52 -0000 1.63 +++ man_macro.c 5 Jul 2011 14:01:28 -0000 @@ -120,6 +120,9 @@ assert(to); + assert(MAN_ROOT != m->last->type); + m->next = MAN_NEXT_SIBLING; + /* LINTED */ while (m->last != to) { /* @@ -140,9 +143,6 @@ if ( ! man_valid_post(m)) return(0); - m->next = MAN_ROOT == m->last->type ? - MAN_NEXT_CHILD : MAN_NEXT_SIBLING; - return(1); } @@ -428,6 +428,9 @@ assert( ! (MAN_SCOPED & man_macros[tok].flags)); m->flags |= MAN_ILINE; } + + assert(MAN_ROOT != m->last->type); + m->next = MAN_NEXT_SIBLING; /* * Rewind our element scope. Note that when TH is pruned, we'll @@ -452,9 +455,6 @@ if (m->last->type != MAN_ROOT && ! man_valid_post(m)) return(0); - - m->next = MAN_ROOT == m->last->type ? - MAN_NEXT_CHILD : MAN_NEXT_SIBLING; return(1); } --------------070704080805030604050206-- -- To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv