unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Problem with tooltip frame parameters
@ 2006-03-10 14:15 David PONCE
  0 siblings, 0 replies; 3+ messages in thread
From: David PONCE @ 2006-03-10 14:15 UTC (permalink / raw)
  Cc: david, emacs-devel

Hi All,

I recently noticed that customization of `tooltip-frame-parameters' no
more works.  I found that, after `x-show-tip' is called in
`tooltip-show', all the `tooltip-frame-parameters' symbols are changed
to nil!

It is easy to reproduce:

emacs -Q

In the *scratch* buffer eval:

tooltip-frame-parameters

==> ((name . "tooltip") (internal-border-width . 2) (border-width . 1))

Move the mouse on the tool bar to show a tooltip.  Then eval again:

tooltip-frame-parameters

==> ((nil . "tooltip") (nil . 2) (nil . 1))

Do others observe that?  If so, I wonder if that could be a side
effect of a recent change in the function `x_get_arg' in frame.c that
introduced this code:

  tem = Fassq (param, alist);

  if (!NILP (tem))
    {
      /* If we find this parm in ALIST, clear it out
	 so that it won't be "left over" at the end.  */
#ifndef WINDOWSNT /* w32fns.c has not yet been changed to cope with this.  */
      Lisp_Object tail;
      XSETCAR (tem, Qnil);
      /* In case the parameter appears more than once in the alist,
	 clear it out.  */
      for (tail = alist; CONSP (tail); tail = XCDR (tail))
	if (CONSP (XCAR (tail))
	    && EQ (XCAR (XCAR (tail)), param))
	  XSETCAR (XCAR (tail), Qnil);
#endif
    }

Particularly, the XSETCAR (tem, Qnil); statement might be the culprit.

Sincerely,
David

In GNU Emacs 22.0.50.1 (i686-pc-linux-gnu, GTK+ Version 2.6.10)
 of 2006-03-10 on localhost
X server distributor `The X.Org Foundation', version 11.0.60802000
configured using `configure '--with-gtk''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8
  default-enable-multibyte-characters: t

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Problem with tooltip frame parameters
@ 2006-03-10 15:31 David PONCE
  2006-03-11 15:46 ` Richard Stallman
  0 siblings, 1 reply; 3+ messages in thread
From: David PONCE @ 2006-03-10 15:31 UTC (permalink / raw)
  Cc: david, emacs-devel

Just to say that I solved the problem with the patch below, inspired
from what `x_create_frame' already does.

David

2006-03-10  David Ponce  <david@dponce.com>

	* xfns.c (x_create_tip_frame): Preserve received parms.

Index: src/xfns.c
===================================================================
RCS file: /sources/emacs/emacs/src/xfns.c,v
retrieving revision 1.661
diff -u -r1.661 xfns.c
--- src/xfns.c	27 Feb 2006 03:35:31 -0000	1.661
+++ src/xfns.c	10 Mar 2006 15:24:35 -0000
@@ -4637,6 +4637,7 @@
 
   check_x ();
 
+  parms = Fcopy_alist (parms);
 
 #ifdef MULTI_KBOARD
   kb = dpyinfo->kboard;

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Problem with tooltip frame parameters
  2006-03-10 15:31 David PONCE
@ 2006-03-11 15:46 ` Richard Stallman
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Stallman @ 2006-03-11 15:46 UTC (permalink / raw)
  Cc: emacs-pretest-bug, david, emacs-devel

Thanks for fixing it.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-03-11 15:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-03-10 14:15 Problem with tooltip frame parameters David PONCE
  -- strict thread matches above, loose matches on Subject: below --
2006-03-10 15:31 David PONCE
2006-03-11 15:46 ` Richard Stallman

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).