===================================================================
RCS file: /cvs/mandoc/term.c,v
retrieving revision 1.18
retrieving revision 1.21
diff -u -p -r1.18 -r1.21
--- mandoc/term.c	2009/02/25 12:09:20	1.18
+++ mandoc/term.c	2009/02/25 15:12:26	1.21
@@ -1,4 +1,4 @@
-/* $Id: term.c,v 1.18 2009/02/25 12:09:20 kristaps Exp $ */
+/* $Id: term.c,v 1.21 2009/02/25 15:12:26 kristaps Exp $ */
 /*
  * Copyright (c) 2009 Kristaps Dzonsons <kristaps@kth.se>
  *
@@ -30,7 +30,7 @@
  */
 
 /* FIXME: indent/tab. */
-/* FIXME: handle nested lists. */
+/* FIXME: macro arguments can be escaped. */
 
 #define	TTYPE_PROG	  0
 #define	TTYPE_CMD_FLAG	  1
@@ -343,87 +343,15 @@ termp_dq_post(DECL_ARGS)
 
 
 /* ARGSUSED */
-static void
-termp_it_post(DECL_ARGS)
-{
-	const struct mdoc_node *n, *it;
-	const struct mdoc_block *bl;
-	int		 i;
-	size_t		 width, offset;
-
-	/*
-	 * This (and termp_it_pre()) are the most complicated functions
-	 * here.  They must account for a considerable number of
-	 * switches that completely change the output behaviour, like
-	 * -tag versus -column.  Yech.
-	 */
-
-	switch (node->type) {
-	case (MDOC_BODY):
-		/* FALLTHROUGH */
-	case (MDOC_HEAD):
-		break;
-	default:
-		return;
-	}
-
-	it = node->parent;
-	assert(MDOC_BLOCK == it->type);
-	assert(MDOC_It == it->tok);
-
-	n = it->parent;
-	assert(MDOC_BODY == n->type);
-	assert(MDOC_Bl == n->tok);
-	n = n->parent;
-	bl = &n->data.block;
-
-	/* If `-tag', adjust our margins accordingly. */
-
-	if (arg_hasattr(MDOC_Tag, bl->argc, bl->argv)) {
-		flushln(p);
-
-		/* FIXME: this should auto-size. */
-		i = arg_getattr(MDOC_Width, bl->argc, bl->argv);
-		width = i >= 0 ? arg_width(&bl->argv[i]) : 10;
-
-		/* FIXME: nesting!  Should happen at block. */
-		i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
-		offset = i >= 0 ? arg_width(&bl->argv[i]) : 0;
-
-		if (MDOC_HEAD == node->type) {
-			p->rmargin = p->maxrmargin;
-			p->offset -= offset;
-			p->flags &= ~TERMP_NOBREAK;
-		} else {
-			p->offset -= width;
-			p->flags &= ~TERMP_NOLPAD;
-		}
-	}
-
-	if (arg_hasattr(MDOC_Ohang, bl->argc, bl->argv)) {
-		i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
-		offset = i >= 0 ? arg_offset(&bl->argv[i]) : 0;
-
-		flushln(p);
-		p->offset -= offset;
-		return;
-	}
-}
-
-
-/* ARGSUSED */
 static int
 termp_it_pre(DECL_ARGS)
 {
 	const struct mdoc_node *n, *it;
 	const struct mdoc_block *bl;
+	char		 buf[7];
 	int		 i;
 	size_t		 width, offset;
 
-	/*
-	 * Also see termp_it_post() for general comments.
-	 */
-
 	switch (node->type) {
 	case (MDOC_BODY):
 		/* FALLTHROUGH */
@@ -437,17 +365,9 @@ termp_it_pre(DECL_ARGS)
 		return(1);
 	}
 
-	assert(MDOC_BLOCK == it->type);
-	assert(MDOC_It == it->tok);
-
-	n = it->parent;
-	assert(MDOC_BODY == n->type);
-	assert(MDOC_Bl == n->tok);
-	n = n->parent;
+	n = it->parent->parent;
 	bl = &n->data.block;
 
-	/* If `-compact', don't assert vertical space. */
-
 	if (MDOC_BLOCK == node->type) {
 		if (arg_hasattr(MDOC_Compact, bl->argc, bl->argv))
 			newln(p);
@@ -456,46 +376,121 @@ termp_it_pre(DECL_ARGS)
 		return(1);
 	}
 
-	assert(MDOC_HEAD == node->type 
-			|| MDOC_BODY == node->type);
+	pair->offset = p->offset;
+	pair->rmargin = p->rmargin;
 
-	/* FIXME: see termp_it_post(). */
+	/* FIXME: auto-size. */
+	i = arg_getattr(MDOC_Width, bl->argc, bl->argv);
+	width = i >= 0 ? arg_width(&bl->argv[i]) : 10;
 
-	/* If `-tag', adjust our margins accordingly. */
+	i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
+	offset = i >= 0 ? arg_offset(&bl->argv[i]) : 0;
 
+	assert(MDOC_HEAD == node->type || 
+			MDOC_BODY == node->type);
+
 	if (arg_hasattr(MDOC_Tag, bl->argc, bl->argv)) {
 		p->flags |= TERMP_NOSPACE;
+		if (MDOC_BODY == node->type) {
+			p->flags |= TERMP_NOLPAD;
+			p->offset += width;
+		} else  {
+			p->flags |= TERMP_NOBREAK;
+			p->rmargin = p->offset + offset + width;
+		}
 
-		i = arg_getattr(MDOC_Width, bl->argc, bl->argv);
-		width = i >= 0 ? arg_width(&bl->argv[i]) : 10;
+	} else if (arg_hasattr(MDOC_Ohang, bl->argc, bl->argv)) {
+		p->flags |= TERMP_NOSPACE;
 
-		i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
-		offset = i >= 0 ? arg_offset(&bl->argv[i]) : 0;
+	} else if (arg_hasattr(MDOC_Diag, bl->argc, bl->argv)) {
+		/* TODO. */
 
-		if (MDOC_HEAD == node->type) {
+	} else if (arg_hasattr(MDOC_Hang, bl->argc, bl->argv)) {
+		/* TODO. */
+
+	} else if (arg_hasattr(MDOC_Bullet, bl->argc, bl->argv)) {
+		p->flags |= TERMP_NOSPACE;
+		if (MDOC_BODY == node->type) {
+			p->flags |= TERMP_NOLPAD;
+			p->offset += 6;
+		} else {
+			word(p, "\\[bu]");
 			p->flags |= TERMP_NOBREAK;
-			p->offset += offset;
-			p->rmargin = p->offset + width;
+			p->rmargin = p->offset + offset + 6;
+		}
+
+	} else if (arg_hasattr(MDOC_Enum, bl->argc, bl->argv)) {
+		p->flags |= TERMP_NOSPACE;
+		if (MDOC_BODY == node->type) {
+			p->flags |= TERMP_NOLPAD;
+			p->offset += 6;
 		} else {
-			p->flags |= TERMP_NOSPACE;
+			(pair->ppair->ppair->count)++;
+			(void)snprintf(buf, sizeof(buf), "%d.", 
+					pair->ppair->ppair->count);
+			word(p, buf);
+			p->flags |= TERMP_NOBREAK;
+			p->rmargin = p->offset + offset + 6;
+		}
+
+	} else if (arg_hasattr(MDOC_Dash, bl->argc, bl->argv) ||
+			arg_hasattr(MDOC_Hyphen, bl->argc, bl->argv)) {
+		p->flags |= TERMP_NOSPACE;
+		if (MDOC_BODY == node->type) {
 			p->flags |= TERMP_NOLPAD;
-			p->offset += width;
+			p->offset += 6;
+			return(1);
+		} else {
+			word(p, "\\-");
+			p->flags |= TERMP_NOBREAK;
+			p->rmargin = p->offset + offset + 6;
 		}
-		return(1);
-	}
+	} 
 
-	/* If `-ohang', adjust left-margin. */
+	p->offset += offset;
+	return(1);
+}
 
