]> Pileus Git - ~andy/gtk/blobdiff - gdk/win32/gdkgc-win32.c
Set graphics_exposures and subwindow_mode, too, even if they aren't
[~andy/gtk] / gdk / win32 / gdkgc-win32.c
index a8a52ba8c52aa7a6c8d46957800c506a73142f5f..1ead0d071217c244f862160ec65d4c052b587f74 100644 (file)
@@ -1,40 +1,41 @@
 /* GDK - The GIMP Drawing Kit
  * Copyright (C) 1995-1997 Peter Mattis, Spencer Kimball and Josh MacDonald
+ * Copyright (C) 1998-2002 Tor Lillqvist
  *
  * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library General Public
+ * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.         See the GNU
- * Library General Public License for more details.
+ * Lesser General Public License for more details.
  *
- * You should have received a copy of the GNU Library General Public
+ * You should have received a copy of the GNU Lesser General Public
  * License along with this library; if not, write to the
  * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
 
 /*
- * Modified by the GTK+ Team and others 1997-1999.  See the AUTHORS
+ * Modified by the GTK+ Team and others 1997-2000.  See the AUTHORS
  * file for a list of people on the GTK+ Team.  See the ChangeLog
  * files for a list of changes.  These files are distributed with
  * GTK+ at ftp://ftp.gtk.org/pub/gtk/. 
  */
 
-#include "config.h"
+#define LINE_ATTRIBUTES (GDK_GC_LINE_WIDTH|GDK_GC_LINE_STYLE| \
+                        GDK_GC_CAP_STYLE|GDK_GC_JOIN_STYLE)
 
 #include <string.h>
 
 #include "gdkgc.h"
 #include "gdkfont.h"
 #include "gdkpixmap.h"
-#include "gdkprivate.h"
-#include "gdkwin32.h"
+#include "gdkregion-generic.h"
+#include "gdkprivate-win32.h"
 
