Fix the behaviour of constructions like \e{index \i{term}} -- the index tag

was causing emphasis to be broken (particularly noticeable in text-like
backends). There are some instances of this in the PuTTY manual, for instance.

Unfortunately, \k references in similar situations still aren't quite right,
but fixing that will be more involved, and I haven't found any instances
yet.

[originally from svn r7049]

1 files changed, 23 insertions, 8 deletions

diff --git a/misc.c b/misc.c
index 1d407de..3f2483c 100644
--- a/misc.c
+++ b/misc.c
@@ -237,17 +237,32 @@ void mark_attr_ends(word *words)
 
     wp = NULL;
     for (w = words; w; w = w->next) {
-	if (isattr(w->type)) {
-	    int before = (wp && isattr(wp->type) &&
-			  sameattr(wp->type, w->type));
-	    int after = (w->next && isattr(w->next->type) &&
-			 sameattr(w->next->type, w->type));
-	    w->aux |= (before ?
-		       (after ? attr_Always : attr_Last) :
-		       (after ? attr_First : attr_Only));
+	int both;
+	if (!isvis(w->type))
+	    /* Invisible elements should not affect this calculation */
+	    continue;
+	both = (isattr(w->type) &&
+		wp && isattr(wp->type) &&
+		sameattr(wp->type, w->type));
+	w->aux |= both ? attr_Always : attr_First;
+	if (wp && !both) {
+	    /* If previous considered word turns out to have been
+	     * the end of a run, tidy it up. */
+	    int wp_attr = attraux(wp->aux);
+	    wp->aux = (wp->aux & ~attr_mask) |
+		((wp_attr == attr_Always) ? attr_Last
+			 /* attr_First */ : attr_Only);
 	}
 	wp = w;
     }
+
+    /* Tidy up last word touched */
+    if (wp) {
+	int wp_attr = attraux(wp->aux);
+	wp->aux = (wp->aux & ~attr_mask) |
+	    ((wp_attr == attr_Always) ? attr_Last
+		     /* attr_First */ : attr_Only);
+    }
 }
 
 /*