First draft of Unicode support in pterm. It's pretty complete: it
[u/mdw/putty] / unix / pterm.c
index 2346176..9c86b32 100644 (file)
@@ -9,6 +9,7 @@
 #include <assert.h>
 #include <stdlib.h>
 #include <string.h>
+#include <signal.h>
 #include <stdio.h>
 #include <time.h>
 #include <errno.h>
 #include <unistd.h>
 #include <sys/types.h>
 #include <sys/wait.h>
-#include <X11/Xlib.h>
-#include <X11/Xutil.h>
 #include <gtk/gtk.h>
 #include <gdk/gdkkeysyms.h>
+#include <gdk/gdkx.h>
+#include <X11/Xlib.h>
+#include <X11/Xutil.h>
 
 #define PUTTY_DO_GLOBALS              /* actually _define_ globals */
+
 #include "putty.h"
 #include "terminal.h"
 
@@ -37,19 +40,24 @@ struct gui_data {
     GtkAdjustment *sbar_adjust;
     GdkPixmap *pixmap;
     GdkFont *fonts[2];                 /* normal and bold (for now!) */
+    struct {
+       int charset;
+       int is_wide;
+    } fontinfo[2];
     GdkCursor *rawcursor, *textcursor, *blankcursor, *currcursor;
     GdkColor cols[NCOLOURS];
     GdkColormap *colmap;
     wchar_t *pastein_data;
     int pastein_data_len;
-    char *pasteout_data;
-    int pasteout_data_len;
+    char *pasteout_data, *pasteout_data_utf8;
+    int pasteout_data_len, pasteout_data_utf8_len;
     int font_width, font_height;
     int ignore_sbar;
     int mouseptr_visible;
     guint term_paste_idle_id;
-    GdkAtom compound_text_atom;
+    GdkAtom compound_text_atom, utf8_string_atom;
     int alt_keycode;
+    int alt_digits;
     char wintitle[sizeof(((Config *)0)->wintitle)];
     char icontitle[sizeof(((Config *)0)->wintitle)];
     int master_fd, master_func_id, exited;
@@ -57,12 +65,23 @@ struct gui_data {
     Backend *back;
     void *backhandle;
     Terminal *term;
+    void *logctx;
+};
+
+struct draw_ctx {
+    GdkGC *gc;
+    struct gui_data *inst;
 };
 
-static struct gui_data the_inst;
-static struct gui_data *inst = &the_inst;   /* so we always write `inst->' */
 static int send_raw_mouse;
 
+static char *app_name = "pterm";
+
+char *x_get_default(char *key)
+{
+    return XGetDefault(GDK_DISPLAY(), app_name, key);
+}
+
 void ldisc_update(void *frontend, int echo, int edit)
 {
     /*
@@ -73,7 +92,7 @@ void ldisc_update(void *frontend, int echo, int edit)
      */
 }
 
-int askappend(char *filename)
+int askappend(void *frontend, char *filename)
 {
     /*
      * Logging in an xterm-alike is liable to be something you only
@@ -84,7 +103,7 @@ int askappend(char *filename)
     return 2;
 }
 
-void logevent(char *string)
+void logevent(void *frontend, char *string)
 {
     /*
      * This is not a very helpful function: events are logged
@@ -93,8 +112,10 @@ void logevent(char *string)
      */
 }
 
-int font_dimension(int which)         /* 0 for width, 1 for height */
+int font_dimension(void *frontend, int which)/* 0 for width, 1 for height */
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
+
     if (which)
        return inst->font_height;
     else
@@ -110,8 +131,10 @@ int font_dimension(int which)             /* 0 for width, 1 for height */
  * mouse or a means of faking it, and there is no need to switch
  * buttons around at all.
  */
-Mouse_Button translate_button(Mouse_Button button)
+Mouse_Button translate_button(void *frontend, Mouse_Button button)
 {
+    /* struct gui_data *inst = (struct gui_data *)frontend; */
+
     if (button == MBT_LEFT)
        return MBT_SELECT;
     if (button == MBT_MIDDLE)
@@ -125,12 +148,13 @@ Mouse_Button translate_button(Mouse_Button button)
  * Minimise or restore the window in response to a server-side
  * request.
  */
-void set_iconic(int iconic)
+void set_iconic(void *frontend, int iconic)
 {
     /*
      * GTK 1.2 doesn't know how to do this.
      */
 #if GTK_CHECK_VERSION(2,0,0)
+    struct gui_data *inst = (struct gui_data *)frontend;
     if (iconic)
        gtk_window_iconify(GTK_WINDOW(inst->window));
     else
@@ -141,8 +165,9 @@ void set_iconic(int iconic)
 /*
  * Move the window in response to a server-side request.
  */
-void move_window(int x, int y)
+void move_window(void *frontend, int x, int y)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     /*
      * I assume that when the GTK version of this call is available
      * we should use it. Not sure how it differs from the GDK one,
@@ -159,8 +184,9 @@ void move_window(int x, int y)
  * Move the window to the top or bottom of the z-order in response
  * to a server-side request.
  */
-void set_zorder(int top)
+void set_zorder(void *frontend, int top)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     if (top)
        gdk_window_raise(inst->window->window);
     else
@@ -170,8 +196,9 @@ void set_zorder(int top)
 /*
  * Refresh the window in response to a server-side request.
  */
-void refresh_window(void)
+void refresh_window(void *frontend)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     term_invalidate(inst->term);
 }
 
@@ -179,12 +206,13 @@ void refresh_window(void)
  * Maximise or restore the window in response to a server-side
  * request.
  */
-void set_zoomed(int zoomed)
+void set_zoomed(void *frontend, int zoomed)
 {
     /*
      * GTK 1.2 doesn't know how to do this.
      */
 #if GTK_CHECK_VERSION(2,0,0)
+    struct gui_data *inst = (struct gui_data *)frontend;
     if (iconic)
        gtk_window_maximize(GTK_WINDOW(inst->window));
     else
@@ -195,16 +223,18 @@ void set_zoomed(int zoomed)
 /*
  * Report whether the window is iconic, for terminal reports.
  */
-int is_iconic(void)
+int is_iconic(void *frontend)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     return !gdk_window_is_viewable(inst->window->window);
 }
 
 /*
  * Report the window's position, for terminal reports.
  */
-void get_window_pos(int *x, int *y)
+void get_window_pos(void *frontend, int *x, int *y)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     /*
      * I assume that when the GTK version of this call is available
      * we should use it. Not sure how it differs from the GDK one,
@@ -220,8 +250,9 @@ void get_window_pos(int *x, int *y)
 /*
  * Report the window's pixel size, for terminal reports.
  */
-void get_window_pixels(int *x, int *y)
+void get_window_pixels(void *frontend, int *x, int *y)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     /*
      * I assume that when the GTK version of this call is available
      * we should use it. Not sure how it differs from the GDK one,
@@ -237,8 +268,9 @@ void get_window_pixels(int *x, int *y)
 /*
  * Return the window or icon title.
  */
-char *get_window_title(int icon)
+char *get_window_title(void *frontend, int icon)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     return icon ? inst->wintitle : inst->icontitle;
 }
 
@@ -251,7 +283,7 @@ gint delete_window(GtkWidget *widget, GdkEvent *event, gpointer data)
     return FALSE;
 }
 