-static void gdk_win32_gc_destroy    (GdkGC           *gc);
 static void gdk_win32_gc_get_values (GdkGC           *gc,
                                     GdkGCValues     *values);
 static void gdk_win32_gc_set_values (GdkGC           *gc,
@@ -42,329 +43,387 @@ static void gdk_win32_gc_set_values (GdkGC           *gc,
                                     GdkGCValuesMask  values_mask);
 static void gdk_win32_gc_set_dashes (GdkGC           *gc,
                                     gint             dash_offset,
-                                    gchar            dash_list[],
+                                    gint8            dash_list[],
                                     gint             n);
 
-static GdkGCClass gdk_win32_gc_class = {
-  gdk_win32_gc_destroy,
-  gdk_win32_gc_get_values,
-  gdk_win32_gc_set_values,
-  gdk_win32_gc_set_dashes
-};
+static void gdk_gc_win32_class_init (GdkGCWin32Class *klass);
+static void gdk_gc_win32_finalize   (GObject         *object);
+
+static gpointer parent_class = NULL;
+
+GType
+_gdk_gc_win32_get_type (void)
+{
+  static GType object_type = 0;
+
+  if (!object_type)
+    {
+      static const GTypeInfo object_info =
+      {
+        sizeof (GdkGCWin32Class),
+        (GBaseInitFunc) NULL,
+        (GBaseFinalizeFunc) NULL,
+        (GClassInitFunc) gdk_gc_win32_class_init,
+        NULL,           /* class_finalize */
+        NULL,           /* class_data */
+        sizeof (GdkGCWin32),
+        0,              /* n_preallocs */
+        (GInstanceInitFunc) NULL,
+      };
+      
+      object_type = g_type_register_static (GDK_TYPE_GC,
+                                            "GdkGCWin32",
+                                            &object_info, 0);
+    }
+  
+  return object_type;
+}
+
+static void
+gdk_gc_win32_class_init (GdkGCWin32Class *klass)
+{
+  GObjectClass *object_class = G_OBJECT_CLASS (klass);
+  GdkGCClass *gc_class = GDK_GC_CLASS (klass);
+  
+  parent_class = g_type_class_peek_parent (klass);
+
+  object_class->finalize = gdk_gc_win32_finalize;
+
+  gc_class->get_values = gdk_win32_gc_get_values;
+  gc_class->set_values = gdk_win32_gc_set_values;
+  gc_class->set_dashes = gdk_win32_gc_set_dashes;
+}
+
+static void
+gdk_gc_win32_finalize (GObject *object)
+{
+  GdkGCWin32 *win32_gc = GDK_GC_WIN32 (object);
+  
+  if (win32_gc->hcliprgn != NULL)
+    DeleteObject (win32_gc->hcliprgn);
+  
+  if (win32_gc->values_mask & GDK_GC_FONT)
+    gdk_font_unref (win32_gc->font);
+  
+  if (win32_gc->values_mask & GDK_GC_TILE)
+    gdk_drawable_unref (win32_gc->tile);
+  
+  if (win32_gc->values_mask & GDK_GC_STIPPLE)
+    gdk_drawable_unref (win32_gc->stipple);
+
+  if (win32_gc->pen_dashes)
+    g_free (win32_gc->pen_dashes);
+  
+  G_OBJECT_CLASS (parent_class)->finalize (object);
+}
 
 static void
 gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
                                    GdkGCValuesMask mask,
-                                   GdkGCWin32Data *data)
+                                   GdkGCWin32     *win32_gc)
 {                                  
   char *s = "";
+  gint sw, sh;
 
-  GDK_NOTE (MISC, g_print ("{"));
+  GDK_NOTE (GC, g_print ("{"));
 
   if (mask & GDK_GC_FOREGROUND)
     {
-      data->foreground = values->foreground.pixel;
-      data->values_mask |= GDK_GC_FOREGROUND;
-      GDK_NOTE (MISC, (g_print ("fg=%.06x", data->foreground),
-                      s = ","));
+      win32_gc->foreground = values->foreground.pixel;
+      win32_gc->values_mask |= GDK_GC_FOREGROUND;
+      GDK_NOTE (GC, (g_print ("fg=%.06lx", win32_gc->foreground),
+                    s = ","));
     }
   
   if (mask & GDK_GC_BACKGROUND)
     {
-      data->background = values->background.pixel;
-      data->values_mask |= GDK_GC_BACKGROUND;
-      GDK_NOTE (MISC, (g_print ("%sbg=%.06x", s, data->background),
-                      s = ","));
+      win32_gc->background = values->background.pixel;
+      win32_gc->values_mask |= GDK_GC_BACKGROUND;
+      GDK_NOTE (GC, (g_print ("%sbg=%.06lx", s, win32_gc->background),
+                    s = ","));
     }
 
   if ((mask & GDK_GC_FONT) && (values->font->type == GDK_FONT_FONT
                               || values->font->type == GDK_FONT_FONTSET))
     {
-      if (data->font != NULL)
-       gdk_font_unref (data->font);
-      data->font = values->font;
-      if (data->font != NULL)
+      if (win32_gc->font != NULL)
+       gdk_font_unref (win32_gc->font);
+      win32_gc->font = values->font;
+      if (win32_gc->font != NULL)
        {
-         gchar *xlfd;
-
-         gdk_font_ref (data->font);
-         data->values_mask |= GDK_GC_FONT;
-         GDK_NOTE (MISC, (xlfd = gdk_font_full_name_get (data->font),
-                          g_print ("%sfont=%s", s, xlfd),
-                          s = ",",
-                          gdk_font_full_name_free (xlfd)));
+         gdk_font_ref (win32_gc->font);
+         win32_gc->values_mask |= GDK_GC_FONT;
+         GDK_NOTE (GC, (g_print ("%sfont=%p", s, win32_gc->font),
+                        s = ","));
        }
       else
        {
-         data->values_mask &= ~GDK_GC_FONT;
-         GDK_NOTE (MISC, (g_print ("%sfont=NULL"),
-                          s = ","));
+         win32_gc->values_mask &= ~GDK_GC_FONT;
+         GDK_NOTE (GC, (g_print ("%sfont=NULL", s),
+                        s = ","));
        }
     }
 
   if (mask & GDK_GC_FUNCTION)
     {
-      GDK_NOTE (MISC, (g_print ("%srop2=", s),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%srop2=", s),
+                    s = ","));
       switch (values->function)
        {
-       case GDK_COPY:
-         data->rop2 = R2_COPYPEN;
-         GDK_NOTE (MISC, g_print ("COPYPEN"));
-         break;
-       case GDK_INVERT:
-         data->rop2 = R2_NOT;
-         GDK_NOTE (MISC, g_print ("NOT"));
-         break;
-       case GDK_XOR:
-         data->rop2 = R2_XORPEN;
-         GDK_NOTE (MISC, g_print ("XORPEN"));
-         break;
-       case GDK_CLEAR:
-         data->rop2 = R2_BLACK;
-         GDK_NOTE (MISC, g_print ("BLACK"));
-         break;
-       case GDK_AND:
-         data->rop2 = R2_MASKPEN;
-         GDK_NOTE (MISC, g_print ("MASKPEN"));
-         break;
-       case GDK_AND_REVERSE:
-         data->rop2 = R2_MASKPENNOT;
-         GDK_NOTE (MISC, g_print ("MASKPENNOT"));
-         break;
-       case GDK_AND_INVERT:
-         data->rop2 = R2_MASKNOTPEN;
-         GDK_NOTE (MISC, g_print ("MASKNOTPEN"));
-         break;
-       case GDK_NOOP:
-         data->rop2 = R2_NOP;
-         GDK_NOTE (MISC, g_print ("NOP"));
-         break;
-       case GDK_OR:
-         data->rop2 = R2_MERGEPEN;
-         GDK_NOTE (MISC, g_print ("MERGEPEN"));
-         break;
-       case GDK_EQUIV:
-         data->rop2 = R2_NOTXORPEN;
-         GDK_NOTE (MISC, g_print ("NOTXORPEN"));
-         break;
-       case GDK_OR_REVERSE:
-         data->rop2 = R2_MERGEPENNOT;
-         GDK_NOTE (MISC, g_print ("MERGEPENNOT"));
-         break;
-       case GDK_COPY_INVERT:
-         data->rop2 = R2_NOTCOPYPEN;
-         GDK_NOTE (MISC, g_print ("NOTCOPYPEN"));
-         break;
-       case GDK_OR_INVERT:
-         data->rop2 = R2_MERGENOTPEN;
-         GDK_NOTE (MISC, g_print ("MERGENOTPEN"));
-         break;
-       case GDK_NAND:
-         data->rop2 = R2_NOTMASKPEN;
-         GDK_NOTE (MISC, g_print ("NOTMASKPEN"));
-         break;
-       case GDK_SET:
-         data->rop2 = R2_WHITE;
-         GDK_NOTE (MISC, g_print ("WHITE"));
-         break;
+#define CASE(x,y) case GDK_##x: win32_gc->rop2 = R2_##y; GDK_NOTE (GC, g_print (#y)); break
+       CASE (COPY, COPYPEN);
+       CASE (INVERT, NOT);
+       CASE (XOR, XORPEN);
+       CASE (CLEAR, BLACK);
+       CASE (AND, MASKPEN);
+       CASE (AND_REVERSE, MASKPENNOT);
+       CASE (AND_INVERT, MASKNOTPEN);
+       CASE (NOOP, NOP);
+       CASE (OR, MERGEPEN);
+       CASE (EQUIV, NOTXORPEN);
+       CASE (OR_REVERSE, MERGEPENNOT);
+       CASE (COPY_INVERT, NOTCOPYPEN);
+       CASE (OR_INVERT, MERGENOTPEN);
+       CASE (NAND, NOTMASKPEN);
+       CASE (NOR, NOTMERGEPEN);
+       CASE (SET, WHITE);
+#undef CASE
        }
-      data->values_mask |= GDK_GC_FUNCTION;
+      win32_gc->values_mask |= GDK_GC_FUNCTION;
     }
 
   if (mask & GDK_GC_FILL)
     {
-      data->fill_style = values->fill;
-      data->values_mask |= GDK_GC_FILL;
-      GDK_NOTE (MISC, (g_print ("%sfill=%d", s, data->fill_style),
-                      s = ","));
+      win32_gc->fill_style = values->fill;
+      win32_gc->values_mask |= GDK_GC_FILL;
+      GDK_NOTE (GC, (g_print ("%sfill=%s", s,
+                             _gdk_win32_fill_style_to_string (win32_gc->fill_style)),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TILE)
     {
-      if (data->tile != NULL)
-       gdk_drawable_unref (data->tile);
-      data->tile = values->tile;
-      if (data->tile != NULL)
+      if (win32_gc->tile != NULL)
+       gdk_drawable_unref (win32_gc->tile);
+      win32_gc->tile = values->tile;
+      if (win32_gc->tile != NULL)
        {
-         gdk_drawable_ref (data->tile);
-         data->values_mask |= GDK_GC_TILE;
-         GDK_NOTE (MISC, (g_print ("%stile=%#x", s,
-                                   GDK_DRAWABLE_XID (data->tile)),
-                          s = ","));
+         gdk_drawable_ref (win32_gc->tile);
+         win32_gc->values_mask |= GDK_GC_TILE;
+         GDK_NOTE (GC,
+                   (g_print ("%stile=%p", s,
+                             GDK_PIXMAP_HBITMAP (win32_gc->tile)),
+                    s = ","));
        }
       else
        {
-         data->values_mask &= ~GDK_GC_TILE;
-         GDK_NOTE (MISC, (g_print ("%stile=NULL", s),
-                          s = ","));
+         win32_gc->values_mask &= ~GDK_GC_TILE;
+         GDK_NOTE (GC, (g_print ("%stile=NULL", s),
+                        s = ","));
        }
     }
 
   if (mask & GDK_GC_STIPPLE)
     {
-      if (data->stipple != NULL)
-       gdk_drawable_unref (data->stipple);
-      data->stipple = values->stipple;
-      if (data->stipple != NULL)
+      if (win32_gc->stipple != NULL)
+       gdk_drawable_unref (win32_gc->stipple);
+      win32_gc->stipple = values->stipple;
+      if (win32_gc->stipple != NULL)
        {
-         gdk_drawable_ref (data->stipple);
-         data->values_mask |= GDK_GC_STIPPLE;
-         GDK_NOTE (MISC, (g_print ("%sstipple=%#x", s,
-                                   GDK_DRAWABLE_XID (data->stipple)),
-                          s = ","));
+         gdk_drawable_get_size (win32_gc->stipple, &sw, &sh);
+
+#if 0 /* HB: this size limitation is disabled to make radio and check
+       * buttons work. I got the impression from the API docs, that
+       * it shouldn't be necessary at all, but win9x would do the clipping
+       */
+         if (   (sw != 8 || sh != 8)
+             && !IS_WIN_NT ()) /* HB: the MSDN says it's a Win95 limitation */
+           {
+             /* It seems that it *must* be 8x8, at least on my machine. 
+              * Thus, tile an 8x8 bitmap with the stipple in case it is
+              * smaller, or simply use just the top left 8x8 in case it is
+              * larger.
+              */
+             gchar dummy[8];
+             GdkPixmap *bm = gdk_bitmap_create_from_data (NULL, dummy, 8, 8);
+             GdkGC *gc = gdk_gc_new (bm);
+             gint i, j;
+
+             i = 0;
+             while (i < 8)
+               {
+                 j = 0;
+                 while (j < 8)
+                   {
+                     gdk_draw_drawable (bm, gc, win32_gc->stipple, 0, 0, i, j, sw, sh);
+                     j += sh;
+                   }
+                 i += sw;
+               }
+             win32_gc->stipple = bm;
+             gdk_gc_unref (gc);
+           }
+         else
+#endif
+           gdk_drawable_ref (win32_gc->stipple);
+         win32_gc->values_mask |= GDK_GC_STIPPLE;
+         GDK_NOTE (GC,
+                   (g_print ("%sstipple=%p", s,
+                             GDK_PIXMAP_HBITMAP (win32_gc->stipple)),
+                    s = ","));
        }
       else
        {
-         data->values_mask &= ~GDK_GC_STIPPLE;
-         GDK_NOTE (MISC, (g_print ("%sstipple=NULL", s),
-                          s = ","));
+         win32_gc->values_mask &= ~GDK_GC_STIPPLE;
+         GDK_NOTE (GC, (g_print ("%sstipple=NULL", s),
+                        s = ","));
        }
     }
 
   if (mask & GDK_GC_CLIP_MASK)
     {
-      if (data->clip_region != NULL)
-       if (!DeleteObject (data->clip_region))
-         WIN32_API_FAILED ("DeleteObject");
+      if (win32_gc->hcliprgn != NULL)
+       DeleteObject (win32_gc->hcliprgn);
+
       if (values->clip_mask != NULL)
        {
-         data->clip_region =
-           BitmapToRegion ((HBITMAP) GDK_DRAWABLE_XID (values->clip_mask));
-         data->values_mask |= GDK_GC_CLIP_MASK;
+         win32_gc->hcliprgn = _gdk_win32_bitmap_to_hrgn (values->clip_mask);
+         win32_gc->values_mask |= GDK_GC_CLIP_MASK;
        }
       else
        {
-         data->clip_region = NULL;
-         data->values_mask &= ~GDK_GC_CLIP_MASK;
+         win32_gc->hcliprgn = NULL;
+         win32_gc->values_mask &= ~GDK_GC_CLIP_MASK;
        }
-      GDK_NOTE (MISC, (g_print ("%sclip=%#x", s, data->clip_region),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sclip=%p", s, win32_gc->hcliprgn),
+                    s = ","));
     }
 
   if (mask & GDK_GC_SUBWINDOW)
     {
-      data->subwindow_mode = values->subwindow_mode;
-      data->values_mask |= GDK_GC_SUBWINDOW;
-      GDK_NOTE (MISC, (g_print ("%ssubw=%d", s, data->subwindow_mode),
-                      s = ","));
+      win32_gc->subwindow_mode = values->subwindow_mode;
+      win32_gc->values_mask |= GDK_GC_SUBWINDOW;
+      GDK_NOTE (GC, (g_print ("%ssubw=%d", s, win32_gc->subwindow_mode),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TS_X_ORIGIN)
     {
-      data->ts_x_origin = values->ts_x_origin;
-      data->values_mask |= GDK_GC_TS_X_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sts_x=%d", s, data->ts_x_origin),
-                      s = ","));
+      win32_gc->values_mask |= GDK_GC_TS_X_ORIGIN;
+      GDK_NOTE (GC, (g_print ("%sts_x=%d", s, values->ts_x_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TS_Y_ORIGIN)
     {
-      data->ts_y_origin = values->ts_y_origin;
-      data->values_mask |= GDK_GC_TS_Y_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sts_y=%d", s, data->ts_y_origin),
-                      s = ","));
+      win32_gc->values_mask |= GDK_GC_TS_Y_ORIGIN;
+      GDK_NOTE (GC, (g_print ("%sts_y=%d", s, values->ts_y_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_CLIP_X_ORIGIN)
     {
-      data->clip_x_origin = values->clip_x_origin;
-      data->values_mask |= GDK_GC_CLIP_X_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sclip_x=%d", s, data->clip_x_origin),
-                      s = ","));
+      win32_gc->values_mask |= GDK_GC_CLIP_X_ORIGIN;
+      GDK_NOTE (GC, (g_print ("%sclip_x=%d", s, values->clip_x_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_CLIP_Y_ORIGIN)
     {
-      data->clip_y_origin = values->clip_y_origin; 
-      data->values_mask |= GDK_GC_CLIP_Y_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sclip_y=%d", s, data->clip_y_origin),
-                      s = ","));
-   }
+      win32_gc->values_mask |= GDK_GC_CLIP_Y_ORIGIN;
+      GDK_NOTE (GC, (g_print ("%sclip_y=%d", s, values->clip_y_origin),
+                    s = ","));
+    }
+
   if (mask & GDK_GC_EXPOSURES)
     {
-      data->graphics_exposures = values->graphics_exposures;
-      data->values_mask |= GDK_GC_EXPOSURES;
-      GDK_NOTE (MISC, (g_print ("%sexp=%d", s, data->graphics_exposures),
-                      s = ","));
+      win32_gc->graphics_exposures = values->graphics_exposures;
+      win32_gc->values_mask |= GDK_GC_EXPOSURES;
+      GDK_NOTE (GC, (g_print ("%sexp=%d", s, win32_gc->graphics_exposures),
+                    s = ","));
     }
 
   if (mask & GDK_GC_LINE_WIDTH)
     {
-      data->pen_width = values->line_width;
-      data->values_mask |= GDK_GC_LINE_WIDTH;
-      GDK_NOTE (MISC, (g_print ("%spw=%d", s, data->pen_width),
-                      s = ","));
+      win32_gc->pen_width = values->line_width;
+      win32_gc->values_mask |= GDK_GC_LINE_WIDTH;
+      GDK_NOTE (GC, (g_print ("%spw=%d", s, win32_gc->pen_width),
+                    s = ","));
     }
 
   if (mask & GDK_GC_LINE_STYLE)
     {
-      data->pen_style &= ~(PS_STYLE_MASK);
-      GDK_NOTE (MISC, (g_print ("%sps|=", s),
-                      s = ","));
       switch (values->line_style)
        {
        case GDK_LINE_SOLID:
-         GDK_NOTE (MISC, g_print ("LINE_SOLID"));
-         data->pen_style |= PS_SOLID;
+         if (win32_gc->pen_dashes)
+           {
+             g_free (win32_gc->pen_dashes);
+             win32_gc->pen_dashes = NULL;
+            win32_gc->pen_num_dashes = 0;
+           }
+          win32_gc->pen_style &= ~(PS_STYLE_MASK);
+         win32_gc->pen_style |= PS_SOLID;
          break;
        case GDK_LINE_ON_OFF_DASH:
-       case GDK_LINE_DOUBLE_DASH: /* ??? */
-         GDK_NOTE (MISC, g_print ("DASH"));
-         data->pen_style |= PS_DASH;
+       case GDK_LINE_DOUBLE_DASH: 
+         if (!win32_gc->pen_dashes)
+           {
+            /* setting to PS_DASH probably isn't correct. If I understand the
+             * xlib docs correctly it should influence the handling of
+             * line endings ? --hb
+             */
+            win32_gc->pen_style &= ~(PS_STYLE_MASK);
+           win32_gc->pen_style |= PS_DASH;
+          }
          break;
        }
-      data->values_mask |= GDK_GC_LINE_STYLE;
+      GDK_NOTE (GC, (g_print ("%sps|=PS_STYLE_%s", s, _gdk_win32_psstyle_to_string (win32_gc->pen_style)),
+                    s = ","));
+      win32_gc->values_mask |= GDK_GC_LINE_STYLE;
     }
 
   if (mask & GDK_GC_CAP_STYLE)
     {
-      data->pen_style &= ~(PS_ENDCAP_MASK);
-      GDK_NOTE (MISC, (g_print ("%sps|=", s),
-                      s = ","));
+      win32_gc->pen_style &= ~(PS_ENDCAP_MASK);
       switch (values->cap_style)
        {
        case GDK_CAP_NOT_LAST:  /* ??? */
        case GDK_CAP_BUTT:
-         GDK_NOTE (MISC, g_print ("ENDCAP_FLAT"));
-         data->pen_style |= PS_ENDCAP_FLAT;
+         win32_gc->pen_style |= PS_ENDCAP_FLAT;
          break;
        case GDK_CAP_ROUND:
-         GDK_NOTE (MISC, g_print ("ENDCAP_ROUND"));
-         data->pen_style |= PS_ENDCAP_ROUND;
+         win32_gc->pen_style |= PS_ENDCAP_ROUND;
          break;
        case GDK_CAP_PROJECTING:
-         GDK_NOTE (MISC, g_print ("ENDCAP_SQUARE"));
-         data->pen_style |= PS_ENDCAP_SQUARE;
+         win32_gc->pen_style |= PS_ENDCAP_SQUARE;
          break;
        }
-      data->values_mask |= GDK_GC_CAP_STYLE;
+      GDK_NOTE (GC, (g_print ("%sps|=PS_ENDCAP_%s", s, _gdk_win32_psendcap_to_string (win32_gc->pen_style)),
+                    s = ","));
+      win32_gc->values_mask |= GDK_GC_CAP_STYLE;
     }
 
   if (mask & GDK_GC_JOIN_STYLE)
     {
-      data->pen_style &= ~(PS_JOIN_MASK);
-      GDK_NOTE (MISC, (g_print ("%sps|=", s),
-                      s = ","));
+      win32_gc->pen_style &= ~(PS_JOIN_MASK);
       switch (values->join_style)
        {
        case GDK_JOIN_MITER:
-         GDK_NOTE (MISC, g_print ("JOIN_MITER"));
-         data->pen_style |= PS_JOIN_MITER;
+         win32_gc->pen_style |= PS_JOIN_MITER;
          break;
        case GDK_JOIN_ROUND:
-         GDK_NOTE (MISC, g_print ("JOIN_ROUND"));
-         data->pen_style |= PS_JOIN_ROUND;
+         win32_gc->pen_style |= PS_JOIN_ROUND;
          break;
        case GDK_JOIN_BEVEL:
-         GDK_NOTE (MISC, g_print ("JOIN_BEVEL"));
-         data->pen_style |= PS_JOIN_BEVEL;
+         win32_gc->pen_style |= PS_JOIN_BEVEL;
          break;
        }
-      data->values_mask |= GDK_GC_JOIN_STYLE;
+      GDK_NOTE (GC, (g_print ("%sps|=PS_JOIN_%s", s, _gdk_win32_psjoin_to_string (win32_gc->pen_style)),
+                    s = ","));
+      win32_gc->values_mask |= GDK_GC_JOIN_STYLE;
     }
-  GDK_NOTE (MISC, g_print ("}\n"));
+  GDK_NOTE (GC, g_print ("} mask=(%s)", _gdk_win32_gcvalues_mask_to_string (win32_gc->values_mask)));
 }
 
 GdkGC*
@@ -373,82 +432,58 @@ _gdk_win32_gc_new (GdkDrawable      *drawable,
                   GdkGCValuesMask mask)
 {
   GdkGC *gc;
-  GdkGCPrivate *private;
-  GdkGCWin32Data *data;
-  static GdkColor black;
-  static GdkColor white;
-  static gboolean beenhere = FALSE;
+  GdkGCWin32 *win32_gc;
 
-  if (!beenhere)
-    {
-      gdk_color_black (gdk_colormap_get_system (), &black);
-      gdk_color_white (gdk_colormap_get_system (), &white);
-      beenhere = TRUE;
-    }
-
-  gc = gdk_gc_alloc ();
-  private = (GdkGCPrivate *)gc;
-
-  private->klass = &gdk_win32_gc_class;
-  private->klass_data = data = g_new (GdkGCWin32Data, 1);
-    
-  data->foreground = black.pixel;
-  data->background = white.pixel;
-  data->font = NULL;
-  data->rop2 = R2_COPYPEN;
-  data->fill_style = GDK_SOLID;
-  data->tile = NULL;
-  data->stipple = NULL;
-  data->clip_region = NULL;
-  data->ts_x_origin = data->ts_y_origin =
-    data->clip_x_origin = data->clip_y_origin = 0;
-  data->pen_style = PS_GEOMETRIC;
-  data->pen_width = 1;
-
-  data->values_mask = GDK_GC_FUNCTION | GDK_GC_FILL;
+  /* NOTICE that the drawable here has to be the impl drawable,
+   * not the publically-visible drawables.
+   */
+  g_return_val_if_fail (GDK_IS_DRAWABLE_IMPL_WIN32 (drawable), NULL);
 
-  GDK_NOTE (MISC, g_print ("_gdk_win32_gc_new: "));
-  gdk_win32_gc_values_to_win32values (values, mask, data);
+  gc = g_object_new (_gdk_gc_win32_get_type (), NULL);
+  win32_gc = GDK_GC_WIN32 (gc);
 
-  data->hwnd = NULL;
-  data->xgc = NULL;
+  win32_gc->hcliprgn = NULL;
 
-  GDK_NOTE (MISC, g_print (" = %p\n", gc));
+  /* Use the same default values as X11 does, even if they don't make
+   * sense per se. But apps always set fg and bg anyway.
+   */
+  win32_gc->foreground = 0;
+  win32_gc->background = 1;
+  win32_gc->font = NULL;
+  win32_gc->rop2 = R2_COPYPEN;
+  win32_gc->fill_style = GDK_SOLID;
+  win32_gc->tile = NULL;
+  win32_gc->stipple = NULL;
+  win32_gc->subwindow_mode = GDK_CLIP_BY_CHILDREN;
+  win32_gc->graphics_exposures = TRUE;
+  win32_gc->pen_width = 0;
+  win32_gc->pen_style = PS_GEOMETRIC|PS_ENDCAP_FLAT|PS_JOIN_MITER;
+  win32_gc->pen_dashes = NULL;
+  win32_gc->pen_num_dashes = 0;
+
+  win32_gc->values_mask = GDK_GC_FUNCTION | GDK_GC_FILL;
+
+  GDK_NOTE (GC, g_print ("_gdk_win32_gc_new: %p: ", win32_gc));
+  gdk_win32_gc_values_to_win32values (values, mask, win32_gc);
+  GDK_NOTE (GC, g_print ("\n"));
+
+  win32_gc->hdc = NULL;
+  win32_gc->hwnd = NULL;
 
   return gc;
 }
 
-static void
-gdk_win32_gc_destroy (GdkGC *gc)
-{
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc);
-
-  if (data->values_mask & GDK_GC_FONT)
-    gdk_font_unref (data->font);
-  
-  if (data->values_mask & GDK_GC_TILE)
-    gdk_drawable_unref (data->tile);
-  
-  if (data->values_mask & GDK_GC_STIPPLE)
-    gdk_drawable_unref (data->stipple);
-  
-  if (data->values_mask & GDK_GC_CLIP_MASK)
-    DeleteObject (data->clip_region);
-
-  g_free (GDK_GC_WIN32DATA (gc));
-}
-
 static void
 gdk_win32_gc_get_values (GdkGC       *gc,
                         GdkGCValues *values)
 {
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc);
+  GdkGCWin32 *win32_gc = GDK_GC_WIN32 (gc);
 
-  values->foreground.pixel = data->foreground;
-  values->background.pixel = data->background;
-  values->font = data->font;
+  values->foreground.pixel = win32_gc->foreground;
+  values->background.pixel = win32_gc->background;
+  values->font = win32_gc->font;
 
-  switch (data->rop2)
+  switch (win32_gc->rop2)
     {
     case R2_COPYPEN:
       values->function = GDK_COPY; break;
@@ -478,73 +513,47 @@ gdk_win32_gc_get_values (GdkGC       *gc,
       values->function = GDK_OR_INVERT; break;
     case R2_NOTMASKPEN:
       values->function = GDK_NAND; break;
+    case R2_NOTMERGEPEN:
+      values->function = GDK_NOR; break;
     case R2_WHITE:
       values->function = GDK_SET; break;
     }
 
-  values->fill = data->fill_style;
+  values->fill = win32_gc->fill_style;
 
-  values->tile = data->tile;
-  values->stipple = data->stipple;
+  values->tile = win32_gc->tile;
+  values->stipple = win32_gc->stipple;
 
-  if (data->clip_region != NULL)
-    {
-      RECT rect;
-      HBRUSH hbr;
-      HDC hdc;
-      HGDIOBJ oldbitmap;
-      GdkPixmap *pixmap;
-
-      GetRgnBox (data->clip_region, &rect);
-      pixmap =
-       gdk_pixmap_new (NULL, rect.right - rect.left, rect.bottom - rect.top,
-                       1);
-      hbr = GetStockObject (WHITE_BRUSH);
-      if ((hdc = CreateCompatibleDC (NULL)) == NULL)
-       WIN32_API_FAILED ("CreateCompatibleDC");
-      if ((oldbitmap =
-          SelectObject (hdc, GDK_DRAWABLE_XID (pixmap))) == NULL)
-       WIN32_API_FAILED ("SelectObject #1");
-      hbr = GetStockObject (BLACK_BRUSH);
-      if (!FillRect (hdc, &rect, hbr))
-       WIN32_API_FAILED ("FillRect");
-      hbr = GetStockObject (WHITE_BRUSH);
-      if (!FillRgn (hdc, data->clip_region, hbr))
-       WIN32_API_FAILED ("FillRgn");
-      if (SelectObject (hdc, oldbitmap) == NULL)
-       WIN32_API_FAILED ("SelectObject #2");
-      DeleteDC (hdc);
-      values->clip_mask = pixmap;
-    }
-  else
-    values->clip_mask = NULL;
-  values->subwindow_mode = data->subwindow_mode;
-  values->ts_x_origin = data->ts_x_origin;
-  values->ts_y_origin = data->ts_y_origin;
-  values->clip_x_origin = data->clip_x_origin;
-  values->clip_y_origin = data->clip_y_origin;
-  values->graphics_exposures = data->graphics_exposures;
-  values->line_width = data->pen_width;
+  /* Also the X11 backend always returns a NULL clip_mask */
+  values->clip_mask = NULL;
+
+  values->subwindow_mode = win32_gc->subwindow_mode;
+  values->ts_x_origin = win32_gc->parent_instance.ts_x_origin;
+  values->ts_y_origin = win32_gc->parent_instance.ts_y_origin;
+  values->clip_x_origin = win32_gc->parent_instance.clip_x_origin;
+  values->clip_y_origin = win32_gc->parent_instance.clip_y_origin;
+  values->graphics_exposures = win32_gc->graphics_exposures;
+  values->line_width = win32_gc->pen_width;
   
-  if (data->pen_style & PS_SOLID)
+  if (win32_gc->pen_style & PS_SOLID)
     values->line_style = GDK_LINE_SOLID;
-  else if (data->pen_style & PS_DASH)
+  else if (win32_gc->pen_style & PS_DASH)
     values->line_style = GDK_LINE_ON_OFF_DASH;
   else
     values->line_style = GDK_LINE_SOLID;
 
   /* PS_ENDCAP_ROUND is zero */
-  if (data->pen_style & PS_ENDCAP_FLAT)
+  if (win32_gc->pen_style & PS_ENDCAP_FLAT)
     values->cap_style = GDK_CAP_BUTT;
-  else if (data->pen_style & PS_ENDCAP_SQUARE)
+  else if (win32_gc->pen_style & PS_ENDCAP_SQUARE)
     values->cap_style = GDK_CAP_PROJECTING;
   else
     values->cap_style = GDK_CAP_ROUND;
     
   /* PS_JOIN_ROUND is zero */
-  if (data->pen_style & PS_JOIN_MITER)
+  if (win32_gc->pen_style & PS_JOIN_MITER)
     values->join_style = GDK_JOIN_MITER;
-  else if (data->pen_style & PS_JOIN_BEVEL)
+  else if (win32_gc->pen_style & PS_JOIN_BEVEL)
     values->join_style = GDK_JOIN_BEVEL;
   else
     values->join_style = GDK_JOIN_ROUND;
@@ -555,566 +564,527 @@ gdk_win32_gc_set_values (GdkGC           *gc,
                         GdkGCValues     *values,
                         GdkGCValuesMask  mask)
 {
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc);
+  g_return_if_fail (GDK_IS_GC (gc));
 
-  GDK_NOTE (MISC, g_print ("gdk_win32_gc_set_values: "));
-
-  gdk_win32_gc_values_to_win32values (values, mask, data);
+  GDK_NOTE (GC, g_print ("gdk_win32_gc_set_values: %p: ", GDK_GC_WIN32 (gc)));
+  gdk_win32_gc_values_to_win32values (values, mask, GDK_GC_WIN32 (gc));
+  GDK_NOTE (GC, g_print ("\n"));
 }
 
 static void
 gdk_win32_gc_set_dashes (GdkGC *gc,
-                        gint     dash_offset,
-                        gchar  dash_list[],
+                        gint   dash_offset,
+                        gint8  dash_list[],
                         gint   n)
 {
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc);
+  GdkGCWin32 *win32_gc;
+  int i;
+
+  g_return_if_fail (GDK_IS_GC (gc));
+  g_return_if_fail (dash_list != NULL);
+
+  win32_gc = GDK_GC_WIN32 (gc);
 
-  /* XXX ??? */
+  /* mark as set, see gdk_win32_gc_values_to_win32values () for the reason */
+  win32_gc->values_mask |= GDK_GC_LINE_STYLE;
 
-  data->pen_style &= ~(PS_STYLE_MASK);
-  data->pen_style |= PS_DASH;
+  win32_gc->pen_style &= ~(PS_STYLE_MASK);
+
+  win32_gc->pen_style |= (PS_GEOMETRIC | PS_USERSTYLE);
+  win32_gc->pen_num_dashes = n;
+  win32_gc->pen_dashes = g_new (DWORD, n);
+  for (i = 0; i < n; i++)
+    win32_gc->pen_dashes[i] = dash_list[i];
 }
 
 void
 gdk_gc_set_clip_rectangle (GdkGC       *gc,
                           GdkRectangle *rectangle)
 {
-  GdkGCWin32Data *data;
-   
-  g_return_if_fail (gc != NULL);
+  GdkGCWin32 *win32_gc;
+
+  g_return_if_fail (GDK_IS_GC (gc));
 
-  data = GDK_GC_WIN32DATA (gc);
+  win32_gc = GDK_GC_WIN32 (gc);
+
+  if (win32_gc->hcliprgn)
+    DeleteObject (win32_gc->hcliprgn);
 
-  if (data->clip_region != NULL)
-    if (!DeleteObject (data->clip_region))
-      WIN32_API_FAILED ("DeleteObject");
   if (rectangle)
     {
-      GDK_NOTE (MISC,
-               g_print ("gdk_gc_set_clip_rectangle: (%d) %dx%d@+%d+%d\n",
-                        data,
-                        rectangle->width, rectangle->height,
-                        rectangle->x, rectangle->y));
-      if ((data->clip_region =
-          CreateRectRgn (rectangle->x, rectangle->y,
-                         rectangle->x + rectangle->width,
-                         rectangle->y + rectangle->height)) == NULL)
-       WIN32_API_FAILED ("CreateRectRgn");
-
-      data->values_mask |= GDK_GC_CLIP_MASK;
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_rectangle: %p: %s\n",
+                            win32_gc,
+                            _gdk_win32_gdkrectangle_to_string (rectangle)));
+      win32_gc->hcliprgn = CreateRectRgn (rectangle->x, rectangle->y,
+                                         rectangle->x + rectangle->width,
+                                         rectangle->y + rectangle->height);
+      win32_gc->values_mask |= GDK_GC_CLIP_MASK;
     }
   else
     {
-      GDK_NOTE (MISC, g_print ("gdk_gc_set_clip_rectangle: (%d) NULL\n",
-                              data));
-      data->clip_region = NULL;
-      data->values_mask &= ~GDK_GC_CLIP_MASK;
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_rectangle: NULL\n"));
+
+      win32_gc->hcliprgn = NULL;
+      win32_gc->values_mask &= ~GDK_GC_CLIP_MASK;
     }
-    data->values_mask &= ~(GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN);
+
+  gc->clip_x_origin = 0;
+  gc->clip_y_origin = 0;
+  
+  win32_gc->values_mask &= ~(GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN);
 } 
 
 void