-	if (arg_hasattr(MDOC_Ohang, bl->argc, bl->argv)) {
-		i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
-		offset = i >= 0 ? arg_offset(&bl->argv[i]) : 0;
 
-		p->flags |= TERMP_NOSPACE;
-		p->offset += offset;
-		return(1);
+/* ARGSUSED */
+static void
+termp_it_post(DECL_ARGS)
+{
+	const struct mdoc_node *n, *it;
+	const struct mdoc_block *bl;
+
+	if (MDOC_BODY != node->type && MDOC_HEAD != node->type)
+		return;
+
+	it = node->parent;
+	n = it->parent->parent;
+	bl = &n->data.block;
+
+	if (arg_hasattr(MDOC_Tag, bl->argc, bl->argv) ||
+			arg_hasattr(MDOC_Bullet, bl->argc, bl->argv) ||
+			arg_hasattr(MDOC_Dash, bl->argc, bl->argv) ||
+			arg_hasattr(MDOC_Enum, bl->argc, bl->argv) ||
+			arg_hasattr(MDOC_Hyphen, bl->argc, bl->argv)) {
+		flushln(p);
+		if (MDOC_HEAD == node->type) {
+			p->rmargin = pair->rmargin;
+			p->flags &= ~TERMP_NOBREAK;
+		} else
+			p->flags &= ~TERMP_NOLPAD;
+
+	} else if (arg_hasattr(MDOC_Ohang, bl->argc, bl->argv)) {
+		flushln(p);
+
+	} else if (arg_hasattr(MDOC_Inset, bl->argc, bl->argv)) {
+		if (MDOC_BODY == node->type)
+			flushln(p);
+
+	} else if (arg_hasattr(MDOC_Item, bl->argc, bl->argv)) {
+		if (MDOC_BODY == node->type)
+			flushln(p);
 	}
 
-	return(1);
+	p->offset = pair->offset;
 }
 
 
@@ -654,25 +649,6 @@ termp_op_post(DECL_ARGS)
 
 
 /* ARGSUSED */
-static void
-termp_sh_post(DECL_ARGS)
-{
-
-	switch (node->type) {
-	case (MDOC_HEAD):
-		newln(p);
-		break;
-	case (MDOC_BODY):
-		newln(p);
-		p->offset = 0;
-		break;
-	default:
-		break;
-	}
-}
-
-
-/* ARGSUSED */
 static int
 termp_xr_pre(DECL_ARGS)
 {
@@ -765,6 +741,25 @@ termp_sh_pre(DECL_ARGS)
 
 
 /* ARGSUSED */
+static void
+termp_sh_post(DECL_ARGS)
+{
+
+	switch (node->type) {
+	case (MDOC_HEAD):
+		newln(p);
+		break;
+	case (MDOC_BODY):
+		newln(p);
+		p->offset = 0;
+		break;
+	default:
+		break;
+	}
+}
+
+
+/* ARGSUSED */
 static int
 termp_op_pre(DECL_ARGS)
 {
@@ -809,7 +804,7 @@ termp_d1_pre(DECL_ARGS)
 	if (MDOC_BODY != node->type)
 		return(1);
 	newln(p);
-	p->offset += INDENT;
+	p->offset += (pair->offset = INDENT);
 	return(1);
 }
 
@@ -822,7 +817,7 @@ termp_d1_post(DECL_ARGS)
 	if (MDOC_BODY != node->type) 
 		return;
 	newln(p);
-	p->offset -= INDENT;
+	p->offset -= pair->offset;
 }
 
 
@@ -982,9 +977,11 @@ termp_bd_pre(DECL_ARGS)
 		return(1);
 
 	assert(MDOC_BLOCK == node->parent->type);
+	pair->offset = p->offset;
 
 	bl = &node->parent->data.block;
 
+
 	i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
 	if (-1 != i) {
 		assert(1 == bl->argv[i].sz);
@@ -1015,20 +1012,11 @@ termp_bd_pre(DECL_ARGS)
 static void
 termp_bd_post(DECL_ARGS)
 {
-	int		 i;
-	const struct mdoc_block *bl;
 
-	if (MDOC_BODY != node->type)
+	if (MDOC_BODY != node->type) 
 		return;
-
-	assert(MDOC_BLOCK == node->parent->type);
-	bl = &node->parent->data.block;
-
-	i = arg_getattr(MDOC_Offset, bl->argc, bl->argv);
-	if (-1 != i) {
-		assert(1 == bl->argv[i].sz);
-		p->offset -= arg_offset(&bl->argv[i]);
-	}
+	newln(p);
+	p->offset = pair->offset;
 }