-void show_mouseptr(int show)
+static void show_mouseptr(struct gui_data *inst, int show)
 {
     if (!cfg.hide_mouseptr)
        show = 1;
@@ -305,7 +337,7 @@ gint configure_area(GtkWidget *widget, GdkEventConfigure *event, gpointer data)
 
 gint expose_area(GtkWidget *widget, GdkEventExpose *event, gpointer data)
 {
-    /* struct gui_data *inst = (struct gui_data *)data; */
+    struct gui_data *inst = (struct gui_data *)data;
 
     /*
      * Pass the exposed rectangle to terminal.c, which will call us
@@ -327,7 +359,7 @@ gint expose_area(GtkWidget *widget, GdkEventExpose *event, gpointer data)
 
 gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
-    /* struct gui_data *inst = (struct gui_data *)data; */
+    struct gui_data *inst = (struct gui_data *)data;
     char output[32];
     int start, end;
 
@@ -337,11 +369,17 @@ gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
     /*
      * If Alt is being released after typing an Alt+numberpad
      * sequence, we should generate the code that was typed.
+     * 
+     * Note that we only do this if more than one key was actually
+     * pressed - I don't think Alt+NumPad4 should be ^D or that
+     * Alt+NumPad3 should be ^C, for example. There's no serious
+     * inconvenience in having to type a zero before a single-digit
+     * character code.
      */
     if (event->type == GDK_KEY_RELEASE &&
        (event->keyval == GDK_Meta_L || event->keyval == GDK_Alt_L ||
         event->keyval == GDK_Meta_R || event->keyval == GDK_Alt_R) &&
-       inst->alt_keycode >= 0) {
+       inst->alt_keycode >= 0 && inst->alt_digits > 1) {
 #ifdef KEY_DEBUGGING
        printf("Alt key up, keycode = %d\n", inst->alt_keycode);
 #endif
@@ -374,6 +412,7 @@ gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
        if ((event->keyval == GDK_Meta_L || event->keyval == GDK_Alt_L ||
             event->keyval == GDK_Meta_R || event->keyval == GDK_Alt_R)) {
            inst->alt_keycode = -1;
+            inst->alt_digits = 0;
            goto done;                 /* this generates nothing else */
        }
 
@@ -408,6 +447,7 @@ gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
                    inst->alt_keycode = digit;   /* one-digit code */
                else
                    inst->alt_keycode = inst->alt_keycode * 10 + digit;
+                inst->alt_digits++;
 #ifdef KEY_DEBUGGING
                printf(" gives new code %d\n", inst->alt_keycode);
 #endif
@@ -433,7 +473,7 @@ gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
         * Neither does Shift-Ins.
         */
        if (event->keyval == GDK_Insert && (event->state & GDK_SHIFT_MASK)) {
-           request_paste();
+           request_paste(inst);
            return TRUE;
        }
 
@@ -796,8 +836,20 @@ gint key_event(GtkWidget *widget, GdkEventKey *event, gpointer data)
        printf("\n");
 #endif
 
-       ldisc_send(inst->ldisc, output+start, end-start, 1);
-       show_mouseptr(0);
+       /*
+        * The stuff we've just generated is assumed to be
+        * ISO-8859-1! This sounds insane, but `man XLookupString'
+        * agrees: strings of this type returned from the X server
+        * are hardcoded to 8859-1. Strictly speaking we should be
+        * doing this using some sort of GtkIMContext, which (if
+        * we're lucky) would give us our data directly in Unicode;
+        * but that's not supported in GTK 1.2 as far as I can
+        * tell, and it's poorly documented even in 2.0, so it'll
+        * have to wait.
+        */
+       lpage_send(inst->ldisc, CS_ISO8859_1, output+start, end-start, 1);
+
+       show_mouseptr(inst, 0);
        term_seen_key_event(inst->term);
        term_out(inst->term);
     }
@@ -810,7 +862,7 @@ gint button_event(GtkWidget *widget, GdkEventButton *event, gpointer data)
     struct gui_data *inst = (struct gui_data *)data;
     int shift, ctrl, alt, x, y, button, act;
 
-    show_mouseptr(1);
+    show_mouseptr(inst, 1);
 
     if (event->button == 4 && event->type == GDK_BUTTON_PRESS) {
        term_scroll(inst->term, 0, -5);
@@ -858,7 +910,7 @@ gint motion_event(GtkWidget *widget, GdkEventMotion *event, gpointer data)
     struct gui_data *inst = (struct gui_data *)data;
     int shift, ctrl, alt, x, y, button;
 
-    show_mouseptr(1);
+    show_mouseptr(inst, 1);
 
     shift = event->state & GDK_SHIFT_MASK;
     ctrl = event->state & GDK_CONTROL_MASK;
@@ -993,29 +1045,32 @@ void destroy(GtkWidget *widget, gpointer data)
 
 gint focus_event(GtkWidget *widget, GdkEventFocus *event, gpointer data)
 {
+    struct gui_data *inst = (struct gui_data *)data;
     inst->term->has_focus = event->in;
     term_out(inst->term);
     term_update(inst->term);
-    show_mouseptr(1);
+    show_mouseptr(inst, 1);
     return FALSE;
 }
 
 /*
  * set or clear the "raw mouse message" mode
  */
-void set_raw_mouse_mode(int activate)
+void set_raw_mouse_mode(void *frontend, int activate)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     activate = activate && !cfg.no_mouse_rep;
     send_raw_mouse = activate;
     if (send_raw_mouse)
        inst->currcursor = inst->rawcursor;
     else
        inst->currcursor = inst->textcursor;
-    show_mouseptr(inst->mouseptr_visible);
+    show_mouseptr(inst, inst->mouseptr_visible);
 }
 
-void request_resize(int w, int h)
+void request_resize(void *frontend, int w, int h)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     int large_x, large_y;
     int offset_x, offset_y;
     int area_x, area_y;
@@ -1041,7 +1096,7 @@ void request_resize(int w, int h)
      * bogus size request which guarantees to be bigger than the
      * current size of the drawing area.
      */
-    get_window_pixels(&large_x, &large_y);
+    get_window_pixels(inst, &large_x, &large_y);
     large_x += 32;
     large_y += 32;
 
@@ -1080,7 +1135,7 @@ void request_resize(int w, int h)
 #endif
 }
 
-void real_palette_set(int n, int r, int g, int b)
+static void real_palette_set(struct gui_data *inst, int n, int r, int g, int b)
 {
     gboolean success[1];
 
@@ -1095,7 +1150,7 @@ void real_palette_set(int n, int r, int g, int b)
                n, r, g, b);
 }
 
