Re: multilingual text in frame

[Kenichi Handa]

In article <address@hidden>, Richard Stallman <address@hidden> writes:
> Are you saying that this problem is a bug or flaw in the window manager
> and that Emacs is doing the right thing?

Yes, I think so.

> If so, how about adding a comment in Emacs near that part of the code
> to explain this.

Ok, I've just commited this change.

---
Ken'ichi HANDA
address@hidden

*** xfns.c.~1.571.~     Mon Jan 27 08:18:59 2003
--- xfns.c      Mon Jan 27 09:58:22 2003
***************
*** 2517,2522 ****
--- 2517,2536 ----
        int bytes, stringp;
        Lisp_Object coding_system;
  
+       /* Note: Encoding strategy
+ 
+          We encode NAME by compound-text and use "COMPOUND-TEXT" in
+          text.encoding.  But, there are non-internationalized window
+          managers which don't support that encoding.  So, if NAME
+          contains only ASCII and 8859-1 characters, encode it by
+          iso-latin-1, and use "STRING" in text.encoding hoping that
+          such window manager at least analize this format correctly,
+          i.e. treat 8-bit bytes as 8859-1 characters.
+ 
+          We may also be able to use "UTF8_STRING" in text.encoding
+          in the feature which can encode all Unicode characters.
+          But, for the moment, there's no way to know that the
+          current window manager supports it or not.  */
        coding_system = Qcompound_text;
        text.value = x_encode_text (name, coding_system, 0, &bytes, &stringp);
        text.encoding = (stringp ? XA_STRING
***************
*** 2530,2535 ****
--- 2544,2550 ----
          }
        else
          {
+           /* See the above comment "Note: Encoding strategy".  */
            icon.value = x_encode_text (f->icon_name, coding_system, 0,
                                        &bytes, &stringp);
            icon.encoding = (stringp ? XA_STRING
***************
*** 2630,2635 ****
--- 2645,2651 ----
        Lisp_Object coding_system;
  
        coding_system = Qcompound_text;
+       /* See the comment "Note: Encoding strategy" in x_set_name.  */
        text.value = x_encode_text (name, coding_system, 0, &bytes, &stringp);
        text.encoding = (stringp ? XA_STRING
                         : FRAME_X_DISPLAY_INFO (f)->Xatom_COMPOUND_TEXT);
***************
*** 2642,2647 ****
--- 2658,2664 ----
          }
        else
          {
+           /* See the comment "Note: Encoding strategy" in x_set_name.  */
            icon.value = x_encode_text (f->icon_name, coding_system, 0,
                                        &bytes, &stringp);
            icon.encoding = (stringp ? XA_STRING