]> asedeno.scripts.mit.edu Git - PuTTY.git/commitdiff
Work around a weird bug in gdk_draw_text_wc().
authorSimon Tatham <anakin@pobox.com>
Fri, 3 Jan 2003 17:52:51 +0000 (17:52 +0000)
committerSimon Tatham <anakin@pobox.com>
Fri, 3 Jan 2003 17:52:51 +0000 (17:52 +0000)
[originally from svn r2437]

unix/pterm.c

index dcae55c5408017faa8d81cd2bd69e23c257dde61..b7e2b93482381229f73631ab619789ed9cfe55b9 100644 (file)
@@ -1590,7 +1590,22 @@ void do_text_internal(Context ctx, int x, int y, char *text, int len,
             * nothing at all; such is life.
             */
        } else if (inst->fontinfo[fontid].is_wide) {
-           gwcs = smalloc(sizeof(GdkWChar) * (len+1));
+           /*
+            * At least one version of gdk_draw_text_wc() has a
+            * weird bug whereby it reads `len' elements of the
+            * input string, but only draws `len/2'. Hence I'm
+            * going to make its input array twice as long as it
+            * theoretically needs to be, and pass in twice the
+            * actual number of characters. If a fixed gdk actually
+            * takes the doubled length seriously, then (a) the
+            * array will stand scrutiny up to the full length, (b)
+            * the spare elements of the array are full of zeroes
+            * which will probably be an empty glyph in the font,
+            * and (c) the clip rectangle should prevent it causing
+            * trouble anyway.
+            */
+           gwcs = smalloc(sizeof(GdkWChar) * (len*2+1));
+           memset(gwcs, 0, sizeof(GdkWChar) * (len*2+1));
            /*
             * FIXME: when we have a wide-char equivalent of
             * from_unicode, use it instead of this.