Re: Crash: XSetWMIconName, x_set_name_internal, prepare_menu_bars

[Kenichi Handa]

In article <address@hidden>, YAMAMOTO Mitsuharu <address@hidden> writes:

>>>>>> On Sun, 03 Jun 2007 12:52:26 -0400, Richard Stallman <address@hidden> 
>>>>>> said:
> > There needs to be a GCPRO in x_set_name_internal around the calls to
> > x_encode_text, at least to protect the variable encoded_name.  Or
> > else the code needs to be rearranged so encoded_name is not used
> > after the calls to x_encode_text.

> Such GCPROs are not necessary because x_encode_text is called with the
> third arg SELECTIONP == 0.  Actually, there's no SELECTIONP != 0 case
> in the current Emacs code.

I remember that x_encode_text had been used to encode X
selection data, but now it is not in Emacs 22.

But, in emacs-unicode-2, x_encode_text uses
encode_coding_object, and that function does call Lisp for
pre-write-conversion.

---
Kenichi Handa
address@hidden