]> 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 9482fa0914487eaa9f3a2f2ff71eb66725b67d1f..1ead0d071217c244f862160ec65d4c052b587f74 100644 (file)
@@ -1,5 +1,6 @@
 /* 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 Lesser General Public
@@ -24,7 +25,8 @@
  * 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>
 
@@ -32,7 +34,6 @@
 #include "gdkfont.h"
 #include "gdkpixmap.h"
 #include "gdkregion-generic.h"
-#include "gdkinternals.h"
 #include "gdkprivate-win32.h"
 
 static void gdk_win32_gc_get_values (GdkGC           *gc,
@@ -51,7 +52,7 @@ static void gdk_gc_win32_finalize   (GObject         *object);
 static gpointer parent_class = NULL;
 
 GType
-gdk_gc_win32_get_type (void)
+_gdk_gc_win32_get_type (void)
 {
   static GType object_type = 0;
 
@@ -98,8 +99,8 @@ gdk_gc_win32_finalize (GObject *object)
 {
   GdkGCWin32 *win32_gc = GDK_GC_WIN32 (object);
   
-  if (win32_gc->clip_region)
-    gdk_region_destroy (win32_gc->clip_region);
+  if (win32_gc->hcliprgn != NULL)
+    DeleteObject (win32_gc->hcliprgn);
   
   if (win32_gc->values_mask & GDK_GC_FONT)
     gdk_font_unref (win32_gc->font);
@@ -109,6 +110,9 @@ gdk_gc_win32_finalize (GObject *object)
   
   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);
 }
@@ -121,22 +125,22 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
   char *s = "";
   gint sw, sh;
 
-  GDK_NOTE (MISC, g_print ("{"));
+  GDK_NOTE (GC, g_print ("{"));
 
   if (mask & GDK_GC_FOREGROUND)
     {
       win32_gc->foreground = values->foreground.pixel;
       win32_gc->values_mask |= GDK_GC_FOREGROUND;
-      GDK_NOTE (MISC, (g_print ("fg=%.06lx", win32_gc->foreground),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("fg=%.06lx", win32_gc->foreground),
+                    s = ","));
     }
   
   if (mask & GDK_GC_BACKGROUND)
     {
       win32_gc->background = values->background.pixel;
       win32_gc->values_mask |= GDK_GC_BACKGROUND;
-      GDK_NOTE (MISC, (g_print ("%sbg=%.06lx", s, win32_gc->background),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sbg=%.06lx", s, win32_gc->background),
+                    s = ","));
     }
 
   if ((mask & GDK_GC_FONT) && (values->font->type == GDK_FONT_FONT
@@ -147,93 +151,43 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
       win32_gc->font = values->font;
       if (win32_gc->font != NULL)
        {
-         gchar *xlfd;
-
          gdk_font_ref (win32_gc->font);
          win32_gc->values_mask |= GDK_GC_FONT;
-         GDK_NOTE (MISC, (xlfd = gdk_font_full_name_get (win32_gc->font),
-                          g_print ("%sfont=%s", s, xlfd),
-                          s = ",",
-                          gdk_font_full_name_free (xlfd)));
+         GDK_NOTE (GC, (g_print ("%sfont=%p", s, win32_gc->font),
+                        s = ","));
        }
       else
        {
          win32_gc->values_mask &= ~GDK_GC_FONT;
-         GDK_NOTE (MISC, (g_print ("%sfont=NULL", s),
-                          s = ","));
+         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:
-         win32_gc->rop2 = R2_COPYPEN;
-         GDK_NOTE (MISC, g_print ("COPYPEN"));
-         break;
-       case GDK_INVERT:
-         win32_gc->rop2 = R2_NOT;
-         GDK_NOTE (MISC, g_print ("NOT"));
-         break;
-       case GDK_XOR:
-         win32_gc->rop2 = R2_XORPEN;
-         GDK_NOTE (MISC, g_print ("XORPEN"));
-         break;
-       case GDK_CLEAR:
-         win32_gc->rop2 = R2_BLACK;
-         GDK_NOTE (MISC, g_print ("BLACK"));
-         break;
-       case GDK_AND:
-         win32_gc->rop2 = R2_MASKPEN;
-         GDK_NOTE (MISC, g_print ("MASKPEN"));
-         break;
-       case GDK_AND_REVERSE:
-         win32_gc->rop2 = R2_MASKPENNOT;
-         GDK_NOTE (MISC, g_print ("MASKPENNOT"));
-         break;
-       case GDK_AND_INVERT:
-         win32_gc->rop2 = R2_MASKNOTPEN;
-         GDK_NOTE (MISC, g_print ("MASKNOTPEN"));
-         break;
-       case GDK_NOOP:
-         win32_gc->rop2 = R2_NOP;
-         GDK_NOTE (MISC, g_print ("NOP"));
-         break;
-       case GDK_OR:
-         win32_gc->rop2 = R2_MERGEPEN;
-         GDK_NOTE (MISC, g_print ("MERGEPEN"));
-         break;
-       case GDK_EQUIV:
-         win32_gc->rop2 = R2_NOTXORPEN;
-         GDK_NOTE (MISC, g_print ("NOTXORPEN"));
-         break;
-       case GDK_OR_REVERSE:
-         win32_gc->rop2 = R2_MERGEPENNOT;
-         GDK_NOTE (MISC, g_print ("MERGEPENNOT"));
-         break;
-       case GDK_COPY_INVERT:
-         win32_gc->rop2 = R2_NOTCOPYPEN;
-         GDK_NOTE (MISC, g_print ("NOTCOPYPEN"));
-         break;
-       case GDK_OR_INVERT:
-         win32_gc->rop2 = R2_MERGENOTPEN;
-         GDK_NOTE (MISC, g_print ("MERGENOTPEN"));
-         break;
-       case GDK_NAND:
-         win32_gc->rop2 = R2_NOTMASKPEN;
-         GDK_NOTE (MISC, g_print ("NOTMASKPEN"));
-         break;
-       case GDK_NOR:
-         win32_gc->rop2 = R2_NOTMERGEPEN;
-         GDK_NOTE (MISC, g_print ("NOTMERGEPEN"));
-         break;
-       case GDK_SET:
-         win32_gc->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
        }
       win32_gc->values_mask |= GDK_GC_FUNCTION;
     }
@@ -242,8 +196,9 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
     {
       win32_gc->fill_style = values->fill;
       win32_gc->values_mask |= GDK_GC_FILL;
-      GDK_NOTE (MISC, (g_print ("%sfill=%d", s, win32_gc->fill_style),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sfill=%s", s,
+                             _gdk_win32_fill_style_to_string (win32_gc->fill_style)),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TILE)
@@ -255,16 +210,16 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
        {
          gdk_drawable_ref (win32_gc->tile);
          win32_gc->values_mask |= GDK_GC_TILE;
-         GDK_NOTE (MISC,
-                   (g_print ("%stile=%#x", s,
-                             (guint) GDK_PIXMAP_HBITMAP (win32_gc->tile)),
+         GDK_NOTE (GC,
+                   (g_print ("%stile=%p", s,
+                             GDK_PIXMAP_HBITMAP (win32_gc->tile)),
                     s = ","));
        }
       else
        {
          win32_gc->values_mask &= ~GDK_GC_TILE;
-         GDK_NOTE (MISC, (g_print ("%stile=NULL", s),
-                          s = ","));
+         GDK_NOTE (GC, (g_print ("%stile=NULL", s),
+                        s = ","));
        }
     }
 
@@ -277,7 +232,12 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
        {
          gdk_drawable_get_size (win32_gc->stipple, &sw, &sh);
 
-         if (sw != 8 || sh != 8)
+#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
@@ -304,172 +264,166 @@ gdk_win32_gc_values_to_win32values (GdkGCValues    *values,
              gdk_gc_unref (gc);
            }
          else
+#endif
            gdk_drawable_ref (win32_gc->stipple);
          win32_gc->values_mask |= GDK_GC_STIPPLE;
-         GDK_NOTE (MISC,
-                   (g_print ("%sstipple=%#x", s,
-                             (guint) GDK_PIXMAP_HBITMAP (win32_gc->stipple)),
+         GDK_NOTE (GC,
+                   (g_print ("%sstipple=%p", s,
+                             GDK_PIXMAP_HBITMAP (win32_gc->stipple)),
                     s = ","));
        }
       else
        {
          win32_gc->values_mask &= ~GDK_GC_STIPPLE;
-         GDK_NOTE (MISC, (g_print ("%sstipple=NULL", s),
-                          s = ","));
+         GDK_NOTE (GC, (g_print ("%sstipple=NULL", s),
+                        s = ","));
        }
     }
 
   if (mask & GDK_GC_CLIP_MASK)
     {
-      if (win32_gc->clip_region != NULL)
-       {
-         gdk_region_destroy (win32_gc->clip_region);
-         win32_gc->clip_region = NULL;
-       }
-
       if (win32_gc->hcliprgn != NULL)
-       {
-         DeleteObject (win32_gc->hcliprgn);
-         win32_gc->hcliprgn = NULL;
-       }
+       DeleteObject (win32_gc->hcliprgn);
 
       if (values->clip_mask != NULL)
        {
-         win32_gc->hcliprgn =
-           BitmapToRegion ((HBITMAP) GDK_PIXMAP_HBITMAP (values->clip_mask));
+         win32_gc->hcliprgn = _gdk_win32_bitmap_to_hrgn (values->clip_mask);
          win32_gc->values_mask |= GDK_GC_CLIP_MASK;
-         OffsetRgn (win32_gc->hcliprgn,
-                    win32_gc->parent_instance.clip_x_origin,
-                    win32_gc->parent_instance.clip_y_origin);
        }
       else
        {
          win32_gc->hcliprgn = NULL;
          win32_gc->values_mask &= ~GDK_GC_CLIP_MASK;
        }
-      GDK_NOTE (MISC, (g_print ("%sclip=%#x", s, (guint) win32_gc->hcliprgn),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sclip=%p", s, win32_gc->hcliprgn),
+                    s = ","));
     }
 
   if (mask & GDK_GC_SUBWINDOW)
     {
       win32_gc->subwindow_mode = values->subwindow_mode;
       win32_gc->values_mask |= GDK_GC_SUBWINDOW;
-      GDK_NOTE (MISC, (g_print ("%ssubw=%d", s, win32_gc->subwindow_mode),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%ssubw=%d", s, win32_gc->subwindow_mode),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TS_X_ORIGIN)
     {
       win32_gc->values_mask |= GDK_GC_TS_X_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sts_x=%d", s, values->ts_x_origin),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sts_x=%d", s, values->ts_x_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_TS_Y_ORIGIN)
     {
       win32_gc->values_mask |= GDK_GC_TS_Y_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sts_y=%d", s, values->ts_y_origin),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sts_y=%d", s, values->ts_y_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_CLIP_X_ORIGIN)
     {
       win32_gc->values_mask |= GDK_GC_CLIP_X_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sclip_x=%d", s, values->clip_x_origin),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sclip_x=%d", s, values->clip_x_origin),
+                    s = ","));
     }
 
   if (mask & GDK_GC_CLIP_Y_ORIGIN)
     {
       win32_gc->values_mask |= GDK_GC_CLIP_Y_ORIGIN;
-      GDK_NOTE (MISC, (g_print ("%sclip_y=%d", s, values->clip_y_origin),
-                      s = ","));
-   }
+      GDK_NOTE (GC, (g_print ("%sclip_y=%d", s, values->clip_y_origin),
+                    s = ","));
+    }
+
   if (mask & GDK_GC_EXPOSURES)
     {
       win32_gc->graphics_exposures = values->graphics_exposures;
       win32_gc->values_mask |= GDK_GC_EXPOSURES;
-      GDK_NOTE (MISC, (g_print ("%sexp=%d", s, win32_gc->graphics_exposures),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%sexp=%d", s, win32_gc->graphics_exposures),
+                    s = ","));
     }
 
   if (mask & GDK_GC_LINE_WIDTH)
     {
       win32_gc->pen_width = values->line_width;
       win32_gc->values_mask |= GDK_GC_LINE_WIDTH;
-      GDK_NOTE (MISC, (g_print ("%spw=%d", s, win32_gc->pen_width),
-                      s = ","));
+      GDK_NOTE (GC, (g_print ("%spw=%d", s, win32_gc->pen_width),
+                    s = ","));
     }
 
   if (mask & GDK_GC_LINE_STYLE)
     {
-      win32_gc->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"));
+         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"));
-         win32_gc->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;
        }
+      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)
     {
       win32_gc->pen_style &= ~(PS_ENDCAP_MASK);
-      GDK_NOTE (MISC, (g_print ("%sps|=", s),
-                      s = ","));
       switch (values->cap_style)
        {
        case GDK_CAP_NOT_LAST:  /* ??? */
        case GDK_CAP_BUTT:
