bug#42232: Fix for lisp/allout-widgets void variable

[Ken Manheimer <ken.manheimer@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 957 bytes --]

The attached patch provides for a background void-variable error that
happens when parts of an outline are being concealed. Because the error
happens via the after-change-functions hook, it was being caught by an
allout provision and reported in the mode line.

To see the error before applying the patch, make a simple outline, activate
allout-mode, and then conceal a part of the outline by positioning on the
item whose contents you can hide and using `allout-hide-current-subtree'
(by default, bound to '\C-c SPC \C-h'). When the problem occurs you will
see a message in the mode line, like:

allout-widgets-last-hook-error stored,
allout-widgets-hook-error-handler/error (void-variable handled-conceal)
 6-Jul-2020 01:10:58 PM

You can then see that message and a full backtrace in the *Messages* buffer.

(The let declaration for the variable was probably removed because use of
the variable is oblique. It happens through `(symbol-value handling)'.)

[-- Attachment #1.2: Type: text/html, Size: 1056 bytes --]

[-- Attachment #2: 0001-Provide-missing-let-definition-to-prevent-background.patch --]
[-- Type: application/octet-stream, Size: 1073 bytes --]

From 5412a6688e34ae6f649486c84483bea290d90f80 Mon Sep 17 00:00:00 2001
From: Ken Manheimer <ken.manheimer@gmail.com>
Date: Mon, 6 Jul 2020 13:19:11 -0400
Subject: [PATCH] Provide missing let definition to prevent background
 void-variable error.

* lisp/allout-widgets.el (allout-widgets-exposure-change-processor)
Let-declare handled-conceal, for reference through `(symbol-value)'
within the let body. (Because the error happens in an
after-change-functions hook, so it is caught and reported as a message
by allout-widgets-hook-error-handler.)
---
 lisp/allout-widgets.el | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lisp/allout-widgets.el b/lisp/allout-widgets.el
index fbdddca7d7..4f028b9612 100644
--- a/lisp/allout-widgets.el
+++ b/lisp/allout-widgets.el
@@ -988,6 +988,7 @@ allout-widgets-exposure-change-processor
         ;; have to distinguish between concealing and exposing so that, eg,
         ;; `allout-expose-topic's mix is handled properly.
         handled-expose
+        handled-conceal
         covered
         deactivate-mark)
 
-- 
2.17.1