all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: 14939@debbugs.gnu.org
Subject: bug#14939: 24.3.50; `make-variable-frame-local' deprecation - alternative?
Date: Tue, 23 Jul 2013 08:51:17 -0700 (PDT)	[thread overview]
Message-ID: <9c76d260-9793-4ed4-a3b5-fc9aca408034@default> (raw)

It is not clear what to use instead of `make-variable-frame-local'.  The
doc string says to "explicitly check for a frame parameter instead".

What do you mean "check" for the frame parameter?  When?

I have code that adds a variable with a nil value as a frame parameter
to a particular frame, like so:

(make-variable-frame-local 'the-variable)
(modify-frame-parameters the-frame '((the-variable)))

That still works (the function is deprecated but presumably not
desupported), but apparently I should be doing something different,
in order not to use `make-variable-frame-local'.  What should I be doing
differently?  If I remove the call to `make-variable-frame-local' then
the code no longer works - the frame parameter value is not used as
the variable value in code that tests the variable value.

Is each piece of code that uses the value of the variable supposed to
check the selected frame to see if it has a parameter, and if so, to use
that frame parameter value instead of the variable value?  That would be
ridiculously heavy-handed.

Please advise.  Is this just a problem of unclear doc (it does not
reallyh tell you what to do in place of using
`make-variable-frame-local')?  Or is the deprecation of this function
misguided, because there is no good replacement for it?



In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-07-14 on ODIEONE
Bzr revision: 113423 lekktu@gmail.com-20130715004922-i67tg2ois14h3fpm
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/Devel/emacs/binary --enable-checking=yes,glyphs
 CFLAGS='-O0 -g3' CPPFLAGS='-Ic:/Devel/emacs/include'
 LDFLAGS='-Lc:/Devel/emacs/lib''





             reply	other threads:[~2013-07-23 15:51 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-23 15:51 Drew Adams [this message]
2013-07-23 17:04 ` bug#14939: 24.3.50; `make-variable-frame-local' deprecation - alternative? Juanma Barranquero
2013-07-23 18:15   ` Drew Adams
2013-07-23 19:56     ` Stefan Monnier
2013-07-23 20:12       ` Drew Adams
2013-07-24  0:02     ` Juanma Barranquero
2013-07-23 17:16 ` 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

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

  git send-email \
    --in-reply-to=9c76d260-9793-4ed4-a3b5-fc9aca408034@default \
    --to=drew.adams@oracle.com \
    --cc=14939@debbugs.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.