-         GDK_NOTE (MISC, g_print ("ENDCAP_FLAT"));
          win32_gc->pen_style |= PS_ENDCAP_FLAT;
          break;
        case GDK_CAP_ROUND:
-         GDK_NOTE (MISC, g_print ("ENDCAP_ROUND"));
          win32_gc->pen_style |= PS_ENDCAP_ROUND;
          break;
        case GDK_CAP_PROJECTING:
-         GDK_NOTE (MISC, g_print ("ENDCAP_SQUARE"));
          win32_gc->pen_style |= PS_ENDCAP_SQUARE;
          break;
        }
+      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)
     {
       win32_gc->pen_style &= ~(PS_JOIN_MASK);
-      GDK_NOTE (MISC, (g_print ("%sps|=", s),
-                      s = ","));
       switch (values->join_style)
        {
        case GDK_JOIN_MITER:
-         GDK_NOTE (MISC, g_print ("JOIN_MITER"));
          win32_gc->pen_style |= PS_JOIN_MITER;
          break;
        case GDK_JOIN_ROUND:
-         GDK_NOTE (MISC, g_print ("JOIN_ROUND"));
          win32_gc->pen_style |= PS_JOIN_ROUND;
          break;
        case GDK_JOIN_BEVEL:
-         GDK_NOTE (MISC, g_print ("JOIN_BEVEL"));
          win32_gc->pen_style |= PS_JOIN_BEVEL;
          break;
        }
+      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*
@@ -485,11 +439,9 @@ _gdk_win32_gc_new (GdkDrawable       *drawable,
    */
   g_return_val_if_fail (GDK_IS_DRAWABLE_IMPL_WIN32 (drawable), NULL);
 
-  gc = g_object_new (gdk_gc_win32_get_type (), NULL);
+  gc = g_object_new (_gdk_gc_win32_get_type (), NULL);
   win32_gc = GDK_GC_WIN32 (gc);
 
-  win32_gc->hdc = NULL;
-  win32_gc->clip_region = NULL;
   win32_gc->hcliprgn = NULL;
 
   /* Use the same default values as X11 does, even if they don't make
@@ -502,18 +454,22 @@ _gdk_win32_gc_new (GdkDrawable      *drawable,
   win32_gc->fill_style = GDK_SOLID;
   win32_gc->tile = NULL;
   win32_gc->stipple = NULL;
-  win32_gc->pen_style = PS_GEOMETRIC|PS_ENDCAP_FLAT|PS_JOIN_MITER;
+  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 (MISC, g_print ("_gdk_win32_gc_new: "));
+  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;
 
-  GDK_NOTE (MISC, g_print (" = %p\n", gc));
-
   return gc;
 }
 
@@ -610,9 +566,9 @@ gdk_win32_gc_set_values (GdkGC           *gc,
 {
   g_return_if_fail (GDK_IS_GC (gc));
 
-  GDK_NOTE (MISC, g_print ("gdk_win32_gc_set_values: "));
-
+  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
@@ -622,68 +578,23 @@ gdk_win32_gc_set_dashes (GdkGC *gc,
                         gint   n)
 {
   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);
 
+  /* mark as set, see gdk_win32_gc_values_to_win32values () for the reason */
+  win32_gc->values_mask |= GDK_GC_LINE_STYLE;
+
   win32_gc->pen_style &= ~(PS_STYLE_MASK);
-  win32_gc->pen_style |= PS_DASH;
-
-  /* 
-   * Set the extended line style. This could be done by 
-   * PS_USERSTYLE and ExtCreatePen; but ONLY on WinNT, 
-   * so let's make a guess (based on the implementation 
-   * in DIA). On Win9x this does only work for lines
-   * with width one ...
-   *
-   * More workarounds for Win9x descibed at:
-   * http://www.codeguru.com/gdi/dashed.shtml
-   */
-  if (!IS_WIN_NT () && win32_gc->pen_width > 1)
-    {
-      GDK_NOTE (MISC, g_print ("gdk_win32_gc_set_dashes: not fully supported\n"));
-      win32_gc->pen_style |= PS_SOLID;
-      return;
-    }
-  
-  /* win32_gc->pen_style = PS_COSMETIC; ??? */
-  if (2 == n)
-    {
-      if ((dash_list[0] == dash_list[1]) && (dash_list[0] > 2))
-        {
-          win32_gc->pen_style |= PS_DASH;
-          GDK_NOTE (MISC, g_print("gdk_win32_gc_set_dashes: PS_DASH (%d,%d)\n", 
-                                  dash_list[0], dash_list[1]));
-        }
-      else
-        {
-          win32_gc->pen_style |= PS_DOT;
-          GDK_NOTE (MISC, g_print("gdk_win32_gc_set_dashes: PS_DOT (%d,%d)\n", 
-                                  dash_list[0], dash_list[1]));
-        }
-    }
-  else if (4 == n)
-    {
-      win32_gc->pen_style |= PS_DASHDOT; 
-      GDK_NOTE (MISC, g_print("gdk_win32_gc_set_dashes: PS_DASHDOT (%d,%d,%d,%d)\n", 
-                              dash_list[0], dash_list[1],
-                              dash_list[2], dash_list[3]));
-    }
-  else if (6 == n)
-    {
-      win32_gc->pen_style |= PS_DASHDOTDOT; 
-      GDK_NOTE (MISC, g_print("gdk_win32_gc_set_dashes: PS_DASHDOTDOT (%d,%d,%d,%d,%d,%d)\n", 
-                              dash_list[0], dash_list[1],
-                              dash_list[2], dash_list[3],
-                              dash_list[4], dash_list[5]));
-    }
-  else
-    {
-      win32_gc->pen_style |= PS_DASH;
-      GDK_NOTE (MISC, g_print("gdk_win32_gc_set_dashes: no guess for %d dashes\n", n));
-    }
+
+  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
@@ -697,36 +608,35 @@ gdk_gc_set_clip_rectangle (GdkGC  *gc,
   win32_gc = GDK_GC_WIN32 (gc);
 
   if (win32_gc->hcliprgn)
-    {
-      DeleteObject (win32_gc->hcliprgn);
-      win32_gc->hcliprgn = NULL;
-    }
-
-  if (win32_gc->clip_region)
-    gdk_region_destroy (win32_gc->clip_region);
+    DeleteObject (win32_gc->hcliprgn);
 
   if (rectangle)
     {
-      GDK_NOTE (MISC,
-               g_print ("gdk_gc_set_clip_rectangle: %dx%d@+%d+%d\n",
-                        rectangle->width, rectangle->height,
-                        rectangle->x, rectangle->y));
-      win32_gc->clip_region = gdk_region_rectangle (rectangle);
+      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: NULL\n"));
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_rectangle: NULL\n"));
 
-      win32_gc->clip_region = NULL;
+      win32_gc->hcliprgn = NULL;
       win32_gc->values_mask &= ~GDK_GC_CLIP_MASK;
     }
-    win32_gc->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)
 {
   GdkGCWin32 *win32_gc;
 
@@ -734,30 +644,29 @@ gdk_gc_set_clip_region (GdkGC              *gc,
 
   win32_gc = GDK_GC_WIN32 (gc);
 
-  if (win32_gc->clip_region)
-    gdk_region_destroy (win32_gc->clip_region);
+  if (win32_gc->hcliprgn)
+    DeleteObject (win32_gc->hcliprgn);
 
   if (region)
     {
-      GDK_NOTE (MISC, g_print ("gdk_gc_set_clip_region: %dx%d+%d+%d\n",
-                              region->extents.x2 - region->extents.x1,
-                              region->extents.y2 - region->extents.y1,
-                              region->extents.x1, region->extents.y1));
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_region: %p: %s\n",
+                            win32_gc,
+                            _gdk_win32_gdkregion_to_string (region)));
 
-      win32_gc->clip_region = gdk_region_copy (region);
+      win32_gc->hcliprgn = _gdk_win32_gdkregion_to_hrgn (region, 0, 0);
       win32_gc->values_mask |= GDK_GC_CLIP_MASK;
     }
   else
     {
-      GDK_NOTE (MISC, g_print ("gdk_gc_set_clip_region: NULL\n"));
+      GDK_NOTE (GC, g_print ("gdk_gc_set_clip_region: NULL\n"));
 
-      win32_gc->clip_region = NULL;
+      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);
 }
 
@@ -774,201 +683,213 @@ gdk_gc_copy (GdkGC *dst_gc,
   dst_win32_gc = GDK_GC_WIN32 (dst_gc);
   src_win32_gc = GDK_GC_WIN32 (src_gc);
 
+  GDK_NOTE (GC, g_print ("gdk_gc_copy: %p := %p\n", dst_win32_gc, src_win32_gc));
+
+  if (dst_gc->colormap)
+    g_object_unref (G_OBJECT (dst_gc->colormap));
+
+  if (dst_win32_gc->hcliprgn != NULL)
+    DeleteObject (dst_win32_gc->hcliprgn);
+
   if (dst_win32_gc->font != NULL)
     gdk_font_unref (dst_win32_gc->font);
+
   if (dst_win32_gc->tile != NULL)
     gdk_drawable_unref (dst_win32_gc->tile);
+
   if (dst_win32_gc->stipple != NULL)
     gdk_drawable_unref (dst_win32_gc->stipple);
-  if (dst_win32_gc->clip_region != NULL)
-    gdk_region_destroy (dst_win32_gc->clip_region);
-  if (dst_win32_gc->hcliprgn != NULL)
-    DeleteObject (dst_win32_gc->hcliprgn);
+
+  if (dst_win32_gc->pen_dashes)
+    g_free (dst_win32_gc->pen_dashes);
   
   *dst_win32_gc = *src_win32_gc;
+  dst_win32_gc->hdc = NULL;
+
+  if (dst_gc->colormap)
+    g_object_ref (G_OBJECT (dst_gc->colormap));
 
   if (dst_win32_gc->hcliprgn)
     {
       /* create a new region, to copy to */
-      dst_win32_gc->hcliprgn = CreateRectRgn(0,0,1,1);
+      dst_win32_gc->hcliprgn = CreateRectRgn (0,0,1,1);
       /* overwrite from source */
       CombineRgn (dst_win32_gc->hcliprgn, src_win32_gc->hcliprgn,
                  NULL, RGN_COPY);
     }
-  if (dst_win32_gc->clip_region != NULL)
-    dst_win32_gc->clip_region = gdk_region_copy (dst_win32_gc->clip_region);
+
   if (dst_win32_gc->font != NULL)
     gdk_font_ref (dst_win32_gc->font);
+
   if (dst_win32_gc->tile != NULL)
     gdk_drawable_ref (dst_win32_gc->tile);
+
   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)
+{
+  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_colormap_color (GdkColormap *colormap,
-                   gulong       pixel)
+_gdk_win32_colormap_color (GdkColormap *colormap,
+                           gulong       pixel)
 {
   const GdkVisual *visual;
   GdkColormapPrivateWin32 *colormap_private;
   guchar r, g, b;
 
   if (colormap == NULL)
-    return PALETTEINDEX (pixel);
-
-  colormap_private = GDK_COLORMAP_PRIVATE_DATA (colormap);
+    return DIBINDEX (pixel & 1);
 
-  g_return_val_if_fail(colormap_private != NULL, RGB (0,0,0));
+  colormap_private = GDK_WIN32_COLORMAP_DATA (colormap);
 
-  if (colormap_private->xcolormap->rc_palette)
-    return PALETTEINDEX (pixel);
+  g_assert (colormap_private != NULL);
 
   visual = colormap->visual;
-  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);
+  switch (visual->type)
+    {
+    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;
+    }
 }
 
-static void
+static COLORREF
 predraw_set_foreground (GdkGC       *gc,
                        GdkColormap *colormap,
                        gboolean    *ok)
 {
+  COLORREF fg;
   GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
   GdkColormapPrivateWin32 *colormap_private;
-  COLORREF fg;
-  LOGBRUSH logbrush;
-  HPEN hpen;
-  HBRUSH hbr;
-
-  if (colormap == NULL)
-    {
-      /* A 1 bit deep bitmap */
-      struct
-      {
-       WORD palVersion;
-       WORD palNumEntries;
-       PALETTEENTRY palPalEntry[2];
-      } logpal;
-      static HPALETTE hpal = NULL;
+  gint k;
 
-      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_GDI_FAILED ("CreatePalette"), *ok = FALSE;
-       }
-      SelectPalette (win32_gc->hdc, hpal, FALSE);
-      RealizePalette (win32_gc->hdc);
-      fg = PALETTEINDEX (win32_gc->foreground);
-    }
-  else
+  if (colormap &&
+      (colormap->visual->type == GDK_VISUAL_PSEUDO_COLOR ||
+       colormap->visual->type == GDK_VISUAL_STATIC_COLOR))
     {
-      colormap_private = GDK_COLORMAP_PRIVATE_DATA (colormap);
-      /* maybe this should bail out with g_return_if_fail
-       * (colormap_private != NULL) ?
-       */
-      if (colormap_private && colormap_private->xcolormap->rc_palette)
-       {
-         int k;
-         if (SelectPalette (win32_gc->hdc, colormap_private->xcolormap->palette,
-                            FALSE) == NULL)
-           WIN32_GDI_FAILED ("SelectPalette"), *ok = FALSE;
-         if (TRUE || colormap_private->xcolormap->stale)
-           {
-             if ((k = RealizePalette (win32_gc->hdc)) == GDI_ERROR)
-               WIN32_GDI_FAILED ("RealizePalette"), *ok = FALSE;
-             colormap_private->xcolormap->stale = FALSE;
-           }
-#if 0
-         g_print ("Selected palette %#x for gc %#x, realized %d colors\n",
-                  colormap_private->xcolormap->palette, win32_gc->hdc, k);
-#endif
-       }
-    }
+      colormap_private = GDK_WIN32_COLORMAP_DATA (colormap);
 
-  fg = gdk_colormap_color (colormap, win32_gc->foreground);
+      g_assert (colormap_private != NULL);
 
-  if (SetTextColor (win32_gc->hdc, fg) == CLR_INVALID)
-    WIN32_GDI_FAILED ("SetTextColor"), *ok = FALSE;
-
-  /* Create and select pen and brush. */
+      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));
+    }
 
