all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: 22339-done@debbugs.gnu.org
Subject: bug#22339: 25.1.50; Shouldn't `abbrev-expand-function' be a user option?
Date: Tue, 16 Feb 2016 06:38:48 -0800 (PST)	[thread overview]
Message-ID: <90daf4b6-8065-47a7-a335-0ad52dcfb1a8@default> (raw)
In-Reply-To: <87y4al3zjs.fsf@gnus.org>

> > Subject line says it all.  Presumably this is a variable intended for
> > users to customize/set to a function of their liking.
> 
> No, it looks more like something for programmers, and not users.  Closing.

Why do you think that?

Command `expand-abbrev' is user-facing.  Its doc string says that it:

  Calls `abbrev-expand-function' with no argument to do the work,
  and returns whatever it does.  (This should be the abbrev symbol
  if expansion occurred, else nil.)

That tells users that they can set `abbrev-expand-function' to a function
that has the behavior they want and return value described.  There would be
no reason to tell users about this if they were not expected to customize
the value to a function of their choice.  In that case, it would just be
part of the internal implementation of the command.

What's more, it used to be a user option, `pre-abbrev-expand-hook'.
It is still is, but is replaced (deprecated) by `abbrev-expand-function'.
There is nothing in NEWS about replacing this user option by an internal
variable.  The most that appears in NEWS, AFAICT, is that in Emacs 23
`pre-abbrev-expand-hook' was deprecated in favor of `abbrev-expand-functions',
and that has now be deprecated in favor of `abbrev-expand-function'.
Nothing in NEWS about this no longer being a user-customizable feature
(option).

The judgment that this variable "looks more like something for
programmers, and not users" makes little sense, to me.  What makes
it look that way, to you?  Why should users not set this variable?
And if they should not, why tell them about it in the command's doc?





      reply	other threads:[~2016-02-16 14:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-09 18:06 bug#22339: 25.1.50; Shouldn't `abbrev-expand-function' be a user option? Drew Adams
2016-02-16  7:05 ` Lars Ingebrigtsen
2016-02-16 14:38   ` 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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=90daf4b6-8065-47a7-a335-0ad52dcfb1a8@default \
    --to=drew.adams@oracle.com \
    --cc=22339-done@debbugs.gnu.org \
    --cc=larsi@gnus.org \
    /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.