-gdk_gc_set_clip_region (GdkGC           *gc,
-                       GdkRegion        *region)
+gdk_gc_set_clip_region (GdkGC    *gc,
+                       GdkRegion *region)
 {
-  GdkGCWin32Data *data;
+  GdkGCWin32 *win32_gc;
 
-  g_return_if_fail (gc != NULL);
+  g_return_if_fail (GDK_IS_GC (gc));
 
-  data = GDK_GC_WIN32DATA (gc);
+  win32_gc = GDK_GC_WIN32 (gc);
 
-  GDK_NOTE (MISC, g_print ("gdk_gc_set_clip_region: (%d) %s\n",
-                          data, (region != NULL ? "xxx" : "None")));
+  if (win32_gc->hcliprgn)
+    DeleteObject (win32_gc->hcliprgn);
 
-  if (data->clip_region != NULL)
-    if (!DeleteObject (data->clip_region))
-      WIN32_API_FAILED ("DeleteObject");
   if (region)
     {
-      GdkRegionPrivate *region_private;
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_region: %p: %s\n",
+                            win32_gc,
+                            _gdk_win32_gdkregion_to_string (region)));
 
-      region_private = (GdkRegionPrivate*) region;
-      data->clip_region = CreateRectRgn (1, 1, 0, 0);
-      CombineRgn (data->clip_region, region_private->xregion, NULL, RGN_COPY);
-      data->values_mask |= GDK_GC_CLIP_MASK;
+      win32_gc->hcliprgn = _gdk_win32_gdkregion_to_hrgn (region, 0, 0);
+      win32_gc->values_mask |= GDK_GC_CLIP_MASK;
     }
   else
     {
-      data->clip_region = NULL;
-      data->values_mask &= ~GDK_GC_CLIP_MASK;
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_region: NULL\n"));
+
+      win32_gc->hcliprgn = NULL;
+      win32_gc->values_mask &= ~GDK_GC_CLIP_MASK;
     }
