Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread Tassilo Horn
Stefan Monnier writes: >> That's this :help function in LaTeX-symbols-toolbar-switch-contents: >> >> --8<---cut here---start->8--- >>:help ,(lambda (&rest _ignore) >> (concat "Turn " >> (if LaTeX-symbols-toolbar-v

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread Stefan Monnier
> That's this :help function in LaTeX-symbols-toolbar-switch-contents: > > --8<---cut here---start->8--- >:help ,(lambda (&rest _ignore) > (concat "Turn " > (if LaTeX-symbols-toolbar-visible-flag "off " "on ") >

Re: [a new patch]

2022-08-30 Thread Uwe Brauer
>>> "AE" == Arash Esbati writes: > Hi Uwe, > Uwe Brauer writes: >> I hope it is now ok. > Thanks, we're almost there, I think. Some comments below. I think that's it. # HG changeset patch # User Uwe Brauer # Date 1661861790 -7200 # Tue Aug 30 14:16:30 2022 +0200 # Node ID 7e3b474a5bb4

Re: [a new patch]

2022-08-30 Thread Uwe Brauer
>>> "AE" == Arash Esbati writes: > Hi Uwe, > Uwe Brauer writes: >> I hope it is now ok. > Thanks, we're almost there, I think. Some comments below. I think that's it. # HG changeset patch # User Uwe Brauer # Date 1661861790 -7200 # Tue Aug 30 14:16:30 2022 +0200 # Node ID 7e3b474a5bb4

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread Tassilo Horn
Ikumi Keita writes: >> Is anybody familiar with this code in order to pacify this? > >> git grep LaTeX-symbols-active-menuitem > >> doesn't show anything useful. > > I think the piece of code which involves this variable is relevant only > when the experimental feature symbol-toolbar: > , >

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread Tassilo Horn
David Kastrup writes: Hi David, > Wait, I was thinking of `(lambda ...) but you quoted ,(lambda ...). > So this apparently already was in quasi-quoted context. So the , > before (lambda) likely needs to remain, but list-strings likely has to > become un-commaed. Ah, indeed, that's it. Then I

master 567458f5: Silence byte-compiler.

2022-08-30 Thread Tassilo Horn
branch: master commit 567458f54f83ed89e29b2b7bb3708843772c Author: Tassilo Horn Commit: Tassilo Horn Silence byte-compiler. --- tex-bar.el | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tex-bar.el b/tex-bar.el index acb01c4d..5c36f24b 100644 --- a/tex-bar.el +++ b/tex-bar.el @@ -

master 99470f08: Close over list-strings from outer scope.

2022-08-30 Thread Tassilo Horn
branch: master commit 99470f080d057b970ba00a61c9c97be8f60d1490 Author: Tassilo Horn Commit: Tassilo Horn Close over list-strings from outer scope. * tex-bar.el (menu-strings-buttons-alist): Close over list-strings from outer scope. --- tex-bar.el | 2 +- 1 file changed, 1 inser

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread David Kastrup
David Kastrup writes: >> The list-strings variable is let-bound around the complete defconst >> LaTeX-symbols-toolbar-switch-contents. I guess it should become >> (quote ("foo" "bar" "baz")) instead of (quote \, list-strings). Stefan, >> what's the issue here? > > This looks like it should rath

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread David Kastrup
Tassilo Horn writes: > Arash Esbati writes: > >> After this change, the compiler says: >> >> In toplevel form: >> tex-bar.el:481:34: Warning: reference to free variable >> `LaTeX-symbols-active-menuitem' >> >> Is anybody familiar with this code in order to pacify this? >> >> git grep LaT

Re: master 6b5f00bf: Unquote lambdas in font-latex.el & tex-bar.el; add some FIXMEs

2022-08-30 Thread Ikumi Keita
Hi Arash, > Arash Esbati writes: > After this change, the compiler says: > In toplevel form: > tex-bar.el:481:34: Warning: reference to free variable > `LaTeX-symbols-active-menuitem' > Is anybody familiar with this code in order to pacify this? > git grep LaTeX-symbols-active-menu