From 270a33558b8c086b899f49f34e1d47da4792b7b8 Mon Sep 17 00:00:00 2001
From: Kristaps Dzonsons <kristaps@bsd.lv>
Date: Fri, 21 Aug 2009 12:32:38 +0000
Subject: Fixed `nf' behaviour (had broken with de-chunking). Added warnings if
 literal context already open/closed.

---
 libman.h     |  4 ++++
 man.c        | 12 +++++++++++-
 man_action.c | 52 +++++++++++++++++++++++++++++++++++++++-------------
 3 files changed, 54 insertions(+), 14 deletions(-)

diff --git a/libman.h b/libman.h
index c7f971b8..2b954dab 100644
--- a/libman.h
+++ b/libman.h
@@ -33,6 +33,7 @@ struct	man {
 #define	MAN_HALT	(1 << 0)
 #define	MAN_ELINE	(1 << 1) 	/* Next-line element scope. */
 #define	MAN_BLINE	(1 << 2) 	/* Next-line block scope. */
+#define	MAN_LITERAL	(1 << 3)	/* Literal input. */
 	enum man_next	 next;
 	struct man_node	*last;
 	struct man_node	*first;
@@ -58,6 +59,8 @@ enum	merr {
 	WMACROFORM,
 	WEXITSCOPE,
 	WNOSCOPE,
+	WOLITERAL,
+	WNLITERAL,
 	WERRMAX
 };
 
@@ -106,6 +109,7 @@ int		  man_verr(struct man *, int, int, const char *, ...);
 int		  man_valid_post(struct man *);
 int		  man_valid_pre(struct man *, const struct man_node *);
 int		  man_action_post(struct man *);
+int		  man_action_pre(struct man *, struct man_node *);
 int		  man_unscope(struct man *, const struct man_node *);
 
 __END_DECLS
diff --git a/man.c b/man.c
index c2f51ed5..5c152e1b 100644
--- a/man.c
+++ b/man.c
@@ -41,7 +41,9 @@ const	char *const __man_merrnames[WERRMAX] = {
 	"unknown macro", /* WMACRO */
 	"ill-formed macro", /* WMACROFORM */
 	"scope open on exit", /* WEXITSCOPE */
-	"no scope context" /* WNOSCOPE */
+	"no scope context", /* WNOSCOPE */
+	"literal context already open", /* WOLITERAL */
+	"no literal context open" /* WNLITERAL */
 };
 
 const	char *const __man_macronames[MAN_MAX] = {		 
@@ -392,6 +394,14 @@ man_ptext(struct man *m, int line, char *buf)
 {
 	int		 i, j;
 
+	/* Literal free-form text whitespace is preserved. */
+
+	if (MAN_LITERAL & m->flags) {
+		if ( ! man_word_alloc(m, line, 0, buf))
+			return(0);
+		goto descope;
+	}
+
 	/* First de-chunk and allocate words. */
 
 	for (i = 0; ' ' == buf[i]; i++)
diff --git a/man_action.c b/man_action.c
index 747708c6..a86e4bfa 100644
--- a/man_action.c
+++ b/man_action.c
@@ -23,17 +23,13 @@
 
 #include "libman.h"
 
-#ifdef __linux__
-extern	char		*strptime(const char *, const char *, struct tm *);
-#endif
-
 struct	actions {
 	int	(*post)(struct man *);
 };
 
-
 static	int	  post_TH(struct man *);
-static	time_t	  man_atotime(const char *);
+static	int	  post_fi(struct man *);
+static	int	  post_nf(struct man *);
 
 const	struct actions man_actions[MAN_MAX] = {
 	{ NULL }, /* br */
@@ -60,14 +56,19 @@ const	struct actions man_actions[MAN_MAX] = {
 	{ NULL }, /* na */
 	{ NULL }, /* i */
 	{ NULL }, /* sp */
-	{ NULL }, /* nf */
-	{ NULL }, /* fi */
+	{ post_nf }, /* nf */
+	{ post_fi }, /* fi */
 	{ NULL }, /* r */
 	{ NULL }, /* RE */
 	{ NULL }, /* RS */
 	{ NULL }, /* DT */
 };
 
+static	time_t	  man_atotime(const char *);
+#ifdef __linux__
+extern	char	 *strptime(const char *, const char *, struct tm *);
+#endif
+
 
 int
 man_action_post(struct man *m)
@@ -79,14 +80,39 @@ man_action_post(struct man *m)
 
 	switch (m->last->type) {
 	case (MAN_TEXT):
-		break;
+		/* FALLTHROUGH */
 	case (MAN_ROOT):
-		break;
+		return(1);
 	default:
-		if (NULL == man_actions[m->last->tok].post)
-			break;
-		return((*man_actions[m->last->tok].post)(m));
+		break;
 	}
+
+	if (NULL == man_actions[m->last->tok].post)
+		return(1);
+	return((*man_actions[m->last->tok].post)(m));
+}
+
+
+static int
+post_fi(struct man *m)
+{
+
+	if ( ! (MAN_LITERAL & m->flags))
+		if ( ! man_nwarn(m, m->last, WNLITERAL))
+			return(0);
+	m->flags &= ~MAN_LITERAL;
+	return(1);
+}
+
+
+static int
+post_nf(struct man *m)
+{
+
+	if (MAN_LITERAL & m->flags)
+		if ( ! man_nwarn(m, m->last, WOLITERAL))
+			return(0);
+	m->flags |= MAN_LITERAL;
 	return(1);
 }
 
-- 
cgit