-  logbrush.lbStyle = BS_SOLID;
-  logbrush.lbColor = fg;
+  fg = _gdk_win32_colormap_color (colormap, win32_gc->foreground);
 
-  if (*ok && (hpen = ExtCreatePen (win32_gc->pen_style,
-                                  (win32_gc->pen_width > 0 ? win32_gc->pen_width : 1),
-                                  &logbrush, 0, NULL)) == NULL)
-    WIN32_GDI_FAILED ("ExtCreatePen");
-  
-  if (SelectObject (win32_gc->hdc, hpen) == NULL)
-    WIN32_GDI_FAILED ("SelectObject"), *ok = FALSE;
-
-  switch (win32_gc->fill_style)
-    {
-    case GDK_OPAQUE_STIPPLED:
-      if (*ok && (hbr = CreatePatternBrush (GDK_PIXMAP_HBITMAP (win32_gc->stipple))) == NULL)
-       WIN32_GDI_FAILED ("CreatePatternBrush"), *ok = FALSE;
-       
-      if (*ok && !SetBrushOrgEx(win32_gc->hdc, gc->ts_x_origin,
-                               gc->ts_y_origin, NULL))
-       WIN32_GDI_FAILED ("SetBrushOrgEx"), *ok = FALSE;
-      break;
-
-    case GDK_SOLID:
-    default:
-      if (*ok && (hbr = CreateSolidBrush (fg)) == NULL)
-       WIN32_GDI_FAILED ("CreateSolidBrush"), *ok = FALSE;
-      break;
-  }
-  if (*ok && SelectObject (win32_gc->hdc, hbr) == NULL)
-    WIN32_GDI_FAILED ("SelectObject"), *ok = FALSE;
-}  
-
-void
-predraw_set_background (GdkGC       *gc,
-                       GdkColormap *colormap,
-                       gboolean    *ok)
-{
-  GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
-  COLORREF bg = gdk_colormap_color (colormap, win32_gc->background);
-
-  if (SetBkColor (win32_gc->hdc, bg) == CLR_INVALID)
-    WIN32_GDI_FAILED ("SetBkColor"), *ok = FALSE;
+  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_win32_hdc_get (GdkDrawable    *drawable,
                   GdkGC          *gc,
                   GdkGCValuesMask usage)
 {
   GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
+  GdkDrawableImplWin32 *impl = NULL;
   gboolean ok = TRUE;
-  int flag;
+  COLORREF fg = RGB (0, 0, 0);
+  LOGBRUSH logbrush;
+  HPEN hpen;
+  HBRUSH hbr;
 
   g_assert (win32_gc->hdc == NULL);
 
-  win32_gc->hwnd = GDK_DRAWABLE_IMPL_WIN32(drawable)->handle;
+  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 (drawable))
+  if (GDK_IS_PIXMAP_IMPL_WIN32 (impl))
     {
       if ((win32_gc->hdc = CreateCompatibleDC (NULL)) == NULL)
        WIN32_GDI_FAILED ("CreateCompatibleDC"), ok = FALSE;
@@ -989,370 +910,181 @@ gdk_win32_hdc_get (GdkDrawable    *drawable,
     }
   
   if (ok && (usage & GDK_GC_FOREGROUND))
-    predraw_set_foreground (gc, GDK_DRAWABLE_IMPL_WIN32 (drawable)->colormap, &ok);
+    {
+      fg = predraw_set_foreground (gc, impl->colormap, &ok);
+      if (ok && (hbr = CreateSolidBrush (fg)) == NULL)
+       WIN32_GDI_FAILED ("CreateSolidBrush"), ok = FALSE;
+
+      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_BACKGROUND)
-      && (win32_gc->values_mask & GDK_GC_BACKGROUND))
-    predraw_set_background (gc, GDK_DRAWABLE_IMPL_WIN32 (drawable)->colormap, &ok);
-  
   if (ok && (usage & GDK_GC_FONT))
     {
       if (SetBkMode (win32_gc->hdc, TRANSPARENT) == 0)
        WIN32_GDI_FAILED ("SetBkMode"), ok = FALSE;
   
-      if (ok && SetTextAlign (win32_gc->hdc, TA_BASELINE) == GDI_ERROR)
+      if (ok && SetTextAlign (win32_gc->hdc, TA_BASELINE|TA_LEFT|TA_NOUPDATECP) == GDI_ERROR)
        WIN32_GDI_FAILED ("SetTextAlign"), ok = FALSE;
     }
   
-  if (ok && (win32_gc->values_mask & GDK_GC_FUNCTION))
+  if (ok && win32_gc->rop2 != R2_COPYPEN)
     if (SetROP2 (win32_gc->hdc, win32_gc->rop2) == 0)
       WIN32_GDI_FAILED ("SetROP2"), ok = FALSE;
 
-  if (win32_gc->values_mask & GDK_GC_CLIP_MASK)
-    g_assert ((win32_gc->clip_region != NULL) != (win32_gc->hcliprgn != NULL));
-
-  if (ok
-      && (win32_gc->values_mask & GDK_GC_CLIP_MASK)
-      && win32_gc->clip_region != NULL)
-    {
-      HRGN hrgn;
-      RGNDATA *rgndata;
-      RECT *rect;
-      GdkRegionBox *boxes = win32_gc->clip_region->rects;
-      guint nbytes =
-       sizeof (RGNDATAHEADER) + (sizeof (RECT) * win32_gc->clip_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 < win32_gc->clip_region->numRects; i++)
-       {
-         rect = ((RECT *) rgndata->Buffer) + rgndata->rdh.nCount++;
-
-         rect->left = CLAMP (boxes[i].x1 + gc->clip_x_origin,
-                             G_MINSHORT, G_MAXSHORT);
-         rect->right = CLAMP (boxes[i].x2 + gc->clip_x_origin,
-                              G_MINSHORT, G_MAXSHORT);
-         rect->top = CLAMP (boxes[i].y1 + gc->clip_y_origin,
-                            G_MINSHORT, G_MAXSHORT);
-         rect->bottom = CLAMP (boxes[i].y2 + gc->clip_y_origin,
-                               G_MINSHORT, G_MAXSHORT);
-
-         GDK_NOTE (MISC, g_print ("clip rgn box %d: %ldx%ld@+%ld+%ld\n",
-                                  i,
-                                  rect->right-rect->left,
-                                  rect->bottom-rect->top,
-                                  rect->left, rect->top));
-
-         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"), ok = FALSE;
-
-      if (ok && SelectClipRgn (win32_gc->hdc, hrgn) == ERROR)
-       WIN32_API_FAILED ("SelectClipRgn"), ok = FALSE;
-
-      if (hrgn != NULL)
-       DeleteObject (hrgn);
-    }
-  else if (ok
-          && (win32_gc->values_mask & GDK_GC_CLIP_MASK)
-          && win32_gc->hcliprgn != NULL)
+  if (ok &&
+      (win32_gc->values_mask & GDK_GC_CLIP_MASK) &&
+      win32_gc->hcliprgn != NULL)
     {
       if (SelectClipRgn (win32_gc->hdc, win32_gc->hcliprgn) == ERROR)
        WIN32_API_FAILED ("SelectClipRgn"), ok = FALSE;
-    }
 
-  if (gdk_debug_flags & GDK_DEBUG_MISC)
-    {
-      HGDIOBJ obj;
-      LOGBRUSH logbrush;
-      EXTLOGPEN extlogpen;
-      HRGN hrgn;
-      RECT rect;
-
-      g_print ("gdk_win32_hdc_get: %#x\n", (guint) win32_gc->hdc);
-      obj = GetCurrentObject (win32_gc->hdc, OBJ_BRUSH);
-      GetObject (obj, sizeof (LOGBRUSH), &logbrush);
-      g_print ("brush: style: %s color: %.06lx hatch: %#lx\n",
-              (logbrush.lbStyle == BS_HOLLOW ? "HOLLOW" :
-               (logbrush.lbStyle == BS_PATTERN ? "PATTERN" :
-                (logbrush.lbStyle == BS_SOLID ? "SOLID" :
-                 "???"))),
-              logbrush.lbColor,
-              logbrush.lbHatch);
-      obj = GetCurrentObject (win32_gc->hdc, OBJ_PEN);
-      GetObject (obj, sizeof (EXTLOGPEN), &extlogpen);
-      g_print ("pen: type: %s style: %s endcap: %s join: %s width: %d brush: %s\n",
-              ((extlogpen.elpPenStyle & PS_TYPE_MASK) ==
-               PS_GEOMETRIC ? "GEOMETRIC" : "COSMETIC"),
-              ((extlogpen.elpPenStyle & PS_STYLE_MASK) ==
-               PS_NULL ? "NULL" :
-               ((extlogpen.elpPenStyle & PS_STYLE_MASK) ==
-                PS_SOLID ? "SOLID" : "???")),
-              ((extlogpen.elpPenStyle & PS_ENDCAP_MASK) ==
-               PS_ENDCAP_FLAT ? "FLAT" :
-               ((extlogpen.elpPenStyle & PS_ENDCAP_MASK) ==
-                PS_ENDCAP_ROUND ? "ROUND" :
-                ((extlogpen.elpPenStyle & PS_ENDCAP_MASK) ==
-                 PS_ENDCAP_SQUARE ? "ROUND" :
-                 ((extlogpen.elpPenStyle & PS_ENDCAP_MASK) ==
-                  PS_ENDCAP_SQUARE ? "ROUND" : "???")))),
-              ((extlogpen.elpPenStyle & PS_JOIN_MASK) ==
-               PS_JOIN_BEVEL ? "BEVEL" :
-               ((extlogpen.elpPenStyle & PS_JOIN_MASK) ==
-                PS_JOIN_MITER ? "MITER" :
-                ((extlogpen.elpPenStyle & PS_JOIN_MASK) ==
-                 PS_JOIN_ROUND ? "ROUND" : "???"))),
-              extlogpen.elpWidth,
-              (extlogpen.elpBrushStyle == BS_DIBPATTERN ? "DIBPATTERN" :
-               (extlogpen.elpBrushStyle == BS_DIBPATTERNPT ? "DIBPATTERNPT" :
-                (extlogpen.elpBrushStyle == BS_HATCHED ? "HATCHED" :
-                 (extlogpen.elpBrushStyle == BS_HOLLOW ? "HOLLOW" :
-                  (extlogpen.elpBrushStyle == BS_PATTERN ? "PATTERN" :
-                   (extlogpen.elpBrushStyle == BS_SOLID ? "SOLID" : "???")))))));
-      hrgn = CreateRectRgn (0, 0, 0, 0);
-      if ((flag = GetClipRgn (win32_gc->hdc, hrgn)) == -1)
-       WIN32_API_FAILED ("GetClipRgn");
-      else if (flag == 0)
-       g_print ("no clip region\n");
-      else if (flag == 1)
-       {
-         GetRgnBox (hrgn, &rect);
-         g_print ("clip region bbox: %ldx%ld@+%ld+%ld\n",
-                  rect.right - rect.left,
-                  rect.bottom - rect.top,
-                  rect.left, rect.top);
-       }
+      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;
     }
 
+  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_win32_hdc_release (GdkDrawable    *drawable,
                       GdkGC          *gc,
                       GdkGCValuesMask usage)
 {
   GdkGCWin32 *win32_gc = (GdkGCWin32 *) gc;
+  GdkDrawableImplWin32 *impl = NULL;
   HGDIOBJ hpen = NULL;
   HGDIOBJ hbr = NULL;
 
-  if (usage & GDK_GC_FOREGROUND)
-    {
-      if ((hpen = GetCurrentObject (win32_gc->hdc, OBJ_PEN)) == NULL)
-       WIN32_GDI_FAILED ("GetCurrentObject");
+  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 (win32_gc->hdc, OBJ_BRUSH)) == NULL)
-       WIN32_GDI_FAILED ("GetCurrentObject");
-    }
+  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 (win32_gc->hdc, win32_gc->saved_dc))
-    WIN32_GDI_FAILED ("RestoreDC");
-#if 0
-  if (colormap_private != NULL
-      && colormap_private->xcolormap->rc_palette
-      && colormap_private->xcolormap->stale)
+  if (win32_gc->holdpal != NULL)
     {
-      SelectPalette (win32_gc->hdc, GetStockObject (DEFAULT_PALETTE), FALSE);
-      if (!UnrealizeObject (colormap_private->xcolormap->palette))
-       WIN32_GDI_FAILED ("UnrealizeObject");
-    }
-#endif
-  if (GDK_IS_PIXMAP_IMPL_WIN32 (drawable))
-    {
-      if (!DeleteDC (win32_gc->hdc))
-       WIN32_GDI_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 (win32_gc->hwnd, win32_gc->hdc);
-    }
+    GDI_CALL (ReleaseDC, (win32_gc->hwnd, win32_gc->hdc));
 
   if (hpen != NULL)
