From: "Drew Adams" <drew.adams@oracle.com>
Subject: RE: how to control isearch for invisible text
Date: Sat, 12 Aug 2006 16:02:09 -0700 [thread overview]
Message-ID: <EIENLHALHGIMHGDOLMIMGEHKCKAA.drew.adams@oracle.com> (raw)
In-Reply-To: <858xlt4mj4.fsf@lola.goethe.zz>
> More importantly, I think the option should be documented - perhaps
> in the section on invisible text, if there is to be no isearch
> toggle for it. If you want to remove the user option and make it a
> plain `defvar', that's your choice; I have no actual example that
> will dissuade you. End of story.
The doc string mentions outline.el and hideshow.el. Both of those
don't have invisible text as much as hidden text, text that clearly is
part of the buffer, with the invisibility being a display aspect and
not a text aspect. The overlays will also get an
`isearch-open-invisible' property allowing them to open. For this
sort of thing, the default setting of search-invisible, namely `open',
makes sense. I don't see where a different setting would make sense
in connection with those modes which cater with appropriate properties
for opening in searches. `open' might be less necessary when someone
uses reveal-mode, but I have not checked it.
In short: I fail to see where a global setting of search-invisible to
anything other than `open' would make even remotely sense with the
mentioned outline.el and hideshow.el.
For dired-mode, this is not as clearcut in my case. When I hide
aspects of the buffer away with an explicit command, it is less likely
that I actually would want to find them. And those overlays don't use
`isearch-open-invisible' seemingly, which again seems to match what is
useful and likely the intent of the user.
So in short, I see nothing worth changing here. If your needs as a
programmer are sufficiently strange that you'll need to meddle with
those variables, you'll be reading the source file for searches,
anyway.
I don't know what you're arguing about. I never asked you to change the
default value of the option. I never argued for special uses in different
modes. I never suggested changing anything about the option's behavior. I
never indicated any "sufficiently strange" "needs as a programmer" or any
interest in "meddling with those variables". We spoke of only one variable,
and I assure you that I have no desire to meddle with it.
I simply said that I'm glad there is such a user option, and I hope it will
remain an option. And I thank you again for pointing it out.
I didn't find such an option when I looked for it in the doc, and I added
that I thought it would help to mention it in the Emacs manual, so people
can be aware of it. You don't agree, so it will remain findable via `M-x
apropos-documentation RET isearch.*invisible RET' or `M-x apropos-variable
RET search.*invisible RET' ("search", not "isearch"). Fair enough.
You also indicated that you think it shouldn't be a user option. If you
change it to a `defvar', I'll still be able to use it, so I don't have a
problem, for my own use. Please leave the doc string, at least, to help
others find the variable using `apropos-*' ;-).
I added that I thought it might be useful to toggle it during `isearch'. You
don't think so. End of story.
next prev parent reply other threads:[~2006-08-12 23:02 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-12 8:30 how to control isearch for invisible text Drew Adams
2006-08-12 8:49 ` David Kastrup
2006-08-12 17:09 ` Drew Adams
2006-08-12 17:16 ` David Kastrup
2006-08-12 17:56 ` Drew Adams
2006-08-12 19:21 ` David Kastrup
2006-08-12 20:54 ` Drew Adams
2006-08-12 21:02 ` David Kastrup
2006-08-12 21:17 ` Drew Adams
2006-08-12 21:34 ` David Kastrup
2006-08-12 22:07 ` Drew Adams
2006-08-12 22:27 ` David Kastrup
2006-08-12 23:02 ` Drew Adams [this message]
2006-08-13 0:28 ` Stefan Monnier
2006-08-13 7:14 ` David Kastrup
2006-08-13 17:52 ` Richard Stallman
2006-08-13 18:00 ` David Kastrup
2006-08-14 0:36 ` Stefan Monnier
2006-08-14 7:12 ` David Kastrup
2006-08-14 12:14 ` Stefan Monnier
2006-08-14 12:24 ` David Kastrup
2006-08-14 12:59 ` Stefan Monnier
2006-08-14 13:48 ` David Kastrup
2006-08-14 14:43 ` Stefan Monnier
2006-08-14 15:05 ` David Kastrup
2006-08-14 15:23 ` Drew Adams
2006-08-14 15:32 ` David Kastrup
2006-08-15 12:41 ` Richard Stallman
2006-08-15 12:40 ` Richard Stallman
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=EIENLHALHGIMHGDOLMIMGEHKCKAA.drew.adams@oracle.com \
--to=drew.adams@oracle.com \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.