+
+  gc->clip_x_origin = 0;
+  gc->clip_y_origin = 0;
+  
+  win32_gc->values_mask &= ~(GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN);
 }
 
 void
-gdk_gc_copy (GdkGC *dst_gc, GdkGC *src_gc)
+gdk_gc_copy (GdkGC *dst_gc,
+            GdkGC *src_gc)
 {
-  GdkGCWin32Data *dst_data = GDK_GC_WIN32DATA (dst_gc);
-  GdkGCWin32Data *src_data = GDK_GC_WIN32DATA (src_gc);
-  DWORD nbytes;
-  LPRGNDATA rgn;
-
-  if (dst_data->font != NULL)
-    gdk_font_unref (dst_data->font);
-  if (dst_data->tile != NULL)
-    gdk_drawable_unref (dst_data->tile);
-  if (dst_data->stipple != NULL)
-    gdk_drawable_unref (dst_data->stipple);
-  if (dst_data->clip_region != NULL)
-    if (!DeleteObject (dst_data->clip_region))
-      WIN32_API_FAILED ("DeleteObject");
-  
-  *dst_data = *src_data;
+  GdkGCWin32 *dst_win32_gc;
+  GdkGCWin32 *src_win32_gc;
 
-  if (dst_data->clip_region != NULL)
-    {
-      nbytes = GetRegionData (dst_data->clip_region, 0, NULL);
-      rgn = g_malloc (nbytes);
-      GetRegionData (dst_data->clip_region, nbytes, rgn);
-      if ((dst_data->clip_region = ExtCreateRegion (NULL, nbytes, rgn)) == NULL)
-       WIN32_API_FAILED ("ExtCreateRegion");
-      g_free (rgn);
-    }
+  g_return_if_fail (GDK_IS_GC_WIN32 (dst_gc));
+  g_return_if_fail (GDK_IS_GC_WIN32 (src_gc));
   
-  if (dst_data->font != NULL)
-    gdk_font_ref (dst_data->font);
-  if (dst_data->tile != NULL)
-    gdk_drawable_ref (dst_data->tile);
-  if (dst_data->stipple != NULL)
-    gdk_drawable_ref (dst_data->stipple);
-}
+  dst_win32_gc = GDK_GC_WIN32 (dst_gc);
+  src_win32_gc = GDK_GC_WIN32 (src_gc);
 