-void set_window_background(void)
+void set_window_background(struct gui_data *inst)
 {
     if (inst->area && inst->area->window)
        gdk_window_set_background(inst->area->window, &inst->cols[18]);
@@ -1103,22 +1158,24 @@ void set_window_background(void)
        gdk_window_set_background(inst->window->window, &inst->cols[18]);
 }
 
-void palette_set(int n, int r, int g, int b)
+void palette_set(void *frontend, int n, int r, int g, int b)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     static const int first[21] = {
        0, 2, 4, 6, 8, 10, 12, 14,
        1, 3, 5, 7, 9, 11, 13, 15,
        16, 17, 18, 20, 22
     };
-    real_palette_set(first[n], r, g, b);
+    real_palette_set(inst, first[n], r, g, b);
     if (first[n] >= 18)
-       real_palette_set(first[n] + 1, r, g, b);
+       real_palette_set(inst, first[n] + 1, r, g, b);
     if (first[n] == 18)
-       set_window_background();
+       set_window_background(inst);
 }
 
-void palette_reset(void)
+void palette_reset(void *frontend)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     /* This maps colour indices in cfg to those used in inst->cols. */
     static const int ww[] = {
        6, 7, 8, 9, 10, 11, 12, 13,
@@ -1150,16 +1207,34 @@ void palette_reset(void)
                    i, cfg.colours[i][0], cfg.colours[i][1], cfg.colours[i][2]);
     }
 
-    set_window_background();
+    set_window_background(inst);
 }
 
-void write_clip(wchar_t * data, int len, int must_deselect)
+void write_clip(void *frontend, wchar_t * data, int len, int must_deselect)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     if (inst->pasteout_data)
        sfree(inst->pasteout_data);
