all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Jan D." <jan.h.d@swipnet.se>
Cc: reiner.steib@gmx.de, emacs-devel@gnu.org
Subject: Re: All platforms fail with Unicode in menus.
Date: Thu, 26 Aug 2004 20:30:12 +0200	[thread overview]
Message-ID: <412E2C34.8070905@swipnet.se> (raw)
In-Reply-To: <x5isb6q47s.fsf@lola.goethe.zz>

David Kastrup wrote:

> The best contender right now is GTK which gets it right, except that
> it completely garbles the menus when garbage collection occurs during
> their creation (the gc-cons-threshold setting at the end of this
> example file more or less ensures that stuff gets garbled).

It seems that the string returned by ENCODE_UTF_8 gets collected during GC,
resulting in passing garbage to the GTK menu code.

I can work around the bug by this patch (the call to inhibit_garbage_collection 
is present in macmenu.c), but I am not sure it is the correct way to fix this:

*** xmenu.c.~1.255.~    2004-01-12 00:15:16.000000000 +0100
--- xmenu.c     2004-08-26 20:18:28.000000000 +0200
***************
*** 1930,1935 ****
--- 1930,1936 ----
         FRAME_MENU_BAR_ITEMS (f) = menu_bar_items (FRAME_MENU_BAR_ITEMS (f));

         items = FRAME_MENU_BAR_ITEMS (f);
+       inhibit_garbage_collection ();

         /* Save the frame's previous menu bar contents data.  */
         if (previous_menu_items_used)

Can someone that knows ENCODE_UTF_8 and garbage collection well comment on this 
patch?

Thanks,

	Jan D.

  parent reply	other threads:[~2004-08-26 18:30 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-25 21:48 All platforms fail with Unicode in menus David Kastrup
2004-08-26  8:13 ` YAMAMOTO Mitsuharu
2004-08-26 18:30 ` Jan D. [this message]
2004-08-28  7:11   ` David Kastrup
2004-08-28  7:42     ` Jan D.
2004-08-28  8:08       ` David Kastrup
2004-08-28  8:24         ` Andreas Schwab
2004-08-28 21:38     ` Stefan
2004-08-28 22:18       ` Jan D.
2004-08-28 22:38         ` Stefan
2004-08-30  7:52   ` YAMAMOTO Mitsuharu
2004-08-30 10:28     ` Jan D.

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=412E2C34.8070905@swipnet.se \
    --to=jan.h.d@swipnet.se \
    --cc=emacs-devel@gnu.org \
    --cc=reiner.steib@gmx.de \
    /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.