[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/gnorb 14ac82a 418/449: [gnorb] Use unwind-protect aroun
From: |
Stefan Monnier |
Subject: |
[elpa] externals/gnorb 14ac82a 418/449: [gnorb] Use unwind-protect around the trigger process, bump to 1.5.4 |
Date: |
Fri, 27 Nov 2020 23:16:24 -0500 (EST) |
branch: externals/gnorb
commit 14ac82a17af00a16bb1890ce708e81b66f9e675d
Author: Eric Abrahamsen <eric@ericabrahamsen.net>
Commit: Stefan Monnier <monnier@iro.umontreal.ca>
[gnorb] Use unwind-protect around the trigger process, bump to 1.5.4
* packages/gnorb/gnorb-gnus.el (gnorb-gnus-incoming-do-todo): Was
using condition-case, but unwind-protect makes more sense.
We *always* want to clear out gnorb-gnus-mesage-info and
gnorb-gnus-capture-attachments, no matter what happens in the
trigger process.
---
gnorb-gnus.el | 11 ++++-------
gnorb.el | 2 +-
2 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/gnorb-gnus.el b/gnorb-gnus.el
index 32410a5..16cf7ac 100644
--- a/gnorb-gnus.el
+++ b/gnorb-gnus.el
@@ -550,7 +550,7 @@ you'll stay in the Gnus summary buffer."
:link ,link :date ,date :refs ,ref-msg-ids
:group ,group))
(gnorb-gnus-collect-all-attachments nil t)
- (condition-case err
+ (unwind-protect
(if id
(progn
(delete-other-windows)
@@ -617,12 +617,9 @@ you'll stay in the Gnus summary buffer."
(mail-header-id (gnus-data-header (gnus-data-find a)))
tags))
(gnus-summary-update-article a))))
- (error
- ;; If these are left populated after an error, it plays hell
- ;; with future trigger processes.
- (setq gnorb-gnus-message-info nil)
- (setq gnorb-gnus-capture-attachments nil)
- (signal (car err) (cdr err))))))
+ ;; No matter what, clear these two variables.
+ (setq gnorb-gnus-message-info nil)
+ (setq gnorb-gnus-capture-attachments nil))))
;;;###autoload
(defun gnorb-gnus-quick-reply ()
diff --git a/gnorb.el b/gnorb.el
index dba4595..8bf57a4 100644
--- a/gnorb.el
+++ b/gnorb.el
@@ -2,7 +2,7 @@
;; Copyright (C) 2018 Free Software Foundation, Inc.
-;; Version: 1.5.3
+;; Version: 1.5.4
;; Package-Requires: ((cl-lib "0.5"))
;; Maintainer: Eric Abrahamsen <eric@ericabrahamsen.net>
- [elpa] externals/gnorb 806775c 272/449: Wrap save-excursion around more of gnorb-org-view, (continued)
- [elpa] externals/gnorb 806775c 272/449: Wrap save-excursion around more of gnorb-org-view, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 982aee9 271/449: Adjust requires and autoloads, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb a55b0c0 273/449: Make gnorb-ids precious earlier in registry setup, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 3877b2d 275/449: Still didn't have incoming-do-todo logic right, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 94f6897 279/449: Don't use with-eval-after-load, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 3801ad7 280/449: Check both gnus version and emacs version., Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 8c333ee 281/449: Merge pull request #20 from totherme/master, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 8e705ea3 380/449: Sneakily add basic EBDB support to Gnorb, bump to 1.3.0, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 4b5a5a1 389/449: Gnorb: Tweak manual a bit, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 5672545 396/449: Gnorb: Typo, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 14ac82a 418/449: [gnorb] Use unwind-protect around the trigger process, bump to 1.5.4,
Stefan Monnier <=
- [elpa] externals/gnorb 471883d 394/449: Gnorb: Only add one registry-related hook to the Org capture process, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb babfa5c 393/449: Fix to after capture function, bump to 1.3.4, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb a6916b6 402/449: Allow gnorb-msg-id-request-head to accept a "group" argument, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb a1c8f0e 417/449: [gnorb] Small gnorb-org bugs, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 034f10a 434/449: [gnorb] Use characterp, not numberp, for testing register key, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb e1dae56 430/449: [gnorb] Only output summary-line info if registry exists, bump, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb c3279d2 284/449: Fix tracking messages from virtual groups, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 41c6778 286/449: nngnorb should be a virtual server, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 6e6ee46 285/449: Zap another with-eval-after-load, Stefan Monnier, 2020/11/27
- [elpa] externals/gnorb 09679fa 287/449: Misspelled function name, Stefan Monnier, 2020/11/27