-    if (!DeleteObject (hpen))
-      WIN32_GDI_FAILED ("DeleteObject");
+    GDI_CALL (DeleteObject, (hpen));
   
   if (hbr != NULL)
-    if (!DeleteObject (hbr))
-      WIN32_GDI_FAILED ("DeleteObject");
+    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_GDI_FAILED ("CreatePalette");
-    }
-
-  /* Create a memory DC inside which we will scan the bitmap content */
-  hMemDC = CreateCompatibleDC (NULL);
-  if (!hMemDC)
-    {
-      WIN32_GDI_FAILED ("CreateCompatibleDC");
-      return NULL;
-    }
-
-  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_GDI_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_GDI_FAILED ("CreateCompatibleDC");
-      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);
+  GdkImage *image;
+  guchar *p;
+  gint x, y;
 
-  if (!BitBlt (hMemDC, 0, 0, bm.bmWidth, bm.bmHeight, hDC, 0, 0, SRCCOPY))
-    {
-      WIN32_GDI_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
@@ -1368,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++;
            }
          
@@ -1431,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*/
@@ -1448,107 +1174,57 @@ BitmapToRegion (HBITMAP hBmp)
     hRgn = h;
 
   /* Clean up*/
-  SelectObject(hMemDC, holdBmp);
-  DeleteObject (hbm8);
-  DeleteDC (hMemDC);
+  g_free (pData);
 
   return hRgn;
 }
 
