From: Lars Ingebrigtsen <larsi@gnus.org>
To: Drew Adams <drew.adams@oracle.com>
Cc: 16483@debbugs.gnu.org
Subject: bug#16483: 24.3.50; `read-face-name' is a mess now (regression)
Date: Fri, 29 Apr 2016 17:37:00 +0200 [thread overview]
Message-ID: <871t5o2zqb.fsf@gnus.org> (raw)
In-Reply-To: <55d774e6-27f4-4e9f-a3e5-4aa1e43f0a0e@default> (Drew Adams's message of "Fri, 17 Jan 2014 15:54:11 -0800 (PST)")
Drew Adams <drew.adams@oracle.com> writes:
> The doc string is a mess now. Quite unclear, and incorrect in more than
> one way. And so is the code a mess.
>
> 1. The doc string no longer says what the behavior is if DEFAULT is nil.
> In Emacs 24.3, it said this for that case:
>
> If DEFAULT is nil, the list of default face names is taken from
> the `read-face-name' property of the text at point, or, if that
> is nil, from the `face' property of the text at point.
>
> What happens now? No information about that. See below for my guess.
> Please tell users of the function what its behavior is.
As far as I can see, if DEFAULT is nil, there is no list of default face
names?
> 2. The doc string says that DEFAULT is returned if the user enters the
> empty string. That is clearly wrong, at least when DEFAULT is a list of
> faces or face names. DEFAULT is not returned in such cases. The most
> that can be said in general is that DEFAULT _determines_ what is
> returned for empty input - not that DEFAULT _is_ what is returned.
I've now made the doc string more correct, if more difficult to parse.
> 5. The code is wrong, at least in this regard: If DEFAULT is a list of
> strings (face names), and if MULTIPLE is nil, both of which are OK per
> the doc string, then you get this:
>
> Debugger entered--Lisp error: (wrong-type-argument symbolp "font-lock-comme=
> nt-face")
> * symbol-name("font-lock-comment-face")
I've fixed that now.
> 8. What happened to the useful defaulting of previous Emacs versions?
> Yes, I know why you made the change, but now any existing code that uses
> `read-face-name' is broken if it depends on `r-f-n' to provide such
> defaulting. Too bad.
>
> Please consider: There is more Elisp code in the world than just what is
> distributed by Emacs Dev. `read-face-name' has been and continues to be
> a poster child of how not to evolve code. It has morphed in incompatible
> ways from version to version. The right way to do what you wanted to do
> for Emacs 24.4 would have been to go ahead and define `face-at-point',
> but to *use* it in `read-face-name', so that that function continues to
> provide the expected defaulting when DEFAULT is nil:
>
> (unless default (setq default (face-at-point)))
>
> I somehow doubt that that part of the regression will be fixed, but
> perhaps some of the other points above have a chance of being addressed.
Yes, that's outside the scope of read-face-name, I think.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
next prev parent reply other threads:[~2016-04-29 15:37 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-17 23:54 bug#16483: 24.3.50; `read-face-name' is a mess now (regression) Drew Adams
2016-04-29 15:37 ` Lars Ingebrigtsen [this message]
2016-04-29 18:09 ` Drew Adams
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=871t5o2zqb.fsf@gnus.org \
--to=larsi@gnus.org \
--cc=16483@debbugs.gnu.org \
--cc=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.