+    if (inst->pasteout_data_utf8)
+       sfree(inst->pasteout_data_utf8);
+
+    inst->pasteout_data_utf8 = smalloc(len*6);
+    inst->pasteout_data_utf8_len = len*6;
+    {
+       wchar_t *tmp = data;
+       int tmplen = len;
+       inst->pasteout_data_utf8_len =
+           charset_from_unicode(&tmp, &tmplen, inst->pasteout_data_utf8,
+                                inst->pasteout_data_utf8_len,
+                                CS_UTF8, NULL, NULL, 0);
+       inst->pasteout_data_utf8 =
+           srealloc(inst->pasteout_data_utf8, inst->pasteout_data_utf8_len);
+    }
+
     inst->pasteout_data = smalloc(len);
     inst->pasteout_data_len = len;
-    wc_to_mb(0, 0, data, len, inst->pasteout_data, inst->pasteout_data_len,
+    wc_to_mb(line_codepage, 0, data, len,
+            inst->pasteout_data, inst->pasteout_data_len,
             NULL, NULL);
 
     if (gtk_selection_owner_set(inst->area, GDK_SELECTION_PRIMARY,
@@ -1168,45 +1243,83 @@ void write_clip(wchar_t * data, int len, int must_deselect)
                                 GDK_SELECTION_TYPE_STRING, 1);
        gtk_selection_add_target(inst->area, GDK_SELECTION_PRIMARY,
                                 inst->compound_text_atom, 1);
+       gtk_selection_add_target(inst->area, GDK_SELECTION_PRIMARY,
+                                inst->utf8_string_atom, 1);
     }
 }
 
 void selection_get(GtkWidget *widget, GtkSelectionData *seldata,
                   guint info, guint time_stamp, gpointer data)
 {
-    gtk_selection_data_set(seldata, GDK_SELECTION_TYPE_STRING, 8,
-                          inst->pasteout_data, inst->pasteout_data_len);
+    struct gui_data *inst = (struct gui_data *)data;
+    if (seldata->target == inst->utf8_string_atom)
+       gtk_selection_data_set(seldata, seldata->target, 8,
+                              inst->pasteout_data_utf8,
+                              inst->pasteout_data_utf8_len);
+    else
+       gtk_selection_data_set(seldata, seldata->target, 8,
+                              inst->pasteout_data, inst->pasteout_data_len);
 }
 
 gint selection_clear(GtkWidget *widget, GdkEventSelection *seldata,
                     gpointer data)
 {
+    struct gui_data *inst = (struct gui_data *)data;
     term_deselect(inst->term);
     if (inst->pasteout_data)
        sfree(inst->pasteout_data);
+    if (inst->pasteout_data_utf8)
+       sfree(inst->pasteout_data_utf8);
     inst->pasteout_data = NULL;
     inst->pasteout_data_len = 0;
+    inst->pasteout_data_utf8 = NULL;
+    inst->pasteout_data_utf8_len = 0;
     return TRUE;
 }
 
-void request_paste(void)
+void request_paste(void *frontend)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     /*
      * In Unix, pasting is asynchronous: all we can do at the
      * moment is to call gtk_selection_convert(), and when the data
      * comes back _then_ we can call term_do_paste().
      */
+
+    /*
+     * First we attempt to retrieve the selection as a UTF-8 string
+     * (which we will convert to the correct code page before
+     * sending to the session, of course). If that fails,
+     * selection_received() will be informed and will fall back to
+     * an ordinary string.
+     */
     gtk_selection_convert(inst->area, GDK_SELECTION_PRIMARY,
-                         GDK_SELECTION_TYPE_STRING, GDK_CURRENT_TIME);
+                         inst->utf8_string_atom, GDK_CURRENT_TIME);
 }
 
 gint idle_paste_func(gpointer data);   /* forward ref */
 
 void selection_received(GtkWidget *widget, GtkSelectionData *seldata,
-                       gpointer data)
+                       guint time, gpointer data)
 {
+    struct gui_data *inst = (struct gui_data *)data;
+
+    if (seldata->target == inst->utf8_string_atom && seldata->length <= 0) {
+       /*
+        * Failed to get a UTF-8 selection string. Try an ordinary
+        * string.
+        */
+       gtk_selection_convert(inst->area, GDK_SELECTION_PRIMARY,
+                             GDK_SELECTION_TYPE_STRING, GDK_CURRENT_TIME);
+       return;
+    }
+
+    /*
+     * Any other failure should just go foom.
+     */
     if (seldata->length <= 0 ||
-       seldata->type != GDK_SELECTION_TYPE_STRING)
+       (seldata->type != GDK_SELECTION_TYPE_STRING &&
+        seldata->type != inst->utf8_string_atom))
        return;                        /* Nothing happens. */
 
     if (inst->pastein_data)
