unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Eli Zaretskii'" <eliz@gnu.org>
Cc: lekktu@gmail.com, dann@ics.uci.edu, lennart.borgman@gmail.com,
	emacs-devel@gnu.org
Subject: RE: make-variable-buffer-local change
Date: Fri, 25 Sep 2009 13:10:51 -0700	[thread overview]
Message-ID: <E6D1D2C2A88C4368945EFC048B07C7E0@us.oracle.com> (raw)
In-Reply-To: <83fxaau9hd.fsf@gnu.org>

> > > I'd say "use" covers both kinds of use, but I'll defer to natives.
> > 
> > It's not a question of native speakers. "Use" here refers 
> > to what the string is _for_; what it is used for.
> 
> Would replacing "use" with "set" solve this problem?

You missed the point. We don't want to talk about places where the value is set.
What's important is what the variable is for.

We can give an example of where it is used, if we like, but not where it is set
(pretty much irrelevant). It so happens that the place of consumption
(buffer-listing code) can be far from the place it is set (e.g. Dired).

What is important is to let users know that they can set it for use by
buffer-listing code.

> > > > Please correct the doc string accordingly.
> > > 
> > > Don't hesitate to send a patch and I'll commit it ASAP.
> > 
> > Just get rid of "Dired, shell and other modes use this variable."
> 
> Sorry, I happen to disagree.  This is important information for such
> an obscure feature.  The alternative is let the user grep the whole
> lot of Lisp sources.

You missed the point.

What's most important about this variable is what it is for, not where in the
existing code it happens to be set. _Why_ it is set is important. Which is to
say _what it does_ is important - what you can use it for.

What this variable does is provide text to use for non-file buffers, when
buffers are listed along with their associated file names. It is the consumer of
this feature, buffer-listing code, that needs the feature.

It is for that use that code (anywhere) sets the variable. It's not for Dired or
shell that Dired and shell set this variable - it's for buffer-listing purposes
by Buffer Menu and the `Buffers' menus.

As I mentioned to Juanma off list, we could add something about the use of the
variable, if that helps to clarify things. But what we add shouldn't be about
which code happens to set the variable, but rather which code lists buffers and
thus makes use of the variable.

I suggested adding something like this, if people think something additional is
needed:

"For example, in the Buffer Menu (`C-x C-b'), if the local value for a given
non-file buffer is non-nil, then it is shown in the `File' column for that
buffer."

(Perhaps "it" should be replaced with "that value", to be clearer.)

That describes (gives an example of) what this variable does and how to use it:
set its local value for code that lists buffers and their "files"
(descriptions).






  reply	other threads:[~2009-09-25 20:10 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-25 16:09 make-variable-buffer-local change Dan Nicolaescu
2009-09-25 16:32 ` Juanma Barranquero
2009-09-25 16:50   ` Lennart Borgman
2009-09-25 16:57     ` Juanma Barranquero
2009-09-25 17:35       ` Drew Adams
2009-09-25 17:41         ` Juanma Barranquero
2009-09-25 18:06           ` Drew Adams
2009-09-25 19:09             ` Eli Zaretskii
2009-09-25 20:10               ` Drew Adams [this message]
2009-09-26  9:10                 ` Eli Zaretskii
2009-09-25 21:04               ` Stefan Monnier
2009-09-25 22:10                 ` Drew Adams
2009-09-26  1:30                   ` Stefan Monnier
2009-09-26  1:42                     ` Juanma Barranquero
2009-09-26  9:03                 ` Eli Zaretskii
2009-09-25 17:07   ` Dan Nicolaescu
2009-09-25 17:31     ` Juanma Barranquero
2009-09-25 19:46       ` Dan Nicolaescu
2009-09-25 20:16         ` Juanma Barranquero
2009-09-25 20:51           ` Dan Nicolaescu
2009-09-25 21:21             ` Juanma Barranquero
2009-09-25 21:13         ` Tom Tromey
2009-09-25 19:49     ` Stefan Monnier
2009-09-25 21:07       ` Stefan Monnier
2009-09-25 21:25         ` Dan Nicolaescu
2009-09-25 21:44           ` Tom Tromey
2009-09-25 19:50 ` Stefan Monnier

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=E6D1D2C2A88C4368945EFC048B07C7E0@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=dann@ics.uci.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=lekktu@gmail.com \
    --cc=lennart.borgman@gmail.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).