*** This bug has been marked as a duplicate of 135201 ***

i don't really know if it is the same problem, the windows appear even if i only
hit F1 once.

Oh, that is different then.  I was too trigger happy is marking duplicate.

Confirming. I've just had that happen to me.

I think I've tracked it down to there being on yelp in /usr/bin and another one
in /opt/gnome-2.10/bin, with the latter being first in $PATH. Starting
/opt/gnome-2.10/bin/yelp activates the bonobo service, which ends up being
/usr/bin/yelp.

Happens also with only one yelp installed  (in /opt/gnome-2.10).

I've tracked it down to the following change to yelp-main.c:

revision 1.37
date: 2005/02/23 19:07:05;  author: pborelli;  state: Exp;  lines: +5 -5
2005-02-23  Paolo Borelli  <pborelli@katamail.com>

        * src/yelp-main.c (main): do not leak display_name.

Diff:
-       bonobo_activation_set_activation_env_value
-               ("DISPLAY",
-                gdk_display_get_name (gdk_display_get_default ()) );
+       display_name = gdk_display_get_name (gdk_display_get_default ());
+       bonobo_activation_set_activation_env_value ("DISPLAY", display_name);
+       g_free (display_name);

However, gdk_display_get_name is CONST! The documentation says "a string
representing the display name. This string is owned by GDK and should not be
modified or freed."

May I revert this change in cvs?

crap. sorry about it.

Valgrind marked the string as "definately lost", so I fixed the leak without
thinking much. It may still be worth investigating why valgrind thinks the
string is leaked, maybe a gtk bug.

Probably because gtk never closes displays (bug 85715).

Ok, I now see what's going on:

the gedit code was originally cut and pasted from yelp, but was then modified to
use gdk_screen_make_display_name (gdk_screen_get_default ()) instead of
gdk_display_get_name (gdk_display_get_deafult ()).
So the leak fix in gedit was correct, while my yelp patch was totally bogus.

sorry again about all this mess.

This was reverted. Marking fixed.