@@ -1214,8 +1327,11 @@ void selection_received(GtkWidget *widget, GtkSelectionData *seldata,
 
     inst->pastein_data = smalloc(seldata->length * sizeof(wchar_t));
     inst->pastein_data_len = seldata->length;
-    mb_to_wc(0, 0, seldata->data, seldata->length,
-            inst->pastein_data, inst->pastein_data_len);
+    inst->pastein_data_len =
+       mb_to_wc((seldata->type == inst->utf8_string_atom ?
+                 CS_UTF8 : line_codepage),
+                0, seldata->data, seldata->length,
+                inst->pastein_data, inst->pastein_data_len);
 
     term_do_paste(inst->term);
 
@@ -1236,30 +1352,35 @@ gint idle_paste_func(gpointer data)
 }
 
 
-void get_clip(wchar_t ** p, int *len)
+void get_clip(void *frontend, wchar_t ** p, int *len)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
+
     if (p) {
        *p = inst->pastein_data;
        *len = inst->pastein_data_len;
     }
 }
 
-void set_title(char *title)
+void set_title(void *frontend, char *title)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     strncpy(inst->wintitle, title, lenof(inst->wintitle));
     inst->wintitle[lenof(inst->wintitle)-1] = '\0';
     gtk_window_set_title(GTK_WINDOW(inst->window), inst->wintitle);
 }
 
-void set_icon(char *title)
+void set_icon(void *frontend, char *title)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     strncpy(inst->icontitle, title, lenof(inst->icontitle));
     inst->icontitle[lenof(inst->icontitle)-1] = '\0';
     gdk_window_set_icon_name(inst->window->window, inst->icontitle);
 }
 
-void set_sbar(int total, int start, int page)
+void set_sbar(void *frontend, int total, int start, int page)
 {
+    struct gui_data *inst = (struct gui_data *)frontend;
     if (!cfg.scrollbar)
        return;
     inst->sbar_adjust->lower = 0;
@@ -1275,25 +1396,34 @@ void set_sbar(int total, int start, int page)
 
 void scrollbar_moved(GtkAdjustment *adj, gpointer data)
 {
+    struct gui_data *inst = (struct gui_data *)data;
+
     if (!cfg.scrollbar)
        return;
     if (!inst->ignore_sbar)
        term_scroll(inst->term, 1, (int)adj->value);
 }
 
-void sys_cursor(int x, int y)
+void sys_cursor(void *frontend, int x, int y)
 {
     /*
      * This is meaningless under X.
      */
 }
 
-void beep(int mode)
+/*
+ * This is still called when mode==BELL_VISUAL, even though the
+ * visual bell is handled entirely within terminal.c, because we
+ * may want to perform additional actions on any kind of bell (for
+ * example, taskbar flashing in Windows).
+ */
+void beep(void *frontend, int mode)
 {
-    gdk_beep();
+    if (mode != BELL_VISUAL)
+       gdk_beep();
 }
 
-int CharWidth(Context ctx, int uc)
+int char_width(Context ctx, int uc)
 {
     /*
      * Under X, any fixed-width font really _is_ fixed-width.
@@ -1303,19 +1433,27 @@ int CharWidth(Context ctx, int uc)
     return 1;
 }
 
-Context get_ctx(void)
+Context get_ctx(void *frontend)
 {
-    GdkGC *gc;
+    struct gui_data *inst = (struct gui_data *)frontend;
+    struct draw_ctx *dctx;
+
     if (!inst->area->window)
        return NULL;
-    gc = gdk_gc_new(inst->area->window);
-    return gc;
+
+    dctx = smalloc(sizeof(*dctx));
+    dctx->inst = inst;
+    dctx->gc = gdk_gc_new(inst->area->window);
+    return dctx;
 }
 
 void free_ctx(Context ctx)
 {
-    GdkGC *gc = (GdkGC *)ctx;
+    struct draw_ctx *dctx = (struct draw_ctx *)ctx;
+    /* struct gui_data *inst = dctx->inst; */
+    GdkGC *gc = dctx->gc;
     gdk_gc_unref(gc);
+    sfree(dctx);
 }
 
 /*
@@ -1327,8 +1465,11 @@ void free_ctx(Context ctx)
 void do_text_internal(Context ctx, int x, int y, char *text, int len,
                      unsigned long attr, int lattr)
 {
-    int nfg, nbg, t, fontid, shadow;
-    GdkGC *gc = (GdkGC *)ctx;
+    struct draw_ctx *dctx = (struct draw_ctx *)ctx;
+    struct gui_data *inst = dctx->inst;
+    GdkGC *gc = dctx->gc;
+
+    int nfg, nbg, t, fontid, shadow, rlen;
 
     /*
      * NYI:
@@ -1365,19 +1506,66 @@ void do_text_internal(Context ctx, int x, int y, char *text, int len,
            return;
        if (x + len*2 > inst->term->cols)
            len = (inst->term->cols-x)/2;    /* trim to LH half */
+       rlen = len * 2;
+    } else
+       rlen = len;
+
+    {
+       GdkRectangle r;
+
+       r.x = x*inst->font_width+cfg.window_border;
+       r.y = y*inst->font_height+cfg.window_border;
+       r.width = rlen*inst->font_width;
+       r.height = inst->font_height;
+       gdk_gc_set_clip_rectangle(gc, &r);
     }
 
     gdk_gc_set_foreground(gc, &inst->cols[nbg]);
     gdk_draw_rectangle(inst->pixmap, gc, 1,
                       x*inst->font_width+cfg.window_border,
                       y*inst->font_height+cfg.window_border,
-                      len*inst->font_width, inst->font_height);
+                      rlen*inst->font_width, inst->font_height);
 
     gdk_gc_set_foreground(gc, &inst->cols[nfg]);
