unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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.

  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

  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=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 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).