notmuch master_view: catch exception so .recursed in reset properly.

If self.list_pane.call(key, **a) raised an exception, self.recursed
would not be reset, and so would cause repeated warnings.
So DTRT.

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

diff --git a/python/module-notmuch.py b/python/module-notmuch.py
index 0f7cc11fbdfb0bec64996d38f8740b0cd181c638..f2890f5c80d0254884bbdc2aea5ee0bc51e72077 100644 (file)
--- a/python/module-notmuch.py
+++ b/python/module-notmuch.py
@@ -1926,14 +1926,17 @@ class notmuch_master_view(edlib.Pane):
     def handle_maindoc(self, key, **a):
         "handle-prefix:doc:notmuch:"
         # any doc:notmuch calls that haven't been handled
-        # are handled to the list_pane
+        # are handed to the list_pane
         if self.recursed == key:
             edlib.LOG("doc:notmuch: recursed!", key)
+            edlib.LOG_BT()
             return edlib.Efail
         prev = self.recursed
         self.recursed = key
-        # FIXME catch exception to return failure state properly
-        ret = self.list_pane.call(key, **a)
+        try:
+            ret = self.list_pane.call(key, **a)
+        except edlib.commandfailed:
+            ret = edlib.Efail
         self.recursed = prev
         return ret