[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#40692: 28.0.50; Constant list modified inside function
From: |
Ivan Andrus |
Subject: |
bug#40692: 28.0.50; Constant list modified inside function |
Date: |
Fri, 17 Apr 2020 17:45:12 -0600 |
Starting from Emacs -Q (I am building off of trunk, but my brother verified the
same behavior in 26.3), if I evaluate the following code I get an error the
second time I call withdraw.
(defmacro show (var)
`(message ,(format "%S %%S" var) ,var))
(defun my-test-fun (amount params)
(when (memq 'tricked-ya params)
(error "What happened here?"))
(show amount)
(show params)
(setcdr (cdr params) (list 'tricked-ya))
(show params))
(defun fun-withdraw (amount)
(my-test-fun amount
`((amount . , amount)
(const . some-constant))))
(fun-withdraw 12)
(fun-withdraw 12) ;; The second time it's called it will error because the
"constant" list was modified.
I believe this is the root cause of a bug in magit/forge
https://github.com/magit/forge/issues/267 in which all subsequent pull requests
created have the same name. The maintainer of magit/forge (tarsius) was unable
to reproduce that bug, so I tried my hand at creating a minimal test case, and
I was able to get it down to this.
Now, I understand reference semantics of lists in general, but it seems like
this should be different. If this behavior is intentional, what's the best way
to for creation of a new list every time so that functions using the alist
don't have to worry about not changing the list?
-Ivan Andrus
In GNU Emacs 28.0.50 (build 18, x86_64-apple-darwin18.7.0, NS appkit-1671.60
Version 10.14.6 (Build 18G4032))
of 2020-04-10 built on iandrus-macOS
Repository revision: 965390ca5f206c6358014574fe5140ba40850391 -- some local
changes on top of e18c24b35a7cf9bb1b91288b706fa448ed28a7c2
Repository branch: master
Windowing system distributor 'Apple', version 10.3.1671
System Description: Mac OS X 10.14.6
Configured using:
'configure
PKG_CONFIG_PATH=/opt/X11/lib/pkgconfig:/usr/local/opt/libxml2/lib/pkgconfig
--with-sound=yes --with-ns --with-modules --with-file-notification=yes
--enable-gcc-warnings=warn-only --with-xpm --with-jpeg --with-tiff
--with-gif --with-png --with-rsvg --with-xml2 --with-imagemagick
--with-json --with-xft --with-libotf --with-gnutls=no --with-makeinfo
--with-libgmp'
Configured features:
RSVG IMAGEMAGICK GLIB NOTIFY KQUEUE ACL LIBXML2 ZLIB TOOLKIT_SCROLL_BARS
XIM NS MODULES THREADS JSON PDUMPER GMP
Important settings:
value of $LANG: en_US.UTF-8
locale-coding-system: utf-8-unix
- bug#40692: 28.0.50; Constant list modified inside function,
Ivan Andrus <=