From: NeilBrown <neil@brown.name>
Date: Fri, 21 Jul 2023 08:08:44 +0000 (+1000)
Subject: popup:  fix popup_child_notify
X-Git-Url: http://git.neil.brown.name/?a=commitdiff_plain;h=26d93a35659647375aa4b973b21cbef372fdf591;p=edlib.git

popup:  fix popup_child_notify

Two silly typos meant this didn't work properly.

Signed-off-by: NeilBrown <neil@brown.name>
---

diff --git a/DOC/TODO.md b/DOC/TODO.md
index 39b66393..2711a537 100644
--- a/DOC/TODO.md
+++ b/DOC/TODO.md
@@ -25,7 +25,7 @@ the file.
       time.
       When searching backwards and result is just above display, the
       result doesn't get shown.  Sometimes.
-- [ ] visiting a new file from a 44 popup is weird.
+- [X] visiting a new file from a 44 popup is weird.
 - [ ] search in history always finds a new history line.  It doesn't
       check if new patterns still matches this line.
 - [ ] in notmuch cursor should go to query result when changing
diff --git a/lib-popup.c b/lib-popup.c
index 793651de..06e858d7 100644
--- a/lib-popup.c
+++ b/lib-popup.c
@@ -415,13 +415,13 @@ DEF_CMD(popup_child_notify)
 		return 1;
 restart:
 	list_for_each_entry(old, &p->children, siblings) {
-		if (c->z == 0)
+		if (c->z != 0)
 			/* Ignore */
 			continue;
 		if (old == c)
 			/* This pane is under control... */
 			continue;
-		if (c->damaged & DAMAGED_CLOSED)
+		if (old->damaged & DAMAGED_CLOSED)
 			continue;
 		if (ci->num > 0) {
 			/* Not the pane we just added, so close it */