no current date in man(7) footers

no current date in man(7) footers

[Ingo Schwarze]

Hi,

i'm currently leaving out the very last (footer) lines when doing
systematic output comparisons of groff and mandoc because they
differ gratuitiously in a number of ways.

Arguably, none of the ways to format the footer are really better
or worse than others, but i'd like to get rid of gratuitious
differences and simplify comparison scripts.

So, expect a series of small diffs to get the mandoc footer lines
in line with groff.  Here is the first one.

In man(7), mandoc uses the current date when .TH sets no date,
but groff uses the empty string instead.

OK?
  Ingo


Index: man_validate.c
===================================================================
RCS file: /cvs/src/usr.bin/mandoc/man_validate.c,v
retrieving revision 1.48
diff -u -p -r1.48 man_validate.c
--- man_validate.c	16 Oct 2011 12:18:32 -0000	1.48
+++ man_validate.c	1 Nov 2011 16:15:16 -0000
@@ -411,10 +411,12 @@ post_TH(CHKARGS)
 
 	if (n)
 		n = n->next;
-	if (n)
+	if (n && n->string && '\0' != n->string[0]) {
 		pos = n->pos;
-	m->meta.date = mandoc_normdate
-		(m->parse, n ? n->string : NULL, line, pos);
+		m->meta.date = mandoc_normdate
+		    (m->parse, n->string, line, pos);
+	} else
+		m->meta.date = mandoc_strdup("");
 
 	/* TITLE MSEC DATE ->SOURCE<- VOL */
 
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv

Re: no current date in man(7) footers

[Kristaps Dzonsons]

> i'm currently leaving out the very last (footer) lines when doing
> systematic output comparisons of groff and mandoc because they
> differ gratuitiously in a number of ways.
>
> Arguably, none of the ways to format the footer are really better
> or worse than others, but i'd like to get rid of gratuitious
> differences and simplify comparison scripts.
>
> So, expect a series of small diffs to get the mandoc footer lines
> in line with groff.  Here is the first one.
>
> In man(7), mandoc uses the current date when .TH sets no date,
> but groff uses the empty string instead.

Ingo,

Yes, please check this in.

Thanks,

Kristaps
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv