GNU bug report logs - #12314
24.2.50; `add-to-history': use `setq' with `delete'

Previous Next

Package: emacs;

Reported by: "Drew Adams" <drew.adams <at> oracle.com>

Date: Thu, 30 Aug 2012 23:10:01 UTC

Severity: normal

Found in version 24.2.50

Done: Chong Yidong <cyd <at> gnu.org>

Bug is archived. No further changes may be made.

Full log

Message #46 received at 12314 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 12314 <at> debbugs.gnu.org, cyd <at> gnu.org, Andreas Schwab <schwab <at> linux-m68k.org>
Subject: Re: bug#12314: 24.2.50; `add-to-history': use `setq' with `delete'
Date: Sat, 08 Sep 2012 18:50:40 +0200

Eli Zaretskii <eliz <at> gnu.org> writes:

> Sorry, I'm not sure how this is related.  Please elaborate.

`delete' may modify the list, so other references into the list may get
surprising results after you've modified it.

The current doc string is kinda obscure, though.  It stars with

---
Delete by side effect any occurrences of ELT as a member of SEQ.
---

which is like "err".

Perhaps something like

Delete any occurrences of ELT from SEQ.  This operation may
destructively modify SEQ.  See `remove' for a non-destructive version.

would be more readable.

-- 
(domestic pets only, the antidote for overdose, milk.)
  http://lars.ingebrigtsen.no  *  Lars Magne Ingebrigtsen
This bug report was last modified 12 years and 251 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.