source@mandoc.bsd.lv
 help / color / mirror / Atom feed
* mdocml: Do not mistreat empty arguments to font alternating macros as
@ 2015-04-06 22:06 schwarze
  0 siblings, 0 replies; only message in thread
From: schwarze @ 2015-04-06 22:06 UTC (permalink / raw)
  To: source

Log Message:
-----------
Do not mistreat empty arguments to font alternating macros
as vertical spacing requests.  Bug found with xmahjongg(6).

Modified Files:
--------------
    mdocml:
        TODO
        man_term.c

Revision Data
-------------
Index: man_term.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/man_term.c,v
retrieving revision 1.178
retrieving revision 1.179
diff -Lman_term.c -Lman_term.c -u -p -r1.178 -r1.179
--- man_term.c
+++ man_term.c
@@ -321,7 +321,10 @@ pre_alternate(DECL_ARGS)
 		term_fontrepl(p, font[i]);
 		if (savelit && NULL == nn->next)
 			mt->fl |= MANT_LITERAL;
-		print_man_node(p, mt, nn, meta);
+		assert(nn->type == ROFFT_TEXT);
+		term_word(p, nn->string);
+		if (nn->flags & MAN_EOS)
+                	p->flags |= TERMP_SENTENCE;
 		if (nn->next)
 			p->flags |= TERMP_NOSPACE;
 	}
Index: TODO
===================================================================
RCS file: /home/cvs/mdocml/mdocml/TODO,v
retrieving revision 1.204
retrieving revision 1.205
diff -LTODO -LTODO -u -p -r1.204 -r1.205
--- TODO
+++ TODO
@@ -66,6 +66,7 @@ are mere guesses, and some may be wrong.
   loc *  exist *  algo *  size *  imp **
 
 - .ns (no-space mode) occurs in xine-config(1)
+  when implementing this, also let .TH set it
   reported by brad@  Sat, 15 Jan 2011 15:45:23 -0500
   loc ***  exist ***  algo ***  size **  imp *
 
@@ -478,6 +479,10 @@ are mere guesses, and some may be wrong.
   and document it in mdoc(7) and man(7) COMPATIBILITY
   found while talking to Chris Bennett
   loc *  exist *  algo *  size *  imp *
+
+- Sequences of multiple man(7) paragraphs (.PP, .IP) interspersed
+  with .ps and .nf/.fi produce execessive blank lines, see libJudy
+  The parser reorg may help with this.
 
 - trailing whitespace must be ignored even when followed by a font escape,
   see for example
--
 To unsubscribe send an email to source+unsubscribe@mdocml.bsd.lv

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2015-04-06 22:06 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-06 22:06 mdocml: Do not mistreat empty arguments to font alternating macros as schwarze

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).