From: Drew Adams <drew.adams@oracle.com>
To: Andreas Politz <politza@hochschule-trier.de>,
Dmitry Gutov <dgutov@yandex.ru>
Cc: 14029-done@debbugs.gnu.org
Subject: bug#14029: 24.2.50; [PATCH] imenu problems with special elements
Date: Fri, 29 Nov 2013 07:34:19 -0800 (PST) [thread overview]
Message-ID: <d6823b17-7ed0-441b-bf68-eea665164682@default> (raw)
In-Reply-To: <87pppj2utr.fsf@hochschule-trier.de>
> >>> Is it possible for (cadr item) to be a list and also be `functionp' and
> >>> yet not have its car be `lambda'?
> >
> > The documentation allows FUNCTION in the third element, but
> > imenu--subalist-p' is checking whether the second element is a
> > function.
>
> Yes, this doesn't make much sense. I guess this function check was
> intended to distinguish a special element (INDEX-NAME POS FN . ARGS)
> from a sub-alist element (INDEX-NAME . SUB-ALIST).
>
> The check would make sense, if this function was applied to the cdr of
> an element, i.e. check if the argument is a SUB-ALIST. But this is not
> how this function is used in imenu.el .
>
> I might have initially (and falsely) determined this as the source of
> some bug.
Sorry, I have not been following this thread at all, and am unaware
of the problem being addressed. So please ignore, if this is irrelevant.
Emacs itself still does not use "special" items anywhere. But I do.
I'm just hoping that they are still going to be taken into
consideration and not obliterated. See bug #12717 for background on
submenus and special items. Yes, the imenu.el code is a bit confusing.
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=12717
prev parent reply other threads:[~2013-11-29 15:34 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-22 1:24 bug#14029: 24.2.50; [PATCH] imenu problems with special elements Andreas Politz
2013-03-22 5:12 ` Drew Adams
[not found] ` <87ip4iwfoe.fsf@fh-trier.de>
2013-03-23 15:54 ` Andreas Politz
2013-03-23 16:15 ` Drew Adams
2013-11-24 23:53 ` Dmitry Gutov
2013-11-25 1:32 ` Andreas Politz
2013-11-25 1:40 ` Andreas Politz
2013-11-25 1:32 ` Andreas Politz
2013-11-29 3:43 ` Dmitry Gutov
2013-11-29 13:36 ` Andreas Politz
2013-11-29 14:35 ` Dmitry Gutov
2013-11-29 15:34 ` Drew Adams [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d6823b17-7ed0-441b-bf68-eea665164682@default \
--to=drew.adams@oracle.com \
--cc=14029-done@debbugs.gnu.org \
--cc=dgutov@yandex.ru \
--cc=politza@hochschule-trier.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).