emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#10621: closed (Incorrect usage of hash procedures


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#10621: closed (Incorrect usage of hash procedures in (ice-9 mapping))
Date: Sun, 24 Feb 2013 14:13:02 +0000

Your message dated Sun, 24 Feb 2013 15:10:21 +0100
with message-id <address@hidden>
and subject line Re: bug#10621: Incorrect usage of hash procedures in (ice-9 
mapping)
has caused the debbugs.gnu.org bug report #10621,
regarding Incorrect usage of hash procedures in (ice-9 mapping)
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
10621: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10621
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: Incorrect usage of hash procedures in (ice-9 mapping) Date: Fri, 27 Jan 2012 03:25:15 -0500
These are genuine errors:

ice-9/mapping.scm:97:48: warning: possibly wrong number of arguments to 
`hashx-get-handle'
ice-9/mapping.scm:94:48: warning: possibly unbound variable 
`hashx-create-handle'

     Mark



--- End Message ---
--- Begin Message --- Subject: Re: bug#10621: Incorrect usage of hash procedures in (ice-9 mapping) Date: Sun, 24 Feb 2013 15:10:21 +0100 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux)
On Wed 28 Nov 2012 16:56, Daniel Hartwig <address@hidden> writes:

> A short module, it is not hard to fix, though given all of the above it
> makes sense to simply remove it.

I think removing it is the right thing.  Even with your patch it still
won't work because the default "table constructor" is a vector instead
of a hash table.

Regardless of whether it were new or old, I would much prefer to remove
this module than to document it properly, especially given the existence
of alternatives like srfi-69 and rnrs hash tables as you mention.

I will apply your patch (so that the compile warnings go away), add a
deprecation notice, and remove the module in master.

Andy
-- 
http://wingolog.org/


--- End Message ---

reply via email to

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