[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master f51f963: Fix some side-effecting uses of make-text-button
From: |
Stefan Monnier |
Subject: |
Re: master f51f963: Fix some side-effecting uses of make-text-button |
Date: |
Thu, 04 Jun 2020 18:44:10 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
> (make-text-button name nil
> 'type 'apropos-library
> 'face 'apropos-symbol
> - 'apropos-symbol name)
> - name)))
> + 'apropos-symbol name))))
So, IIUC we consider `make-text-button` to be functional rather than
side-effecting, right?
> @@ -202,7 +202,7 @@ The format has been repaired and the variable modified
> accordingly.
> You can save the current value through the customize system by
> either clicking or hitting return "
> (make-text-button
> - "here" nil
> + (copy-sequence "here") nil
> 'face '(:weight bold :inherit button)
> 'mouse-face '(:weight normal :background "gray50" :inherit button)
> 'follow-link t
So, here why do we need to `copy-sequence`?
Stefan
- Re: master f51f963: Fix some side-effecting uses of make-text-button,
Stefan Monnier <=
- Re: master f51f963: Fix some side-effecting uses of make-text-button, Paul Eggert, 2020/06/04
- Re: master f51f963: Fix some side-effecting uses of make-text-button, Basil L. Contovounesios, 2020/06/05
- Re: master f51f963: Fix some side-effecting uses of make-text-button, Paul Eggert, 2020/06/05
- Re: master f51f963: Fix some side-effecting uses of make-text-button, Pip Cet, 2020/06/06
- Re: master f51f963: Fix some side-effecting uses of make-text-button, Paul Eggert, 2020/06/06