-#ifdef G_ENABLE_DEBUG
-
-gchar *
-gdk_win32_cap_style_to_string (GdkCapStyle cap_style)
-{
-  switch (cap_style)
-    {
-#define CASE(x) case x: return #x + strlen ("GDK_CAP_")
-    CASE (GDK_CAP_NOT_LAST);
-    CASE (GDK_CAP_BUTT);
-    CASE (GDK_CAP_ROUND);
-    CASE (GDK_CAP_PROJECTING);
-#undef CASE
-    default: return ("illegal GdkCapStyle value");
-    }
-  /* NOTREACHED */
-  return NULL;
-}
-
-gchar *
-gdk_win32_fill_style_to_string (GdkFill fill)
-{
-  switch (fill)
-    {
-#define CASE(x) case x: return #x + strlen ("GDK_")
-    CASE (GDK_SOLID);
-    CASE (GDK_TILED);
-    CASE (GDK_STIPPLED);
-    CASE (GDK_OPAQUE_STIPPLED);
-#undef CASE
-    default: return ("illegal GdkFill value");
-    }
-  /* NOTREACHED */
-  return NULL;
-}
-
-gchar *
-gdk_win32_function_to_string (GdkFunction function)
+HRGN
+_gdk_win32_gdkregion_to_hrgn (GdkRegion *region,
+                             gint       x_origin,
+                             gint       y_origin)
 {
-  switch (function)
+  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++)
     {
-#define CASE(x) case x: return #x + strlen ("GDK_")
-    CASE (GDK_COPY);
-    CASE (GDK_INVERT);
-    CASE (GDK_XOR);
-    CASE (GDK_CLEAR);
-    CASE (GDK_AND);
-    CASE (GDK_AND_REVERSE);
-    CASE (GDK_AND_INVERT);
-    CASE (GDK_NOOP);
-    CASE (GDK_OR);
-    CASE (GDK_EQUIV);
-    CASE (GDK_OR_REVERSE);
-    CASE (GDK_COPY_INVERT);
-    CASE (GDK_OR_INVERT);
-    CASE (GDK_NAND);
-    CASE (GDK_SET);
-#undef CASE
-    default: return ("illegal GdkFunction value");
+      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;
     }
