GNU bug report logs - #12788
cl-loop-handler, cl-loop-for-handler

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 12788 in the body.
You can then email your comments to 12788 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Glenn Morris <rgm <at> gnu.org>
To: submit <at> debbugs.gnu.org
Subject: cl-loop-handler, cl-loop-for-handler
Date: Fri, 02 Nov 2012 20:03:45 -0400

Package: emacs
Version: 24.2.50

(The inverse of http://debbugs.gnu.org/12773 )

It seems to me that cl-loop-handler, cl-loop-for-handler should not have
been renamed to cl--loop-handler and cl--loop-for-handler.
They are advertised methods to customize the `cl-loop', rather than
internal details..

From cl.texi in 24.2:

    4.7.6 Other Clauses:
    [...]
       While there is no high-level way to add user extensions to
    `loop', this package does offer two properties called
    `cl-loop-handler' and `cl-loop-for-handler' which are functions to
    be called when a given symbol is encountered as a top-level loop
    clause or `for' clause, respectively. Consult the source code in
    file `cl-macs.el' for details.

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

From: Glenn Morris <rgm <at> gnu.org>
To: 12788 <at> debbugs.gnu.org
Subject: Re: bug#12788: cl-loop-handler, cl-loop-for-handler
Date: Fri, 02 Nov 2012 20:39:06 -0400

PS I don't know what (if anything) should happen in this case, but I
also want to point out that cl-gensym-counter and cl-random-state are
documented in cl.texi, but have been renamed with cl-- prefixes.

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

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 12788 <at> debbugs.gnu.org
Subject: Re: bug#12788: cl-loop-handler, cl-loop-for-handler
Date: Fri, 02 Nov 2012 22:29:29 -0400

> It seems to me that cl-loop-handler, cl-loop-for-handler should not have
> been renamed to cl--loop-handler and cl--loop-for-handler.
> They are advertised methods to customize the `cl-loop', rather than
> internal details..

Oops, looks like you're right.  Could you rename them back in emacs-24?

> PS I don't know what (if anything) should happen in this case, but I
> also want to point out that cl-gensym-counter and cl-random-state are
> documented in cl.texi, but have been renamed with cl-- prefixes.

I think we should de-document them.  We can add compatibility aliases
for them to cl.el if needed.


        Stefan

Previous Next

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