GNU bug report logs - #15226
24.3; frame-background-mode docstring on not setq

Previous Next

Package: emacs;

Reported by: Kevin Ryde <user42 <at> zip.com.au>

Date: Sat, 31 Aug 2013 00:27:01 UTC

Severity: minor

Found in version 24.3

Fixed in version 24.4

Done: Glenn Morris <rgm <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Glenn Morris <rgm <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#15226: closed (24.3; frame-background-mode docstring on not setq)
Date: Sun, 01 Sep 2013 01:31:01 +0000
[Message part 1 (text/plain, inline)]
Your message dated Sat, 31 Aug 2013 21:30:06 -0400
with message-id <xqhae5uy5d.fsf <at> fencepost.gnu.org>
and subject line Re: bug#15226: 24.3; frame-background-mode docstring on not setq
has caused the debbugs.gnu.org bug report #15226,
regarding 24.3; frame-background-mode docstring on not setq
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
15226: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15226
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Kevin Ryde <user42 <at> zip.com.au>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3; frame-background-mode docstring on not setq
Date: Sat, 31 Aug 2013 10:26:01 +1000
[Message part 3 (text/plain, inline)]
The docstring of frame-background-mode says

    Don't set this variable with `setq'; this won't have the expected
    effect.

but doesn't say what you should in fact do.  Perhaps something like
below.  Unless frame-set-background-mode is meant to be internals ...

2013-08-31  Kevin Ryde  <user42 <at> zip.com.au>

	* frame.el (frame-background-mode): Docstring describe
	frame-set-background-mode after setting and suggest
	custom-set-variables from .emacs.

[frame.el.frame-background-mode.diff (text/x-diff, inline)]
diff -c /down/emacs-bzr/lisp/frame.el.orig /down/emacs-bzr/lisp/frame.el
*** /down/emacs-bzr/lisp/frame.el.orig	2013-08-31 10:01:50.000000000 +1000
--- /down/emacs-bzr/lisp/frame.el	2013-08-31 10:23:16.000000000 +1000
***************
*** 872,879 ****
    "The brightness of the background.
  Set this to the symbol `dark' if your background color is dark,
  `light' if your background is light, or nil (automatic by default)
! if you want Emacs to examine the brightness for you.  Don't set this
! variable with `setq'; this won't have the expected effect."
    :group 'faces
    :set #'(lambda (var value)
  	   (set-default var value)
--- 872,886 ----
    "The brightness of the background.
  Set this to the symbol `dark' if your background color is dark,
  `light' if your background is light, or nil (automatic by default)
! if you want Emacs to examine the brightness for you.
! 
! If this variable is changed with `setq' then call
! `frame-set-background-mode' on each frame to update existing
! faces.  The customize interface updates after setting and that
! can be used from .emacs or similar with for example
! 
!     (custom-set-variables '(frame-background-mode 'dark))"
! 
    :group 'faces
    :set #'(lambda (var value)
  	   (set-default var value)

Diff finished.  Sat Aug 31 10:23:38 2013
[Message part 5 (text/plain, inline)]


In GNU Emacs 24.3.1 (i486-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
 of 2013-05-29 on blah.blah, modified by Debian
System Description:	Debian GNU/Linux unstable (sid)

Configured using:
 `configure '--build' 'i486-linux-gnu' '--build' 'i486-linux-gnu'
 '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib'
 '--localstatedir=/var/lib' '--infodir=/usr/share/info'
 '--mandir=/usr/share/man' '--with-pop=yes'
 '--enable-locallisppath=/etc/emacs24:/etc/emacs:/usr/local/share/emacs/24.3/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/24.3/site-lisp:/usr/share/emacs/site-lisp'
 '--with-crt-dir=/usr/lib/i386-linux-gnu' '--with-x=yes'
 '--with-x-toolkit=lucid' '--with-toolkit-scroll-bars' '--without-gconf'
 'build_alias=i486-linux-gnu' 'CFLAGS=-g -O2 -fstack-protector
 --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wall'
 'LDFLAGS=-Wl,-z,relro -Wl,-znocombreloc'
 'CPPFLAGS=-D_FORTIFY_SOURCE=2''

Important settings:
  value of $LANG: en_AU
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t
[Message part 6 (message/rfc822, inline)]
From: Glenn Morris <rgm <at> gnu.org>
To: 15226-done <at> debbugs.gnu.org
Subject: Re: bug#15226: 24.3; frame-background-mode docstring on not setq
Date: Sat, 31 Aug 2013 21:30:06 -0400
Version: 24.4

Fixed.


This bug report was last modified 11 years and 261 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.