From: ben <ben@cda61777-01e9-0310-a592-d414129be87e>
Date: Tue, 31 Dec 2002 22:49:03 +0000 (+0000)
Subject: Use the shiny new character-set library to handle conversion from Unicode
X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/commitdiff_plain/801e70ea26818b146f92f0b5af886a1c7790b86e

Use the shiny new character-set library to handle conversion from Unicode
to Mac OS Roman for display if the Unicode Converter isn't around.  Support
for Mac character sets other than Roman (e.g. the variant used by the Apple
VT100 font) is still absent.


git-svn-id: svn://svn.tartarus.org/sgt/putty@2401 cda61777-01e9-0310-a592-d414129be87e
---

diff --git a/Recipe b/Recipe
index 2ec14c7b..12df5d39 100644
--- a/Recipe
+++ b/Recipe
@@ -146,4 +146,4 @@ plink    : [U] uxplink uxcons NONSSH UXSSH be_all logging UXMISC
 
 PuTTY    : [M] terminal wcwidth tree234 misc ldisc ldiscucs
          + logging settings be_none mac macdlg macstore macterm macucs
-         + mac_res.rsrc testback
+         + mac_res.rsrc testback CHARSET
diff --git a/mac/macterm.c b/mac/macterm.c
index 8218f2b1..9a26abed 100644
--- a/mac/macterm.c
+++ b/mac/macterm.c
@@ -1,4 +1,4 @@
-/* $Id: macterm.c,v 1.28 2002/12/31 20:11:38 ben Exp $ */
+/* $Id: macterm.c,v 1.29 2002/12/31 22:49:03 ben Exp $ */
 /*
  * Copyright (c) 1999 Simon Tatham
  * Copyright (c) 1999, 2002 Ben Harris
@@ -60,6 +60,7 @@
 
 #include "macresid.h"
 #include "putty.h"
+#include "charset.h"
 #include "mac.h"
 #include "storage.h"
 #include "terminal.h"
@@ -951,7 +952,10 @@ void do_text(Context ctx, int x, int y, char *text, int len,
     RgnHandle textrgn;
     char mactextbuf[1024];
     UniChar unitextbuf[1024];
+    wchar_t *unitextptr;
     int i;
+    ByteCount iread, olen;
+    OSStatus err;
 
     assert(len <= 1024);
 
@@ -965,23 +969,29 @@ void do_text(Context ctx, int x, int y, char *text, int len,
     if (!RectInRgn(&a.textrect, s->window->visRgn))
 	return;
 
-    if (s->uni_to_font != NULL) {
-	ByteCount iread, olen;
-	OSStatus err;
+    /* Unpack Unicode from the mad format we get passed */
+    for (i = 0; i < len; i++)
+	unitextbuf[i] = (unsigned char)text[i] | (attr & CSET_MASK);
 
-	for (i = 0; i < len; i++)
-	    unitextbuf[i] = (unsigned char)text[i] | (attr & CSET_MASK);
+    if (s->uni_to_font != NULL) {
 	err = ConvertFromUnicodeToText(s->uni_to_font, len * sizeof(UniChar),
 				       unitextbuf, kUnicodeUseFallbacksMask,
 				       0, NULL, NULL, NULL,
 				       1024, &iread, &olen, mactextbuf);
-	if (err == noErr || err == kTECUsedFallbacksStatus)
-	    text = mactextbuf; len = olen;
+	if (err != noErr && err != kTECUsedFallbacksStatus)
+	    /* XXX Should handle this more sensibly */
+	    return;
+    } else {
+	/* XXX this is bogus if wchar_t and UniChar are different sizes. */
+	unitextptr = (wchar_t *)unitextbuf;
+	/* XXX Should choose charset based on script, font etc. */
+	olen = charset_from_unicode(&unitextptr, &len, mactextbuf, 1024,
+				    CS_MAC_ROMAN, NULL, ".", 1);
     }
 
     a.s = s;
-    a.text = text;
-    a.len = len;
+    a.text = mactextbuf;
+    a.len = olen;
     a.attr = attr;
     a.lattr = lattr;
     a.numer.h = a.numer.v = a.denom.h = a.denom.v = 1;