all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alan Mackenzie <acm@muc.de>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-devel@gnu.org
Subject: Re: Difficulties with `purecopy' discarding text properties from strings
Date: Fri, 19 May 2017 19:54:58 +0000	[thread overview]
Message-ID: <20170519195458.GB5330@acm.fritz.box> (raw)
In-Reply-To: <jwvo9uohczs.fsf-monnier+gmane.emacs.devel@gnu.org>

Hello, Stefan.

On Fri, May 19, 2017 at 15:22:50 -0400, Stefan Monnier wrote:
> > Presumably there are good reasons for discarding text properties when
> > purecopying strings.

> I don't think so.

> > Or, maybe, there was just no particular reason to amend the workings
> > of `purecopy' when text properties came into existence.

> I think that's what happens.

Yes, that's probably what happened.

> But wait a minute:
> > I naively constructed such a defcustom, with text properties on the
> > value option strings.

> How exactly did you do that?  Using #("..." ...)
> or using (propertize "..." ...)?

By using propertize:

(defun mode-line-%-propertize (str)
  "Add standard mode-line properties to STR, a string.
These enable certain mouse operations when the mouse pointer is
hovering over the display of STR in the mode line."
  (propertize
   str
   'local-map mode-line-column-line-number-mode-map
   'mouse-face 'mode-line-highlight
   ;; XXX needs better description
   'help-echo "Size indication mode\n\
mouse-1: Display Line and Column Mode Menu"))

(defcustom mode-line-percent-position `(,(mode-line-%-propertize "%p"))
  "Type of \"percentage offset\" of window through buffer to display
This option specifies the type of offset displayed in
`mode-line-position',
a component of the default `mode-line-format'."
  :type `(radio
          (const :tag "nil:  No offset is displayed" nil)
          (const :tag "\"%o\": Proportion of \"travel\" of the window through the buffer"
                 (,(mode-line-%-propertize "%o")))
          (const :tag "\"%p\": Offset of top of window through buffer"
                 (,(mode-line-%-propertize "%p")))
          (const :tag "\"%P\": Offset of bottom of window through buffer"
                 (,(mode-line-%-propertize "%P")))
          (const :tag "\"%q\": Offsets of both top and bottom of window"
                 (,(mode-line-%-propertize "%q"))))
  :version "26.1"
  :group 'mode-line)
(put 'mode-line-percent-position 'risky-local-variable t)


> > After some debugging, I discover that defcustom strips those text
> > properties off the strings.  It does this by using `purecopy' on these
> > strings (in the function `custom-declare-variable').

> But AFAICT custom-declare-variable only applies purecopy to its
> `default` argument which is supposed to be an expression (which
> evaluates to the intended value) rather than a value.  So if you use the
> (propertize "..." ...)  form, the properties should not be stripped.

> Or am I missing something?

purecopy is applied to the entire type: argument.

>         Stefan

-- 
Alan Mackenzie (Nuremberg, Germany).



  reply	other threads:[~2017-05-19 19:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-19 18:55 Difficulties with `purecopy' discarding text properties from strings Alan Mackenzie
2017-05-19 19:11 ` Daniel Colascione
2017-05-19 20:04   ` Alan Mackenzie
2017-05-19 19:22 ` Stefan Monnier
2017-05-19 19:54   ` Alan Mackenzie [this message]
2017-05-19 21:04     ` Eli Zaretskii
2017-05-20 10:29       ` Alan Mackenzie
2017-05-20  2:58     ` Stefan Monnier
2017-05-20 10:40       ` Alan Mackenzie

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=20170519195458.GB5330@acm.fritz.box \
    --to=acm@muc.de \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.