GNU bug report logs - #46754
28.0.50; Various map.el fixes and improvements

Previous Next

Package: emacs;

Reported by: "Basil L. Contovounesios" <contovob <at> tcd.ie>

Date: Wed, 24 Feb 2021 15:16:02 UTC

Severity: normal

Tags: fixed, patch

Found in version 28.0.50

Fixed in version 28.1

Done: "Basil L. Contovounesios" <contovob <at> tcd.ie>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: 46754 <at> debbugs.gnu.org
Cc: nicolas petton <nicolas <at> petton.fr>, stefan monnier <monnier <at> iro.umontreal.ca>
Subject: bug#46754: 28.0.50; Various map.el fixes and improvements
Date: Wed, 24 Feb 2021 15:15:07 +0000
[Message part 1 (text/plain, inline)]
X-Debbugs-Cc: Nicolas Petton <nicolas <at> petton.fr>, Stefan Monnier <monnier <at> iro.umontreal.ca>
Tags: patch

The attached patch for map.el and seq.el was initially motivated by the
fact that (map--plist-p '(nil 1)) currently returns nil, whereas I'd
expect an alist to begin with a cons.

Along the way I noticed some other inconsistencies, such as map-do
returning non-nil for alists, and the use of 5-arg alist-get when map.el
expects Emacs 25; and edge cases such as an OBOE on arrays in
map-delete.

Throughout the patch I assume that cl-defgeneric is generally preferred
over map--dispatch, except where some aspect of the latter can't be
matched by the former, e.g. the use of advertised-calling-convention.

WDYT?

Thanks,

-- 
Basil

[0001-Various-map.el-improvements.patch (text/x-diff, attachment)]

This bug report was last modified 4 years and 160 days ago.

Previous Next


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