-static guint mask[9] = { 0, 1, 3, 7, 15, 31, 63, 127, 255 };
+  GDK_NOTE (GC, g_print ("gdk_gc_copy: %p := %p\n", dst_win32_gc, src_win32_gc));
 
-static void
-predraw_set_foreground (GdkGCWin32Data          *data,
-                       GdkColormapPrivateWin32 *colormap_private)
-{
-  COLORREF fg;
-  LOGBRUSH logbrush;
-  HPEN hpen;
-  HBRUSH hbr;
-  GdkVisual *visual;
-  guchar r, g, b;
+  if (dst_gc->colormap)
+    g_object_unref (G_OBJECT (dst_gc->colormap));
 
-  if (colormap_private == NULL)
-    {
-      /* A 1 bit deep bitmap */
-      struct
-      {
-       WORD palVersion;
-       WORD palNumEntries;
-       PALETTEENTRY palPalEntry[2];
-      } logpal;
-      static HPALETTE hpal = NULL;
+  if (dst_win32_gc->hcliprgn != NULL)
+    DeleteObject (dst_win32_gc->hcliprgn);
 
-      if (hpal == NULL)
-       {
-         /* Create a b&w palette */
-         logpal.palVersion = 0x300;
-         logpal.palNumEntries = 2;
-         logpal.palPalEntry[0].peRed = 
-           logpal.palPalEntry[0].peGreen = 
-           logpal.palPalEntry[0].peBlue = 0x00;
-         logpal.palPalEntry[0].peFlags = 0x00;
-         logpal.palPalEntry[1].peRed = 
-           logpal.palPalEntry[1].peGreen = 
-           logpal.palPalEntry[1].peBlue = 0xFF;
-         logpal.palPalEntry[1].peFlags = 0x00;
-         if ((hpal = CreatePalette ((LOGPALETTE *) &logpal)) == NULL)
-           WIN32_API_FAILED ("CreatePalette");
-       }
-      SelectPalette (data->xgc, hpal, FALSE);
-      RealizePalette (data->xgc);
-      fg = PALETTEINDEX (data->foreground);
-    }
-  else if (colormap_private->xcolormap->rc_palette)
-    {
-      int k;
-      if (SelectPalette (data->xgc, colormap_private->xcolormap->palette,
-                        FALSE) == NULL)
-       WIN32_API_FAILED ("SelectPalette");
-      if (TRUE || colormap_private->xcolormap->stale)
-       {
-         if ((k = RealizePalette (data->xgc)) == GDI_ERROR)
-           WIN32_API_FAILED ("RealizePalette");
-         colormap_private->xcolormap->stale = FALSE;
-       }
-#if 0
-      g_print ("Selected palette %#x for gc %#x, realized %d colors\n",
-              colormap_private->xcolormap->palette, data->xgc, k);
-#endif
-      fg = PALETTEINDEX (data->foreground);
-    }
-  else
-    {
-      visual = colormap_private->base.visual;
-      r = (data->foreground & visual->red_mask) >> visual->red_shift;
-      r = (r * 255) / mask[visual->red_prec];
-      g = (data->foreground & visual->green_mask) >> visual->green_shift;
-      g = (g * 255) / mask[visual->green_prec];
-      b = (data->foreground & visual->blue_mask) >> visual->blue_shift;
-      b = (b * 255) / mask[visual->blue_prec];
-
-      fg = GetNearestColor (data->xgc, RGB (r, g, b));
-    }
+  if (dst_win32_gc->font != NULL)
+    gdk_font_unref (dst_win32_gc->font);
 
-  if (SetTextColor (data->xgc, fg) == CLR_INVALID)
-    WIN32_API_FAILED ("SetTextColor");
+  if (dst_win32_gc->tile != NULL)
+    gdk_drawable_unref (dst_win32_gc->tile);
 
-  /* Create and select pen and brush. */
+  if (dst_win32_gc->stipple != NULL)
+    gdk_drawable_unref (dst_win32_gc->stipple);
 
-  logbrush.lbStyle = BS_SOLID;
-  logbrush.lbColor = fg;
-
-  if ((hpen = ExtCreatePen (data->pen_style, data->pen_width,
-                           &logbrush, 0, NULL)) == NULL)
-    WIN32_API_FAILED ("ExtCreatePen");
+  if (dst_win32_gc->pen_dashes)
+    g_free (dst_win32_gc->pen_dashes);
   
-  if (SelectObject (data->xgc, hpen) == NULL)
-    WIN32_API_FAILED ("SelectObject #2");
+  *dst_win32_gc = *src_win32_gc;
+  dst_win32_gc->hdc = NULL;
+
+  if (dst_gc->colormap)
+    g_object_ref (G_OBJECT (dst_gc->colormap));
 
-  switch (data->fill_style)
+  if (dst_win32_gc->hcliprgn)
     {
-#if 1
-    case GDK_OPAQUE_STIPPLED:
-      if ((hbr = CreatePatternBrush (GDK_DRAWABLE_XID (data->stipple))) == NULL)
-       WIN32_API_FAILED ("CreatePatternBrush");
-       
-      SetBrushOrgEx(data->xgc, data->ts_x_origin,
-                   data->ts_y_origin, NULL);
-      break;
-#endif
+      /* create a new region, to copy to */
+      dst_win32_gc->hcliprgn = CreateRectRgn (0,0,1,1);
+      /* overwrite from source */
+      CombineRgn (dst_win32_gc->hcliprgn, src_win32_gc->hcliprgn,
+                 NULL, RGN_COPY);
+    }
 
-    case GDK_SOLID:
-    default:
-      if ((hbr = CreateSolidBrush (fg)) == NULL)
-       WIN32_API_FAILED ("CreateSolidBrush");
-      break;
-  }
+  if (dst_win32_gc->font != NULL)
+    gdk_font_ref (dst_win32_gc->font);
 
-  if (SelectObject (data->xgc, hbr) == NULL)
-    WIN32_API_FAILED ("SelectObject #3");
-}  
+  if (dst_win32_gc->tile != NULL)
+    gdk_drawable_ref (dst_win32_gc->tile);
 
