# HG changeset patch
# User Ben Schmidt
# Date 1330435595 -39600
# Node ID 31ac95b2d625a3a59e72933012fc60fcdc62c387
# Parent  efd01230d20a8727e6a76374b473bf822e3db718
Simplify how line breaking is done internally.

diff -r efd01230d20a -r 31ac95b2d625 src/prepstdreply.c
--- a/src/prepstdreply.c	Wed Feb 29 00:03:46 2012 +1100
+++ b/src/prepstdreply.c	Wed Feb 29 00:26:35 2012 +1100
@@ -307,6 +307,9 @@
 static void substitute_one(char **line_p, char **pos_p, const char *listaddr,
 			const char *listdelim, const char *listdir, text *txt)
 {
+	/* It is important for this function to leave the length of the
+	 * processed portion unchanged, or increase it by just one ASCII
+	 * character (for $$). */
 	char *line = *line_p;
 	char *pos = *pos_p;
 	char *token = pos + 1;
@@ -983,12 +986,12 @@
 {
 	char *line;
 	const char *item;
-	char *pos, *subpos;
+	char *pos;
 	char *tmp;
 	char *prev = NULL;
-	int incision, spc, spcnext;
 	int len, i;
 	int directive;
+	int incision, spc;
 	int peeking = 0; /* for a failed conditional without an else */
 	int skipwhite; /* skip whitespace after a conditional directive */
 	int swallow;
@@ -1077,40 +1080,25 @@
 				pos = line + len;
 				skipwhite = 1;
 			}
+			/* We can always line-break where the input had one */
+			spc = len - 1;
 			prev = NULL;
 		} else {
 			len = 0;
 			pos = line;
+			spc = -1;
 			skipwhite = 0;
 		}
 
 		if (txt->skip != NULL) {
-			incision = pos - line;
-			len = incision;
+			incision = len;
 		} else {
 			incision = -1;
 		}
-		spc = -1;
-		spcnext = 0;
 		directive = 0;
 		while (*pos != '\0') {
 			if (txt->wrapwidth != 0 && len >= txt->wrapwidth &&
-					!peeking) {
-				if (spcnext < txt->wrapwidth || spc == -1) {
-					subpos = line + spcnext;
-					while (subpos < pos) {
-						if (*subpos == ' ') {
-							spc = subpos - line;
-						}
-						spcnext++;
-						if (spcnext >= txt->wrapwidth &&
-								spc != -1)
-								break;
-						subpos++;
-					}
-				}
-				if (spc != -1) break;
-			}
+					!peeking && spc != -1) break;
 			if (*pos == '\r') {
 				*pos = '\0';
 				pos++;
@@ -1127,7 +1115,6 @@
 			} else if (*pos == ' ') {
 				if (txt->skip == NULL) {
 					spc = pos - line;
-					spcnext = spc + 1;
 				}
 			} else if (*pos == '\t') {
 				/* Avoid breaking due to peeking */
@@ -1155,7 +1142,9 @@
 				if (peeking) break;
 				substitute_one(&line, &pos, listaddr,
 						listdelim, listdir, txt);
+				if (len != pos - line) {
 				len = pos - line;
+				}
 				skipwhite = 0;
 				/* The function sets up for the next character
 				 * to process, so continue straight away. */
@@ -1172,7 +1161,6 @@
 						 * later */
 						incision = pos - line;
 					}
-					len = incision;
 				} else {
 					if (incision != -1) {
 					    /* Time to cut */
@@ -1185,6 +1173,8 @@
 					    }
 					    incision = -1;
 					}
+				}
+				if (len != pos - line) {
 					len = pos - line;
 				}
 				/* handle_directive() sets up for the next
@@ -1195,7 +1185,6 @@
 				break;
 			}
 			if (txt->skip == NULL) {
-				if (spcnext == len) spcnext++;
 				len++;
 			}
 			pos++;