* bug#19962: 25.0.50; window-configuration-change-hook not running on trunk
@ 2015-02-27 18:40 Jorgen Schaefer
2015-02-27 19:57 ` martin rudalics
2015-02-28 18:50 ` martin rudalics
0 siblings, 2 replies; 3+ messages in thread
From: Jorgen Schaefer @ 2015-02-27 18:40 UTC (permalink / raw)
To: 19962
Hi!
With recent Emacs versions from trunk, it seems
`window-configuration-change-hook' is not run anymore.
Reproduction:
(setq window-configuration-change-hook
'((lambda ()
(message "Change!"))))
C-x 2 => does not show "Change!"
In Emacs 24.4, this still happened.
In GNU Emacs 25.0.50.1 (x86_64-unknown-linux-gnu)
of 2015-02-21 on loki.jorgenschaefer.de
Repository revision: 9074a684990600abd9dfad0477c7cd1d2f339ed3
System Description: Debian GNU/Linux 7.8 (wheezy)
Same behavior in b2a590d4e3dc692a97c1b53e015b945d84b4b4c7 from
2015-02-27.
Configured using:
`configure --without-x'
Configured features:
SOUND NOTIFY LIBXML2 ZLIB
Important settings:
value of $LC_ALL:
value of $LC_COLLATE: de_DE.UTF-8
value of $LC_CTYPE: de_DE.UTF-8
value of $LC_MESSAGES: POSIX
value of $LC_MONETARY: POSIX
value of $LC_NUMERIC: POSIX
value of $LC_TIME: POSIX
value of $LANG: POSIX
locale-coding-system: utf-8-unix
^ permalink raw reply [flat|nested] 3+ messages in thread* bug#19962: 25.0.50; window-configuration-change-hook not running on trunk
2015-02-27 18:40 bug#19962: 25.0.50; window-configuration-change-hook not running on trunk Jorgen Schaefer
@ 2015-02-27 19:57 ` martin rudalics
2015-02-28 18:50 ` martin rudalics
1 sibling, 0 replies; 3+ messages in thread
From: martin rudalics @ 2015-02-27 19:57 UTC (permalink / raw)
To: Jorgen Schaefer, 19962
> With recent Emacs versions from trunk, it seems
> `window-configuration-change-hook' is not run anymore.
>
> Reproduction:
>
> (setq window-configuration-change-hook
> '((lambda ()
> (message "Change!"))))
>
> C-x 2 => does not show "Change!"
>
> In Emacs 24.4, this still happened.
>
> In GNU Emacs 25.0.50.1 (x86_64-unknown-linux-gnu)
> of 2015-02-21 on loki.jorgenschaefer.de
> Repository revision: 9074a684990600abd9dfad0477c7cd1d2f339ed3
> System Description: Debian GNU/Linux 7.8 (wheezy)
>
> Same behavior in b2a590d4e3dc692a97c1b53e015b945d84b4b4c7 from
> 2015-02-27.
>
>
> Configured using:
> `configure --without-x'
Bad. Does the following patch fix it?
diff --git a/src/window.c b/src/window.c
index d59616d..efbe70f 100644
--- a/src/window.c
+++ b/src/window.c
@@ -3328,7 +3328,9 @@ run_window_configuration_change_hook (struct frame *f)
XSETFRAME (frame, f);
if (NILP (Vrun_hooks)
+#ifdef HAVE_WINDOW_SYSTEM
|| !(f->can_x_set_window_size)
+#endif
|| !(f->after_make_frame))
return;
martin
^ permalink raw reply related [flat|nested] 3+ messages in thread* bug#19962: 25.0.50; window-configuration-change-hook not running on trunk
2015-02-27 18:40 bug#19962: 25.0.50; window-configuration-change-hook not running on trunk Jorgen Schaefer
2015-02-27 19:57 ` martin rudalics
@ 2015-02-28 18:50 ` martin rudalics
1 sibling, 0 replies; 3+ messages in thread
From: martin rudalics @ 2015-02-28 18:50 UTC (permalink / raw)
To: Jorgen Schaefer, 19962
> With recent Emacs versions from trunk, it seems
> `window-configuration-change-hook' is not run anymore.
>
> Reproduction:
>
> (setq window-configuration-change-hook
> '((lambda ()
> (message "Change!"))))
>
> C-x 2 => does not show "Change!"
Hopefully fixed now on trunk as
e605af8..5dd2272 master -> master
Please try again.
Thanks, martin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-02-28 18:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-27 18:40 bug#19962: 25.0.50; window-configuration-change-hook not running on trunk Jorgen Schaefer
2015-02-27 19:57 ` martin rudalics
2015-02-28 18:50 ` martin rudalics
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).