-void
-predraw_set_background (GdkGCWin32Data          *data,
-                       GdkColormapPrivateWin32 *colormap_private)
+  if (dst_win32_gc->stipple != NULL)
+    gdk_drawable_ref (dst_win32_gc->stipple);
+
+  if (dst_win32_gc->pen_dashes)
+    dst_win32_gc->pen_dashes = g_memdup (src_win32_gc->pen_dashes, 
+                                         sizeof (DWORD) * src_win32_gc->pen_num_dashes);
+}
+
+GdkScreen *  
+gdk_gc_get_screen (GdkGC *gc)
 {
-  COLORREF bg;
-  GdkVisual *visual;
+  g_return_val_if_fail (GDK_IS_GC_WIN32 (gc), NULL);
+  
+  return _gdk_screen;
+}
+
+static guint bitmask[9] = { 0, 1, 3, 7, 15, 31, 63, 127, 255 };
+
+COLORREF
+_gdk_win32_colormap_color (GdkColormap *colormap,
+                           gulong       pixel)
+{
+  const GdkVisual *visual;
+  GdkColormapPrivateWin32 *colormap_private;
   guchar r, g, b;
 
-  if (colormap_private == NULL)
-    {
-      /* a bitmap */
-      bg = PALETTEINDEX (data->background);
-    }
-  else if (colormap_private->xcolormap->rc_palette)
+  if (colormap == NULL)
+    return DIBINDEX (pixel & 1);
+
+  colormap_private = GDK_WIN32_COLORMAP_DATA (colormap);
+
+  g_assert (colormap_private != NULL);
+
+  visual = colormap->visual;
+  switch (visual->type)
     {
-      bg = PALETTEINDEX (data->background);
+    case GDK_VISUAL_GRAYSCALE:
+    case GDK_VISUAL_PSEUDO_COLOR:
+    case GDK_VISUAL_STATIC_COLOR:
+      return PALETTEINDEX (pixel);
+
+    case GDK_VISUAL_TRUE_COLOR:
+      r = (pixel & visual->red_mask) >> visual->red_shift;
+      r = (r * 255) / bitmask[visual->red_prec];
+      g = (pixel & visual->green_mask) >> visual->green_shift;
+      g = (g * 255) / bitmask[visual->green_prec];
+      b = (pixel & visual->blue_mask) >> visual->blue_shift;
+      b = (b * 255) / bitmask[visual->blue_prec];
+      return RGB (r, g, b);
+
+    default:
+      g_assert_not_reached ();
+      return 0;
     }
-  else
+}
+
+static COLORREF
+predraw_set_foreground (GdkGC       *gc,
+                       GdkColormap *colormap,
+                       gboolean    *ok)
+{
+  COLORREF fg;
+  GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
+  GdkColormapPrivateWin32 *colormap_private;
+  gint k;
+
+  if (colormap &&
+      (colormap->visual->type == GDK_VISUAL_PSEUDO_COLOR ||
+       colormap->visual->type == GDK_VISUAL_STATIC_COLOR))
     {
-      visual = colormap_private->base.visual;
-      r = (data->background & visual->red_mask) >> visual->red_shift;
-      r = (r * 255) / mask[visual->red_prec];
-      g = (data->background & visual->green_mask) >> visual->green_shift;
-      g = (g * 255) / mask[visual->green_prec];
-      b = (data->background & visual->blue_mask) >> visual->blue_shift;
-      b = (b * 255) / mask[visual->green_prec];
-
-      bg = GetNearestColor (data->xgc, RGB (r, g, b));
+      colormap_private = GDK_WIN32_COLORMAP_DATA (colormap);
+
+      g_assert (colormap_private != NULL);
+
+      if (!(win32_gc->holdpal = SelectPalette (win32_gc->hdc, colormap_private->hpal, FALSE)))
+       WIN32_GDI_FAILED ("SelectPalette"), *ok = FALSE;
+      else if ((k = RealizePalette (win32_gc->hdc)) == GDI_ERROR)
+       WIN32_GDI_FAILED ("RealizePalette"), *ok = FALSE;
+      else if (k > 0)
+       GDK_NOTE (COLORMAP, g_print ("predraw_set_foreground: realized %p: %d colors\n",
+                                    colormap_private->hpal, k));
     }
-  if (SetBkColor (data->xgc, bg) == CLR_INVALID)
-    WIN32_API_FAILED ("SetBkColor");
-}
 
+  fg = _gdk_win32_colormap_color (colormap, win32_gc->foreground);
+
+  GDK_NOTE (GC, g_print ("predraw_set_foreground: fg=%06lx\n", fg));
+  return fg;
+}
 
+/**
+ * gdk_win32_hdc_get:
+ * @drawable: destination #GdkDrawable
+ * @gc: #GdkGC to use for drawing on @drawable
+ * @usage: mask indicating what properties needs to be set up
+ *
+ * Allocates a Windows device context handle (HDC) for drawing into
+ * @drawable, and sets it up appropriately according to @usage.
+ *
+ * Each #GdkGC can at one time have only one HDC associated with it.
+ *
+ * The following flags in @mask are handled:
+ *
+ * If %GDK_GC_FOREGROUND is set in @mask, a solid brush of the
+ * foreground color in @gc is selected into the HDC. The text color of
+ * the HDC is also set. If the @drawable has a palette (256-color
+ * mode), the palette is selected and realized.
+ *
+ * If any of the line attribute flags (%GDK_GC_LINE_WIDTH,
+ * %GDK_GC_LINE_STYLE, %GDK_GC_CAP_STYLE and %GDK_GC_JOIN_STYLE) is
+ * set in @mask, a solid pen of the foreground color and appropriate
+ * width and stule is created and selected into the HDC. Note that the
+ * dash properties are not completely implemented.
+ *
+ * If the %GDK_GC_FONT flag is set, the background mix mode is set to
+ * %TRANSPARENT. and the text alignment is set to
+ * %TA_BASELINE|%TA_LEFT. Note that no font gets selected into the HDC
+ * by this function.
+ *
+ * Some things are done regardless of @mask: If the function in @gc is
+ * any other than %GDK_COPY, the raster operation of the HDC is
+ * set. If @gc has a clip mask, the clip region of the HDC is set.
+ *
+ * Note that the fill style, tile, stipple, and tile and stipple
+ * origins in the @gc are ignored by this function. (In general, tiles
+ * and stipples can't be implemented directly on Win32; you need to do
+ * multiple pass drawing and blitting to implement tiles or
+ * stipples. GDK does just that when you call the GDK drawing
+ * functions with a GC that asks for tiles or stipples.)
+ *
+ * When the HDC is no longer used, it should be released by calling
+ * <function>gdk_win32_hdc_release()</function> with the same
+ * parameters.
+ *
+ * If you modify the HDC by calling <function>SelectObject</function>
+ * you should undo those modifications before calling
+ * <function>gdk_win32_hdc_release()</function>.
+ *
+ * Return value: The HDC.
+ **/
 HDC
-gdk_gc_predraw (GdkDrawable    *drawable,
-               GdkGCPrivate   *gc_private,
-               GdkGCValuesMask usage)
+gdk_win32_hdc_get (GdkDrawable    *drawable,
+                  GdkGC          *gc,
+                  GdkGCValuesMask usage)
 {
-  GdkDrawablePrivate *drawable_private = (GdkDrawablePrivate *) drawable;
-  GdkColormapPrivateWin32 *colormap_private =
-    (GdkColormapPrivateWin32 *) drawable_private->colormap;
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc_private);
+  GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
+  GdkDrawableImplWin32 *impl = NULL;
+  gboolean ok = TRUE;
+  COLORREF fg = RGB (0, 0, 0);
+  LOGBRUSH logbrush;
+  HPEN hpen;
+  HBRUSH hbr;
 
-  g_assert (data->xgc == NULL);
+  g_assert (win32_gc->hdc == NULL);
 
