[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master dbe3e41: Don't try to eval local variables in Gnus article
From: |
Katsumi Yamaoka |
Subject: |
Re: master dbe3e41: Don't try to eval local variables in Gnus article |
Date: |
Tue, 20 Jun 2017 08:40:38 +0900 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (i686-pc-cygwin) |
On Mon, 19 Jun 2017 16:15:02 -0400, Glenn Morris wrote:
> Katsumi Yamaoka wrote:
>> branch: master
>> commit dbe3e416af5d845dc774341eb66971ab1a72983b
> [...]
>> diff --git a/lisp/gnus/mm-view.el b/lisp/gnus/mm-view.el
>> index dd64bfe..4276f9e 100644
>> --- a/lisp/gnus/mm-view.el
>> +++ b/lisp/gnus/mm-view.el
>> @@ -488,7 +488,8 @@ If MODE is not set, try to find mode automatically."
>> (funcall mode))
>> (let ((auto-mode-alist
>> (delq (rassq 'doc-view-mode-maybe auto-mode-alist)
>> - (copy-sequence auto-mode-alist))))
>> + (copy-sequence auto-mode-alist)))
>> + (local-enable-local-variables nil))
> Hi - local-enable-local-variables, as opposed to enable-local-variables,
> is seldom used. Do you really need the former here?
Yes, binding enable-local-variables did have no effect. Instead,
I found these two options
(let ((inhibit-local-variables-regexps '(""))) ...)
(let ((local-enable-local-variables nil)) ...)
to inhibit set-auto-mode from trying to evaluate local variables
in the article <address@hidden> that can be found as
bug#27391 in the bug-gnu-emacs list, or:
<http://lists.gnu.org/archive/html/bug-gnu-emacs/2017-06/msg00519.html>
> I see
> mm-display-inline-fontify already binds enable-local-variables to nil.
Oh, I overlooked it! I should rearrange them anyway.
> See comments at definition of local-enable-local-variables in files.el.
I did read it, but I haven't investigated why binding
enable-local-variables to nil is not effective thoroughly. I'll
do it.
Thanks.