emacs-devel
[Top][All Lists]
Advanced

[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




reply via email to

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