commit 366ae1e6c60a3a5c110897666b8193584cacfbfd
author Nikolay Sivov <bunglehead@gmail.com> Sun Aug 24 14:45:18 2008 +0400
committer Alexandre Julliard <julliard@winehq.org> Mon Aug 25 11:38:02 2008 +0200
tree 66b4ecf4b1257307c1bc69d28d2cdc6edc7c993e
parent 4570501155d745bb424511c7564232b63044233c

gdiplus: Basic GdipGetDC/GdipReleaseDC implementation.

dlls/gdiplus/gdiplus_private.h

diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h
index 5a18660..8df1c74 100644
--- a/dlls/gdiplus/gdiplus_private.h
+++ b/dlls/gdiplus/gdiplus_private.h
@@ -95,6 +95,7 @@
     GpUnit unit;    /* page unit */
     REAL scale;     /* page scale */
     GpMatrix * worldtrans; /* world transform */
+    BOOL busy;      /* hdc handle obtained by GdipGetDC */
 };
 
 struct GpBrush{

dlls/gdiplus/graphics.c

diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c
index b852103..7bc0f03 100644
--- a/dlls/gdiplus/graphics.c
+++ b/dlls/gdiplus/graphics.c
@@ -752,6 +752,7 @@
     (*graphics)->compmode = CompositingModeSourceOver;
     (*graphics)->unit = UnitDisplay;
     (*graphics)->scale = 1.0;
+    (*graphics)->busy = FALSE;
 
     return Ok;
 }
@@ -2679,26 +2680,29 @@
 
 GpStatus WINGDIPAPI GdipGetDC(GpGraphics *graphics, HDC *hdc)
 {
-    FIXME("(%p, %p): stub\n", graphics, hdc);
-
     if(!graphics || !hdc)
         return InvalidParameter;
 
-    *hdc = NULL;
-    return NotImplemented;
+    if(graphics->busy)
+        return ObjectBusy;
+
+    *hdc = graphics->hdc;
+    graphics->busy = TRUE;
+
+    return Ok;
 }
 
 GpStatus WINGDIPAPI GdipReleaseDC(GpGraphics *graphics, HDC hdc)
 {
-    FIXME("(%p, %p): stub\n", graphics, hdc);
-
     if(!graphics)
         return InvalidParameter;
 
-    if(graphics->hdc != hdc)
+    if(graphics->hdc != hdc || !(graphics->busy))
         return InvalidParameter;
 
-    return NotImplemented;
+    graphics->busy = FALSE;
+
+    return Ok;
 }
 
 GpStatus WINGDIPAPI GdipGetClip(GpGraphics *graphics, GpRegion *region)

dlls/gdiplus/tests/graphics.c

diff --git a/dlls/gdiplus/tests/graphics.c b/dlls/gdiplus/tests/graphics.c
index f70caf5..b933f34 100644
--- a/dlls/gdiplus/tests/graphics.c
+++ b/dlls/gdiplus/tests/graphics.c
@@ -545,13 +545,17 @@
     status = GdipReleaseDC(NULL, (HDC)0xdeadbeef);
     expect(InvalidParameter, status);
 
+    /* Release without Get */
+    status = GdipReleaseDC(graphics, hdc);
+    expect(InvalidParameter, status);
+
     retdc = NULL;
     status = GdipGetDC(graphics, &retdc);
-    todo_wine expect(Ok, status);
-    todo_wine ok(retdc == hdc, "Invalid HDC returned\n");
+    expect(Ok, status);
+    ok(retdc == hdc, "Invalid HDC returned\n");
     /* call it once more */
     status = GdipGetDC(graphics, &retdc);
-    todo_wine expect(ObjectBusy, status);
+    expect(ObjectBusy, status);
 
     /* try all Graphics calls here */
     status = Ok;
@@ -712,7 +716,7 @@
     todo_wine expect(ObjectBusy, status); status = Ok;
 
     status = GdipReleaseDC(graphics, retdc);
-    todo_wine expect(Ok, status);
+    expect(Ok, status);
 
     GdipDeletePen(pen);
     GdipDeleteGraphics(graphics);