-  data->hwnd = GDK_DRAWABLE_XID (drawable);
-  
-  if (GDK_DRAWABLE_TYPE (drawable) == GDK_DRAWABLE_PIXMAP)
+  if (GDK_IS_DRAWABLE_IMPL_WIN32 (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32(drawable);
+  else if (GDK_IS_WINDOW (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32 ((GDK_WINDOW_OBJECT (drawable))->impl);
+  else if (GDK_IS_PIXMAP (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32 ((GDK_PIXMAP_OBJECT (drawable))->impl);
+  else
+    g_assert_not_reached ();
+
+  win32_gc->hwnd = impl->handle;
+
+  if (GDK_IS_PIXMAP_IMPL_WIN32 (impl))
     {
-      if ((data->xgc = CreateCompatibleDC (NULL)) == NULL)
-       WIN32_API_FAILED ("CreateCompatibleDC");
+      if ((win32_gc->hdc = CreateCompatibleDC (NULL)) == NULL)
+       WIN32_GDI_FAILED ("CreateCompatibleDC"), ok = FALSE;
 
-      if ((data->saved_dc = SaveDC (data->xgc)) == 0)
-       WIN32_API_FAILED ("SaveDC #1");
+      if (ok && (win32_gc->saved_dc = SaveDC (win32_gc->hdc)) == 0)
+       WIN32_GDI_FAILED ("SaveDC"), ok = FALSE;
       
-      if (SelectObject (data->xgc, data->hwnd) == NULL)
-       WIN32_API_FAILED ("SelectObject #1");
+      if (ok && SelectObject (win32_gc->hdc, win32_gc->hwnd) == NULL)
+       WIN32_GDI_FAILED ("SelectObject"), ok = FALSE;
     }
   else
     {
-      if ((data->xgc = GetDC (data->hwnd)) == NULL)
-       WIN32_API_FAILED ("GetDC");
+      if ((win32_gc->hdc = GetDC (win32_gc->hwnd)) == NULL)
+       WIN32_GDI_FAILED ("GetDC");
       
-      if ((data->saved_dc = SaveDC (data->xgc)) == 0)
-       WIN32_API_FAILED ("SaveDC #2");
+      if (ok && (win32_gc->saved_dc = SaveDC (win32_gc->hdc)) == 0)
+       WIN32_GDI_FAILED ("SaveDC");
     }
   
-  if (usage & GDK_GC_FOREGROUND)
-    predraw_set_foreground (data, colormap_private);
+  if (ok && (usage & GDK_GC_FOREGROUND))
+    {
+      fg = predraw_set_foreground (gc, impl->colormap, &ok);
+      if (ok && (hbr = CreateSolidBrush (fg)) == NULL)
+       WIN32_GDI_FAILED ("CreateSolidBrush"), ok = FALSE;
 
-  if ((usage & GDK_GC_BACKGROUND) && (data->values_mask & GDK_GC_BACKGROUND))
-    predraw_set_background (data, colormap_private);
-  
-  if (usage & GDK_GC_FONT)
+      if (ok && SelectObject (win32_gc->hdc, hbr) == NULL)
+       WIN32_GDI_FAILED ("SelectObject"), ok = FALSE;
+
+      if (ok && SetTextColor (win32_gc->hdc, fg) == CLR_INVALID)
+       WIN32_GDI_FAILED ("SetTextColor"), ok = FALSE;
+    }
+
+  if (ok && (usage & LINE_ATTRIBUTES))
+    {
+      /* Create and select pen */
+      logbrush.lbStyle = BS_SOLID;
+      logbrush.lbColor = fg;
+      logbrush.lbHatch = 0;
+      
+      if (win32_gc->pen_num_dashes > 0 && !IS_WIN_NT ())
+       {
+         /* The Win9x GDI is rather limited so we either draw dashed
+          * lines ourselves (only horizontal and vertical) or let them be
+          * drawn solid to avoid implementing a whole line renderer.
+          */
+         if ((hpen = ExtCreatePen (
+                                   (win32_gc->pen_style & ~(PS_STYLE_MASK)) | PS_SOLID,
+                                   MAX (win32_gc->pen_width, 1),
+                                   &logbrush, 
+                                   0, NULL)) == NULL)
+           WIN32_GDI_FAILED ("ExtCreatePen"), ok = FALSE;
+       }
+      else
+       {
+         if ((hpen = ExtCreatePen (win32_gc->pen_style,
+                                   MAX (win32_gc->pen_width, 1),
+                                   &logbrush, 
+                                   win32_gc->pen_num_dashes,
+                                   win32_gc->pen_dashes)) == NULL)
+           WIN32_GDI_FAILED ("ExtCreatePen"), ok = FALSE;
+       }
+      
+      if (ok && SelectObject (win32_gc->hdc, hpen) == NULL)
+       WIN32_GDI_FAILED ("SelectObject"), ok = FALSE;
+    }
+
+  if (ok && (usage & GDK_GC_FONT))
     {
-      if (SetBkMode (data->xgc, TRANSPARENT) == 0)
-       WIN32_API_FAILED ("SetBkMode");
+      if (SetBkMode (win32_gc->hdc, TRANSPARENT) == 0)
+       WIN32_GDI_FAILED ("SetBkMode"), ok = FALSE;
   
-      if (SetTextAlign (data->xgc, TA_BASELINE) == GDI_ERROR)
-       WIN32_API_FAILED ("SetTextAlign");
+      if (ok && SetTextAlign (win32_gc->hdc, TA_BASELINE|TA_LEFT|TA_NOUPDATECP) == GDI_ERROR)
+       WIN32_GDI_FAILED ("SetTextAlign"), ok = FALSE;
     }
   
-  if (data->values_mask & GDK_GC_FUNCTION)
-    if (SetROP2 (data->xgc, data->rop2) == 0)
-      WIN32_API_FAILED ("SetROP2");
+  if (ok && win32_gc->rop2 != R2_COPYPEN)
+    if (SetROP2 (win32_gc->hdc, win32_gc->rop2) == 0)
+      WIN32_GDI_FAILED ("SetROP2"), ok = FALSE;
 
-  if (data->values_mask & GDK_GC_CLIP_MASK
-      && data->clip_region != NULL)
+  if (ok &&
+      (win32_gc->values_mask & GDK_GC_CLIP_MASK) &&
+      win32_gc->hcliprgn != NULL)
     {
-      if (data->values_mask & (GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN))
-       OffsetRgn (data->clip_region,
-                  data->clip_x_origin, data->clip_y_origin);
-      SelectClipRgn (data->xgc, data->clip_region);
+      if (SelectClipRgn (win32_gc->hdc, win32_gc->hcliprgn) == ERROR)
+       WIN32_API_FAILED ("SelectClipRgn"), ok = FALSE;
+
+      if (ok && win32_gc->values_mask & (GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN) &&
+         OffsetClipRgn (win32_gc->hdc,
+           win32_gc->values_mask & GDK_GC_CLIP_X_ORIGIN ? gc->clip_x_origin : 0,
+           win32_gc->values_mask & GDK_GC_CLIP_Y_ORIGIN ? gc->clip_y_origin : 0) == ERROR)
+       WIN32_API_FAILED ("OffsetClipRgn"), ok = FALSE;
     }
 
-  return data->xgc;
+  GDK_NOTE (GC, (g_print ("gdk_win32_hdc_get: %p (%s): ",
+                         win32_gc, _gdk_win32_gcvalues_mask_to_string (usage)),
+                _gdk_win32_print_dc (win32_gc->hdc)));
+
+  return win32_gc->hdc;
 }
 
+/**
+ * gdk_win32_hdc_release:
+ * @drawable: destination #GdkDrawable
+ * @gc: #GdkGC to use for drawing on @drawable
+ * @usage: mask indicating what properties were set up
+ *
+ * This function deallocates the Windows device context allocated by
+ * <funcion>gdk_win32_hdc_get()</function>. It should be called with
+ * the same parameters.
+ **/
 void
-gdk_gc_postdraw (GdkDrawable    *drawable,
-                GdkGCPrivate   *gc_private,
-                GdkGCValuesMask usage)
+gdk_win32_hdc_release (GdkDrawable    *drawable,
+                      GdkGC          *gc,
+                      GdkGCValuesMask usage)
 {
-  GdkDrawablePrivate *drawable_private = (GdkDrawablePrivate *) drawable;
-  GdkColormapPrivateWin32 *colormap_private =
-    (GdkColormapPrivateWin32 *) drawable_private->colormap;
-  GdkGCWin32Data *data = GDK_GC_WIN32DATA (gc_private);
+  GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
+  GdkDrawableImplWin32 *impl = NULL;
   HGDIOBJ hpen = NULL;
   HGDIOBJ hbr = NULL;
 
-  if (usage & GDK_GC_FOREGROUND)
-    {
-      if ((hpen = GetCurrentObject (data->xgc, OBJ_PEN)) == NULL)
-       WIN32_API_FAILED ("GetCurrentObject #1");
+  GDK_NOTE (GC, g_print ("gdk_win32_hdc_release: %p: %p (%s)\n",
+                        win32_gc, win32_gc->hdc,
+                        _gdk_win32_gcvalues_mask_to_string (usage)));
 
-      if ((hbr = GetCurrentObject (data->xgc, OBJ_BRUSH)) == NULL)
-       WIN32_API_FAILED ("GetCurrentObject #2");
-    }
+  if (GDK_IS_DRAWABLE_IMPL_WIN32 (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32(drawable);
+  else if (GDK_IS_WINDOW (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32 ((GDK_WINDOW_OBJECT (drawable))->impl);
+  else if (GDK_IS_PIXMAP (drawable))
+    impl = GDK_DRAWABLE_IMPL_WIN32 ((GDK_PIXMAP_OBJECT (drawable))->impl);
+  else
+    g_assert_not_reached ();
 
-  if (!RestoreDC (data->xgc, data->saved_dc))
-    WIN32_API_FAILED ("RestoreDC");
-#if 0
-  if (colormap_private != NULL
-      && colormap_private->xcolormap->rc_palette
-      && colormap_private->xcolormap->stale)
+  if (win32_gc->holdpal != NULL)
     {
-      SelectPalette (data->xgc, GetStockObject (DEFAULT_PALETTE), FALSE);
-      if (!UnrealizeObject (colormap_private->xcolormap->palette))
-       WIN32_API_FAILED ("UnrealizeObject");
-    }
-#endif
-  if (GDK_DRAWABLE_TYPE (drawable) == GDK_DRAWABLE_PIXMAP)
-    {
-      if (!DeleteDC (data->xgc))
-       WIN32_API_FAILED ("DeleteDC");
+      gint k;
+      
+      if (!SelectPalette (win32_gc->hdc, win32_gc->holdpal, FALSE))
+       WIN32_GDI_FAILED ("SelectPalette");
+      else if ((k = RealizePalette (win32_gc->hdc)) == GDI_ERROR)
+       WIN32_GDI_FAILED ("RealizePalette");
+      else if (k > 0)
+       GDK_NOTE (COLORMAP, g_print ("gdk_win32_hdc_release: realized %p: %d colors\n",
+                                    win32_gc->holdpal, k));
+      win32_gc->holdpal = NULL;
     }
+
+  if (usage & LINE_ATTRIBUTES)
+    if ((hpen = GetCurrentObject (win32_gc->hdc, OBJ_PEN)) == NULL)
+      WIN32_GDI_FAILED ("GetCurrentObject");
+  
+  if (usage & GDK_GC_FOREGROUND)
+    if ((hbr = GetCurrentObject (win32_gc->hdc, OBJ_BRUSH)) == NULL)
+      WIN32_GDI_FAILED ("GetCurrentObject");
+
+  GDI_CALL (RestoreDC, (win32_gc->hdc, win32_gc->saved_dc));
+
+  if (GDK_IS_PIXMAP_IMPL_WIN32 (impl))
+    GDI_CALL (DeleteDC, (win32_gc->hdc));
   else
-    {
-      ReleaseDC (data->hwnd, data->xgc);
-    }
+    GDI_CALL (ReleaseDC, (win32_gc->hwnd, win32_gc->hdc));
 
   if (hpen != NULL)
-    if (!DeleteObject (hpen))
-      WIN32_API_FAILED ("DeleteObject #1");
+    GDI_CALL (DeleteObject, (hpen));
   
   if (hbr != NULL)
-    if (!DeleteObject (hbr))
-      WIN32_API_FAILED ("DeleteObject #2");
-
-  if (data->values_mask & GDK_GC_CLIP_MASK
-      && data->clip_region != NULL
-      && (data->values_mask & (GDK_GC_CLIP_X_ORIGIN | GDK_GC_CLIP_Y_ORIGIN)))
-    OffsetRgn (data->clip_region,
-              -data->clip_x_origin, -data->clip_y_origin);
-  data->xgc = NULL;
+    GDI_CALL (DeleteObject, (hbr));
+
+  win32_gc->hdc = NULL;
 }
 
 /* This function originally from Jean-Edouard Lachand-Robert, and
- * available at www.codeguru.com. Simplified for our needs, now
- * handles just one-bit deep bitmaps (in Window parlance, ie those
- * that GDK calls bitmaps (and not pixmaps), with zero pixels being
- * transparent.
+ * available at www.codeguru.com. Simplified for our needs, not sure
+ * how much of the original code left any longer. Now handles just
+ * one-bit deep bitmaps (in Window parlance, ie those that GDK calls
+ * bitmaps (and not pixmaps), with zero pixels being transparent.
  */
 
-/*
- *  BitmapToRegion :  Create a region from the "non-transparent" pixels of
- *  a bitmap
- *  Author :      Jean-Edouard Lachand-Robert
- *  (http://www.geocities.com/Paris/LeftBank/1160/resume.htm), June 1998.
+/* _gdk_win32_bitmap_to_hrgn : Create a region from the
+ * "non-transparent" pixels of a bitmap.
  */
 
 HRGN
-BitmapToRegion (HBITMAP hBmp)
+_gdk_win32_bitmap_to_hrgn (GdkPixmap *pixmap)
 {
   HRGN hRgn = NULL;
-  HDC hMemDC;
-  BITMAP bm;
-
-  struct
-  {
-    BITMAPINFOHEADER bmiHeader;
-#if 1
-    WORD bmiColors[2];
-#else
-    RGBQUAD bmiColors[2];
-#endif
-  } bmi;
-  VOID *pbits8; 
-  HBITMAP hbm8;
-  struct
-  {
-    WORD palVersion;
-    WORD palNumEntries;
-    PALETTEENTRY palPalEntry[2];
-  } logpal;
-  static HPALETTE bwPalette = NULL;
-
-  HBITMAP holdBmp;
-  HDC hDC;
-
-  BITMAP bm8;
-  HBITMAP holdBmp2;
+  HRGN h;
   DWORD maxRects;
   RGNDATA *pData;
-  BYTE *p8;
-  int x, y;
-  HRGN h;
-
-  /* Create a B&W palette */
-  if (bwPalette == NULL)
-    {
-      /* Create a b&w palette */
-      logpal.palVersion = 0x300;
-      logpal.palNumEntries = 2;
-      logpal.palPalEntry[0].peRed = 
-       logpal.palPalEntry[0].peGreen = 
-       logpal.palPalEntry[0].peBlue = 0;
-      logpal.palPalEntry[0].peFlags = 0;
-      logpal.palPalEntry[1].peRed = 
-       logpal.palPalEntry[1].peGreen = 
-       logpal.palPalEntry[1].peBlue = 0xFF;
-      logpal.palPalEntry[1].peFlags = 0;
-      if ((bwPalette = CreatePalette ((LOGPALETTE *) &logpal)) == NULL)
-       WIN32_API_FAILED ("CreatePalette");
-    }
-
-  /* Create a memory DC inside which we will scan the bitmap content */
-  hMemDC = CreateCompatibleDC (NULL);
-  if (!hMemDC)
-    {
-      WIN32_API_FAILED ("CreateCompatibleDC #1");
-      return NULL;
-    }
+  GdkImage *image;
+  guchar *p;
+  gint x, y;
 
-  SelectPalette (hMemDC, bwPalette, FALSE);
-  RealizePalette (hMemDC);
-
-  /* Get bitmap size */
-  GetObject(hBmp, sizeof(bm), &bm);
-  
-  /* Create a 8 bits depth bitmap and select it into the memory DC */
-  bmi.bmiHeader.biSize = sizeof (BITMAPINFOHEADER);
-  bmi.bmiHeader.biWidth = bm.bmWidth;
-  bmi.bmiHeader.biHeight = bm.bmHeight;
-  bmi.bmiHeader.biPlanes = 1;
-  bmi.bmiHeader.biBitCount = 8;
-  bmi.bmiHeader.biCompression = BI_RGB;
-  bmi.bmiHeader.biSizeImage = 0;
-  bmi.bmiHeader.biXPelsPerMeter = 0;
-  bmi.bmiHeader.biYPelsPerMeter = 0;
-  bmi.bmiHeader.biClrUsed = 2;
-  bmi.bmiHeader.biClrImportant = 2;
-#if 1
-  bmi.bmiColors[0] = 0;
-  bmi.bmiColors[1] = 1;
-  hbm8 = CreateDIBSection (hMemDC, (BITMAPINFO *)&bmi,
-                           DIB_PAL_COLORS, &pbits8, NULL, 0);
-#else
-  bmi.bmiColors[0].rgbBlue =
-    bmi.bmiColors[0].rgbGreen =
-    bmi.bmiColors[0].rgbRed = 0x00;
-  bmi.bmiColors[0].rgbReserved = 0x00;
-
-  bmi.bmiColors[1].rgbBlue =
-    bmi.bmiColors[1].rgbGreen =
-    bmi.bmiColors[1].rgbRed = 0xFF;
-  bmi.bmiColors[0].rgbReserved = 0x00;
-
-  hbm8 = CreateDIBSection (hMemDC, (BITMAPINFO *)&bmi,
-                           DIB_RGB_COLORS, &pbits8, NULL, 0);
-#endif
-  if (!hbm8)
-    {
-      WIN32_API_FAILED ("CreateDIBSection");
-      DeleteDC (hMemDC);
-      return NULL;
-    }
-
-  holdBmp = (HBITMAP) SelectObject (hMemDC, hbm8);
-
-  /* Create a DC just to copy the bitmap into the memory DC*/
-  hDC = CreateCompatibleDC (hMemDC);
-  if (!hDC)
-    {
-      WIN32_API_FAILED ("CreateCompatibleDC #2");
-      SelectObject (hMemDC, holdBmp);
-      DeleteObject (hbm8);
-      DeleteDC (hMemDC);
-      return NULL;
-    }
-
-  /* Get how many bytes per row we have for the bitmap bits */
-  GetObject (hbm8, sizeof (bm8), &bm8);
-
-  /* Hans Breuer found a fix to the long-standing erroneous behaviour
-   * on NT 4.0: There seems to be a bug in Win NT 4.0 GDI: scanlines
-   * in bitmaps are dword aligned on both Win95 and NT. In the case of
-   * a bitmap with 22 bytes worth of width, GetObject above returns
-   * with bmWidth == 22. On Win95 bmWidthBytes == 24, as it should be,
-   * but on NT is it 22. We need to correct this here.
-   */
-  bm8.bmWidthBytes = (((bm8.bmWidthBytes-1)/4)+1)*4; /* dword aligned!! */
-
-  /* Copy the bitmap into the memory DC*/
-  holdBmp2 = (HBITMAP) SelectObject (hDC, hBmp);
-
-  if (!BitBlt (hMemDC, 0, 0, bm.bmWidth, bm.bmHeight, hDC, 0, 0, SRCCOPY))
-    {
-      WIN32_API_FAILED ("BitBlt");
-      SelectObject (hDC, holdBmp2);
-      SelectObject (hMemDC, holdBmp);
-      DeleteObject (hbm8);
-      DeleteDC (hMemDC);
-      return NULL;
-    }
-  SelectObject (hDC, holdBmp2);
-  DeleteDC (hDC);
+  image = GDK_PIXMAP_IMPL_WIN32 (GDK_PIXMAP_OBJECT (pixmap)->impl)->image;
+  g_assert (image->depth == 1);
 
   /* For better performances, we will use the ExtCreateRegion()
    * function to create the region. This function take a RGNDATA
@@ -1130,22 +1100,19 @@ BitmapToRegion (HBITMAP hBmp)
   pData->rdh.nCount = pData->rdh.nRgnSize = 0;
   SetRect (&pData->rdh.rcBound, MAXLONG, MAXLONG, 0, 0);
 
-  /* Scan each bitmap from bottom to top (the bitmap is inverted vertically)*/
-  p8 = (BYTE *) pbits8 + (bm8.bmHeight - 1) * bm8.bmWidthBytes;
-  for (y = 0; y < bm.bmHeight; y++)
+  for (y = 0; y < image->height; y++)
     {
       /* Scan each bitmap row from left to right*/
-      for (x = 0; x < bm.bmWidth; x++)
+      p = (guchar *) image->mem + y * image->bpl;
+      for (x = 0; x < image->width; x++)
        {
          /* Search for a continuous range of "non transparent pixels"*/
-         int x0 = x;
-         BYTE *p = p8 + x;
-         while (x < bm.bmWidth)
+         gint x0 = x;
+         while (x < image->width)
            {
-             if (*p == 0)
+             if ((((p[x/8])>>(7-(x%8)))&1) == 0)
                /* This pixel is "transparent"*/
                break;
-             p++;
              x++;
            }
          
@@ -1193,9 +1160,6 @@ BitmapToRegion (HBITMAP hBmp)
                }
            }
        }
-      
-      /* Go to next row (remember, the bitmap is inverted vertically)*/
-      p8 -= bm8.bmWidthBytes;
     }
   
   /* Create or extend the region with the remaining rectangles*/
@@ -1210,9 +1174,57 @@ BitmapToRegion (HBITMAP hBmp)
     hRgn = h;
 
   /* Clean up*/
-  SelectObject(hMemDC, holdBmp);
-  DeleteObject (hbm8);
-  DeleteDC (hMemDC);
+  g_free (pData);
 
   return hRgn;
 }
+
+HRGN
+_gdk_win32_gdkregion_to_hrgn (GdkRegion *region,
+                             gint       x_origin,
+                             gint       y_origin)
+{
+  HRGN hrgn;
+  RGNDATA *rgndata;
+  RECT *rect;
+  GdkRegionBox *boxes = region->rects;
+  guint nbytes =
+    sizeof (RGNDATAHEADER) + (sizeof (RECT) * region->numRects);
+  int i;
+
+  rgndata = g_malloc (nbytes);
+  rgndata->rdh.dwSize = sizeof (RGNDATAHEADER);
+  rgndata->rdh.iType = RDH_RECTANGLES;
+  rgndata->rdh.nCount = rgndata->rdh.nRgnSize = 0;
+  SetRect (&rgndata->rdh.rcBound,
+          G_MAXSHORT, G_MAXSHORT, G_MINSHORT, G_MINSHORT);
+
+  for (i = 0; i < region->numRects; i++)
+    {
+      rect = ((RECT *) rgndata->Buffer) + rgndata->rdh.nCount++;
+
+      rect->left = CLAMP (boxes[i].x1 + x_origin,
+                         G_MINSHORT, G_MAXSHORT);
+      rect->right = CLAMP (boxes[i].x2 + x_origin,
+                          G_MINSHORT, G_MAXSHORT);
+      rect->top = CLAMP (boxes[i].y1 + y_origin,
+                        G_MINSHORT, G_MAXSHORT);
+      rect->bottom = CLAMP (boxes[i].y2 + y_origin,
+                           G_MINSHORT, G_MAXSHORT);
+
+      if (rect->left < rgndata->rdh.rcBound.left)
+       rgndata->rdh.rcBound.left = rect->left;
+      if (rect->right > rgndata->rdh.rcBound.right)
+       rgndata->rdh.rcBound.right = rect->right;
+      if (rect->top < rgndata->rdh.rcBound.top)
+       rgndata->rdh.rcBound.top = rect->top;
+      if (rect->bottom > rgndata->rdh.rcBound.bottom)
+       rgndata->rdh.rcBound.bottom = rect->bottom;
+    }
+  if ((hrgn = ExtCreateRegion (NULL, nbytes, rgndata)) == NULL)
+    WIN32_API_FAILED ("ExtCreateRegion");
+
+  g_free (rgndata);
+
+  return (hrgn);
+}