-  /* NOTREACHED */
-  return NULL; 
-}
+  if ((hrgn = ExtCreateRegion (NULL, nbytes, rgndata)) == NULL)
+    WIN32_API_FAILED ("ExtCreateRegion");
 
-gchar *
-gdk_win32_join_style_to_string (GdkJoinStyle join_style)
-{
-  switch (join_style)
-    {
-#define CASE(x) case x: return #x + strlen ("GDK_JOIN_")
-    CASE (GDK_JOIN_MITER);
-    CASE (GDK_JOIN_ROUND);
-    CASE (GDK_JOIN_BEVEL);
-#undef CASE
-    default: return ("illegal GdkJoinStyle value");
-    }
-  /* NOTREACHED */
-  return NULL; 
-}
+  g_free (rgndata);
 
-gchar *
-gdk_win32_line_style_to_string (GdkLineStyle line_style)
-{
-  switch (line_style)
-    {
-#define CASE(x) case x: return #x + strlen ("GDK_LINE_")
-    CASE(GDK_LINE_SOLID);
-    CASE(GDK_LINE_ON_OFF_DASH);  
-    CASE(GDK_LINE_DOUBLE_DASH);  
-#undef CASE
-    default: return ("illegal GdkLineStyle value");
-    }
-  /* NOTREACHED */
-  return NULL; 
+  return (hrgn);
 }
-
-#endif