all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: storm@cua.dk (Kim F. Storm)
To: rms@gnu.org
Cc: miles@gnu.org, drew.adams@oracle.com, emacs-devel@gnu.org
Subject: Re: cannot understand Elisp manual node Glyphs
Date: Fri, 09 Feb 2007 12:12:46 +0100	[thread overview]
Message-ID: <m3lkj7vb2p.fsf@kfs-l.imdomain.dk> (raw)
In-Reply-To: <E1HFIxs-0001OY-2q@fencepost.gnu.org> (Richard Stallman's message of "Thu\, 08 Feb 2007 18\:46\:08 -0500")

Richard Stallman <rms@gnu.org> writes:

> Please do NOT add a function make-glyph-code now.
> This is not the time for new features.

The initial reason behind this change was to improve _documentation_,
for an _existing_ feature which is definitely non-obvious, and
basically has no clean API (or an API at all).

In fact, I have code which I wrote for Emacs 19 / Emacs 20, which
stopped working in Emacs 21 because the encoding of faces in 
glyph codes changed ...  

IMO, it is a very bad API design to force Lisp code to relying on
internal numeric constants, which have already changed once between
Emacs releases.

AND IT WILL CHANGE AGAIN IN EMACS 23 (using 22 bits instead of 19).

So if _this_ is not the right time for cleaning up this mess, I don't
know when is!!



BTW, the unicode-2 branch seems to have bugs in this area, exactly
because the changes to this encoding at the C-leve are not reflected in
the Lisp code.  My patch would fix those bugs too (with the obvious
changes to the new functions) in Emacs 23 too.  

Furthermore, the unicode branch only allows the first 511 faces
defined to be used in glyph-code (vs. 4088 in Emacs 22) ... so maybe
the whole implementation of glyph-codes need to be revised in Emacs 23
-- yet another reason for making a clean interface for this NOW.

WDOT ??

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

  parent reply	other threads:[~2007-02-09 11:12 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-05 23:11 cannot understand Elisp manual node Glyphs Drew Adams
2007-02-07 13:29 ` Kim F. Storm
2007-02-07 14:54   ` Drew Adams
2007-02-07 15:24     ` Kim F. Storm
2007-02-07 15:53       ` Drew Adams
2007-02-07 16:16         ` Stuart D. Herring
2007-02-07 16:21           ` Drew Adams
2007-02-08 16:38             ` Stuart D. Herring
2007-02-07 22:52   ` Miles Bader
2007-02-08  8:26     ` Kim F. Storm
2007-02-08  8:51       ` David Kastrup
2007-02-08 10:39         ` Kim F. Storm
2007-02-08 23:46           ` Richard Stallman
2007-02-09  7:17             ` David Kastrup
2007-02-09  9:12               ` Markus Triska
2007-02-09  9:43                 ` Nick Roberts
2007-02-09 23:48                 ` Richard Stallman
2007-02-09 14:23               ` Richard Stallman
2007-02-09 11:12             ` Kim F. Storm [this message]
2007-02-09 11:32               ` Juanma Barranquero
2007-02-09 23:48                 ` Richard Stallman
2007-02-09 14:05               ` Kim F. Storm
2007-02-09 23:49                 ` Richard Stallman
2007-02-10  0:40                   ` Drew Adams
2007-02-10 17:40                     ` Richard Stallman
2007-02-11 14:18                       ` Miles Bader
2007-02-11 21:07                       ` Kim F. Storm
2007-02-12 17:53                         ` Richard Stallman
2007-02-14 23:32                           ` Kim F. Storm
2007-02-10 10:19                   ` Eli Zaretskii
2007-02-10 17:41                     ` Richard Stallman
2007-02-10 13:59                   ` Miles Bader
2007-02-11  0:20                     ` Richard Stallman
2007-02-11  1:34                       ` Drew Adams
2007-02-11 14:16                       ` Miles Bader
2007-02-12 17:52                         ` Richard Stallman
2007-02-09 18:16               ` Stuart D. Herring
2007-02-08 16:21       ` Stefan Monnier
2007-02-08 16:36         ` Juanma Barranquero
2007-02-08  9:58     ` Stephen J. Turnbull

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=m3lkj7vb2p.fsf@kfs-l.imdomain.dk \
    --to=storm@cua.dk \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@gnu.org \
    --cc=miles@gnu.org \
    --cc=rms@gnu.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.