-    gdk_draw_text(inst->pixmap, inst->fonts[fontid], gc,
-                 x*inst->font_width+cfg.window_border,
-                 y*inst->font_height+cfg.window_border+inst->fonts[0]->ascent,
-                 text, len);
+    {
+       GdkWChar *gwcs;
+       gchar *gcs;
+       wchar_t *wcs;
+       int i;
+
+       wcs = smalloc(sizeof(wchar_t) * (len+1));
+       for (i = 0; i < len; i++) {
+           wcs[i] = (wchar_t) ((attr & CSET_MASK) + (text[i] & CHAR_MASK));
+       }
+
+       if (inst->fontinfo[fontid].is_wide) {
+           gwcs = smalloc(sizeof(GdkWChar) * (len+1));
+           /*
+            * FIXME: when we have a wide-char equivalent of
+            * from_unicode, use it instead of this.
+            */
+           for (i = 0; i <= len; i++)
+               gwcs[i] = wcs[i];
+           gdk_draw_text_wc(inst->pixmap, inst->fonts[fontid], gc,
+                            x*inst->font_width+cfg.window_border,
+                            y*inst->font_height+cfg.window_border+inst->fonts[0]->ascent,
+                            gwcs, len*2);
+           sfree(gwcs);
+       } else {
+           wchar_t *wcstmp = wcs;
+           int lentmp = len;
+           gcs = smalloc(sizeof(GdkWChar) * (len+1));
+           charset_from_unicode(&wcstmp, &lentmp, gcs, len,
+                                inst->fontinfo[fontid].charset,
+                                NULL, ".", 1);
+           gdk_draw_text(inst->pixmap, inst->fonts[fontid], gc,
+                         x*inst->font_width+cfg.window_border,
+                         y*inst->font_height+cfg.window_border+inst->fonts[0]->ascent,
+                         gcs, len);
+           sfree(gcs);
+       }
+       sfree(wcs);
+    }
 
     if (shadow) {
        gdk_draw_text(inst->pixmap, inst->fonts[fontid], gc,
@@ -1437,7 +1625,9 @@ void do_text_internal(Context ctx, int x, int y, char *text, int len,
 void do_text(Context ctx, int x, int y, char *text, int len,
             unsigned long attr, int lattr)
 {
-    GdkGC *gc = (GdkGC *)ctx;
+    struct draw_ctx *dctx = (struct draw_ctx *)ctx;
+    struct gui_data *inst = dctx->inst;
+    GdkGC *gc = dctx->gc;
 
     do_text_internal(ctx, x, y, text, len, attr, lattr);
 
@@ -1461,8 +1651,11 @@ void do_text(Context ctx, int x, int y, char *text, int len,
 void do_cursor(Context ctx, int x, int y, char *text, int len,
               unsigned long attr, int lattr)
 {
+    struct draw_ctx *dctx = (struct draw_ctx *)ctx;
+    struct gui_data *inst = dctx->inst;
+    GdkGC *gc = dctx->gc;
+
     int passive;
-    GdkGC *gc = (GdkGC *)ctx;
 
     if (attr & TATTR_PASCURS) {
        attr &= ~TATTR_PASCURS;
@@ -1551,7 +1744,7 @@ void do_cursor(Context ctx, int x, int y, char *text, int len,
                    len*inst->font_width, inst->font_height);
 }
 
-GdkCursor *make_mouse_ptr(int cursor_val)
+GdkCursor *make_mouse_ptr(struct gui_data *inst, int cursor_val)
 {
     /*
      * Truly hideous hack: GTK doesn't allow us to set the mouse
@@ -1654,12 +1847,37 @@ void modalfatalbox(char *p, ...)
     exit(1);
 }
 
-char *get_x_display(void)
+char *get_x_display(void *frontend)
 {
     return gdk_get_display();
 }
 
-char *app_name = "pterm";
+static void help(FILE *fp) {
+    if(fprintf(fp,
+"pterm option summary:\n"
+"\n"
+"  --display DISPLAY         Specify X display to use (note '--')\n"
+"  -name PREFIX              Prefix when looking up resources (default: pterm)\n"
+"  -fn FONT                  Normal text font\n"
+"  -fb FONT                  Bold text font\n"
+"  -geometry WIDTHxHEIGHT    Size of terminal in characters\n"
+"  -sl LINES                 Number of lines of scrollback\n"
+"  -fg COLOUR, -bg COLOUR    Foreground/background colour\n"
+"  -bfg COLOUR, -bbg COLOUR  Foreground/background bold colour\n"
+"  -cfg COLOUR, -bfg COLOUR  Foreground/background cursor colour\n"
+"  -T TITLE                  Window title\n"
+"  -ut, +ut                  Do(default) or do not update utmp\n"
+"  -ls, +ls                  Do(default) or do not make shell a login shell\n"
+"  -sb, +sb                  Do(default) or do not display a scrollbar\n"
+"  -log PATH                 Log all output to a file\n"
+"  -nethack                  Map numeric keypad to hjklyubn direction keys\n"
+"  -xrm RESOURCE-STRING      Set an X resource\n"
+"  -e COMMAND [ARGS...]      Execute command (consumes all remaining args)\n"
+        ) < 0 || fflush(fp) < 0) {
+       perror("output error");
+       exit(1);
+    }
+}
 
 int do_cmdline(int argc, char **argv, int do_everything)
 {
@@ -1667,18 +1885,22 @@ int do_cmdline(int argc, char **argv, int do_everything)
     extern char **pty_argv;           /* declared in pty.c */
 
     /*
-     * Macros to make argument handling easier.
+     * Macros to make argument handling easier. Note that because
+     * they need to call `continue', they cannot be contained in
+     * the usual do {...} while (0) wrapper to make them
+     * syntactically single statements; hence it is not legal to
+     * use one of these macros as an unbraced statement between
+     * `if' and `else'.
      */
-#define EXPECTS_ARG do { \
+#define EXPECTS_ARG { \
     if (--argc <= 0) { \
        err = 1; \
        fprintf(stderr, "pterm: %s expects an argument\n", p); \
+        continue; \
     } else \
        val = *++argv; \
-} while (0)
-#define SECOND_PASS_ONLY do { \
-    if (!do_everything) continue; \
-} while (0)
+}
+#define SECOND_PASS_ONLY { if (!do_everything) continue; }
 
     /*
      * TODO:
@@ -1701,6 +1923,12 @@ int do_cmdline(int argc, char **argv, int do_everything)
            strncpy(cfg.boldfont, val, sizeof(cfg.boldfont));
            cfg.boldfont[sizeof(cfg.boldfont)-1] = '\0';
 
+       } else if (!strcmp(p, "-cs")) {
+           EXPECTS_ARG;
+           SECOND_PASS_ONLY;
+           strncpy(cfg.line_codepage, val, sizeof(cfg.line_codepage));
+           cfg.line_codepage[sizeof(cfg.line_codepage)-1] = '\0';
+
        } else if (!strcmp(p, "-geometry")) {
            int flags, x, y, w, h;
            EXPECTS_ARG;
@@ -1814,6 +2042,10 @@ int do_cmdline(int argc, char **argv, int do_everything)
            EXPECTS_ARG;
            provide_xrm_string(val);
 
+       } else if(!strcmp(p, "-help") || !strcmp(p, "--help")) {
+           help(stdout);
+           exit(0);
+           
        } else {
            err = 1;
            fprintf(stderr, "pterm: unrecognized option '%s'\n", p);
@@ -1823,10 +2055,88 @@ int do_cmdline(int argc, char **argv, int do_everything)
     return err;
 }
 
+static void block_signal(int sig, int block_it) {
+  sigset_t ss;
+
+  sigemptyset(&ss);
+  sigaddset(&ss, sig);
+  if(sigprocmask(block_it ? SIG_BLOCK : SIG_UNBLOCK, &ss, 0) < 0) {
+    perror("sigprocmask");
+    exit(1);
+  }
+}
+
+static void set_font_info(struct gui_data *inst, int fontid)
+{
+    GdkFont *font = inst->fonts[fontid];
+    XFontStruct *xfs = GDK_FONT_XFONT(font);
+    Display *disp = GDK_FONT_XDISPLAY(font);
+    Atom charset_registry, charset_encoding;
+    unsigned long registry_ret, encoding_ret;
+    charset_registry = XInternAtom(disp, "CHARSET_REGISTRY", False);
+    charset_encoding = XInternAtom(disp, "CHARSET_ENCODING", False);
+    inst->fontinfo[fontid].charset = CS_NONE;
+    inst->fontinfo[fontid].is_wide = 0;
+    if (XGetFontProperty(xfs, charset_registry, &registry_ret) &&
+       XGetFontProperty(xfs, charset_encoding, &encoding_ret)) {
+       char *reg, *enc;
+       reg = XGetAtomName(disp, (Atom)registry_ret);
+       enc = XGetAtomName(disp, (Atom)encoding_ret);
+       if (reg && enc) {
+           char *encoding = dupcat(reg, "-", enc, NULL);
+           inst->fontinfo[fontid].charset = charset_from_xenc(encoding);
+           /* FIXME: when libcharset supports wide encodings fix this. */
+           if (!strcasecmp(encoding, "iso10646-1"))
+               inst->fontinfo[fontid].is_wide = 1;
+
+           /*
+            * Hack for X line-drawing characters: if the primary
+            * font is encoded as ISO-8859-anything, and has valid
+            * glyphs in the first 32 char positions, it is assumed
+            * that those glyphs are the VT100 line-drawing
+            * character set.
+            * 
+            * Actually, we'll hack even harder by only checking
+            * position 0x19 (vertical line, VT100 linedrawing
+            * `x'). Then we can check it easily by seeing if the
+            * ascent and descent differ.
+            */
+           if (inst->fontinfo[fontid].charset == CS_ISO8859_1) {
+               int lb, rb, wid, asc, desc;
+               gchar text[2];
+
+               text[1] = '\0';
+               text[0] = '\x12';
+               gdk_string_extents(inst->fonts[fontid], text,
+                                  &lb, &rb, &wid, &asc, &desc);
+               if (asc != desc)
+                   inst->fontinfo[fontid].charset = CS_ISO8859_1_X11;
+           }
+
+           /*
+            * FIXME: this is a hack. Currently fonts with
+            * incomprehensible encodings are dealt with by
+            * pretending they're 8859-1. It's ugly, but it's good
+            * enough to stop things crashing. Should do something
+            * better here.
+            */
+           if (inst->fontinfo[fontid].charset == CS_NONE)
+               inst->fontinfo[fontid].charset = CS_ISO8859_1;
+
+           sfree(encoding);
+       }
+    }
+}
+
 int main(int argc, char **argv)
 {
     extern int pty_master_fd;         /* declared in pty.c */
     extern void pty_pre_init(void);    /* declared in pty.c */
+    struct gui_data *inst;
+
+    /* defer any child exit handling until we're ready to deal with
+     * it */
+    block_signal(SIGCHLD, 1);
 
     pty_pre_init();
 
@@ -1839,15 +2149,18 @@ int main(int argc, char **argv)
        exit(1);
 
     /*
-     * Initialise the whole instance structure to zeroes
+     * Create an instance structure and initialise to zeroes
      */
+    inst = smalloc(sizeof(*inst));
     memset(inst, 0, sizeof(*inst));
+    inst->alt_keycode = -1;            /* this one needs _not_ to be zero */
 
     inst->fonts[0] = gdk_font_load(cfg.font);
     if (!inst->fonts[0]) {
        fprintf(stderr, "pterm: unable to load font \"%s\"\n", cfg.font);
        exit(1);
     }
+    set_font_info(inst, 0);
     if (cfg.boldfont[0]) {
        inst->fonts[1] = gdk_font_load(cfg.boldfont);
        if (!inst->fonts[1]) {
@@ -1855,6 +2168,7 @@ int main(int argc, char **argv)
                    cfg.boldfont);
            exit(1);
        }
+       set_font_info(inst, 1);
     } else
        inst->fonts[1] = NULL;
 
@@ -1862,20 +2176,21 @@ int main(int argc, char **argv)
     inst->font_height = inst->fonts[0]->ascent + inst->fonts[0]->descent;
 
     inst->compound_text_atom = gdk_atom_intern("COMPOUND_TEXT", FALSE);
+    inst->utf8_string_atom = gdk_atom_intern("UTF8_STRING", FALSE);
 
     init_ucs();
 
     inst->window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
 
     if (cfg.wintitle[0])
-       set_title(cfg.wintitle);
+       set_title(inst, cfg.wintitle);
     else
-       set_title("pterm");
+       set_title(inst, "pterm");
 
     /*
      * Set up the colour map.
      */
-    palette_reset();
+    palette_reset(inst);
 
     inst->area = gtk_drawing_area_new();
     gtk_drawing_area_size(GTK_DRAWING_AREA(inst->area),
@@ -1954,25 +2269,29 @@ int main(int argc, char **argv)
     gtk_widget_show(GTK_WIDGET(inst->hbox));
     gtk_widget_show(inst->window);
 
-    set_window_background();
+    set_window_background(inst);
 
-    inst->textcursor = make_mouse_ptr(GDK_XTERM);
-    inst->rawcursor = make_mouse_ptr(GDK_LEFT_PTR);
-    inst->blankcursor = make_mouse_ptr(-1);
-    make_mouse_ptr(-2);                       /* clean up cursor font */
+    inst->textcursor = make_mouse_ptr(inst, GDK_XTERM);
+    inst->rawcursor = make_mouse_ptr(inst, GDK_LEFT_PTR);
+    inst->blankcursor = make_mouse_ptr(inst, -1);
+    make_mouse_ptr(inst, -2);         /* clean up cursor font */
     inst->currcursor = inst->textcursor;
-    show_mouseptr(1);
+    show_mouseptr(inst, 1);
 
-    inst->term = term_init();
+    inst->term = term_init(&cfg, inst);
+    inst->logctx = log_init(inst);
+    term_provide_logctx(inst->term, inst->logctx);
 
     inst->back = &pty_backend;
     inst->back->init((void *)inst->term, &inst->backhandle, NULL, 0, NULL, 0);
+    inst->back->provide_logctx(inst->backhandle, inst->logctx);
 
     term_provide_resize_fn(inst->term, inst->back->size, inst->backhandle);
 
     term_size(inst->term, cfg.height, cfg.width, cfg.savelines);
 
-    inst->ldisc = ldisc_create(inst->term, inst->back, inst->backhandle, inst);
+    inst->ldisc =
+       ldisc_create(&cfg, inst->term, inst->back, inst->backhandle, inst);
     ldisc_send(inst->ldisc, NULL, 0, 0);/* cause ldisc to notice changes */
 
     inst->master_fd = pty_master_fd;
@@ -1980,6 +2299,10 @@ int main(int argc, char **argv)
     inst->master_func_id = gdk_input_add(pty_master_fd, GDK_INPUT_READ,
                                         pty_input_func, inst);
 
+    /* now we're reday to deal with the child exit handler being
+     * called */
+    block_signal(SIGCHLD, 0);
+    
     gtk_main();
 
     return 0;