[Top][All Lists]

[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:

> 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.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]