[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: add-to-list on lexical vars (Re: Error on loading polyglossia.el)
From: |
Tassilo Horn |
Subject: |
Re: add-to-list on lexical vars (Re: Error on loading polyglossia.el) |
Date: |
Sat, 09 Jan 2021 17:14:18 +0100 |
User-agent: |
mu4e 1.5.7; emacs 28.0.50 |
Ikumi Keita <ikumi@ikumi.que.jp> writes:
> OK, the attached revised patch includes a new test.
Looks good. Is there a problem with just adding a
(defvar TeX-test-dummy-alist nil)
for the test variable and then declaring lexical-binding t for the file?
Usually, the test are only run thru make, and then there's no problem
with declaring new variables.
>>> and `TeX-add-to-alist' runs without error for me, even without the
>>> patch.
>
>> Not for me. Just eval in *scratch* which has lexical-binding at least
>> in my emacs master checkout:
>> (let ((alist nil))
>> (TeX-add-to-alist 'alist '((a 1)))
>> alist) ;; <== C-j here
>
> Well, that wouldn't work even after my patch is applied.
No, it's expected that it doesn't work in the same way as it doesn't
work for `add-to-list'. As you say, it's only useful for special
variables.
Bye,
Tassilo
Re: Error on loading polyglossia.el, Joost Kremers, 2021/01/09