[jimtabor] 72625: [NtGDI] - Fix Path issues, Sync/Port wine. See CORE-4990 (Fill Path painting?), CORE-9612 (Fixed) and CORE-11916 (Fixed). Maybe more. - Pen width might be related to another issue...

Author: jimtabor
Date: Thu Sep  8 22:37:41 2016
New Revision: 72625

URL: http://svn.reactos.org/svn/reactos?rev=72625&view=rev
Log:
[NtGDI]
- Fix Path issues, Sync/Port wine. See CORE-4990 (Fill Path painting?), CORE-9612 (Fixed)
and CORE-11916 (Fixed). Maybe more.
- Pen width might be related to another issue outside Path. See CORE-2527.

Modified:
    trunk/reactos/win32ss/gdi/ntgdi/arc.c
    trunk/reactos/win32ss/gdi/ntgdi/dclife.c
    trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c
    trunk/reactos/win32ss/gdi/ntgdi/dcstate.c
    trunk/reactos/win32ss/gdi/ntgdi/path.c
    trunk/reactos/win32ss/gdi/ntgdi/path.h

Modified: trunk/reactos/win32ss/gdi/ntgdi/arc.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/arc.c?re…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/arc.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/arc.c	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -211,6 +211,7 @@
                YStartArc,
                  XEndArc,
                  YEndArc,
+                       0,
                  arctype);
   }
 

Modified: trunk/reactos/win32ss/gdi/ntgdi/dclife.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dclife.c…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/dclife.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/dclife.c	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -387,8 +387,13 @@
     /* Free CLIPOBJ resources */
     IntEngFreeClipResources(&pdc->co);
 
-    PATH_Delete(pdc->dclevel.hPath);
-
+    if (pdc->dclevel.hPath)
+    {
+       DPRINT("DC_vCleanup Path\n");
+       PATH_Delete(pdc->dclevel.hPath);
+       pdc->dclevel.hPath = 0;
+       pdc->dclevel.flPath = 0;
+    }
     if(pdc->dclevel.pSurface)
         SURFACE_ShareUnlockSurface(pdc->dclevel.pSurface);
 

Modified: trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -496,7 +496,7 @@
     int Mode)
 {
     PREGION  RgnPath;
-    PPATH pPath;
+    PPATH pPath, pNewPath;
     BOOL  success = FALSE;
     PDC_ATTR pdcattr;
     PDC pdc;
@@ -520,8 +520,8 @@
     if (pPath->state != PATH_Closed)
     {
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
-        DC_UnlockDc(pdc);
-        return FALSE;
+        success = FALSE;
+        goto Exit;
     }
 
     /* Construct a region from the path */
@@ -533,24 +533,23 @@
         return FALSE;
     }
 
-    if (!PATH_PathToRegion(pPath, pdcattr->jFillMode, RgnPath))
-    {
-        EngSetLastError(ERROR_CAN_NOT_COMPLETE);
-        REGION_Delete(RgnPath);
-        DC_UnlockDc(pdc);
-        return FALSE;
-    }
-
-    success = IntGdiExtSelectClipRgn(pdc, RgnPath, Mode) != ERROR;
+    pNewPath = PATH_FlattenPath(pPath);
+
+    success = PATH_PathToRegion(pNewPath, pdcattr->jFillMode, RgnPath);
+
+    PATH_UnlockPath(pNewPath);
+    PATH_Delete(pNewPath->BaseObject.hHmgr);
+
+    if (success) success = IntGdiExtSelectClipRgn(pdc, RgnPath, Mode) != ERROR;
+
     REGION_Delete(RgnPath);
 
-    /* Empty the path */
-    if (success)
-        PATH_EmptyPath(pPath);
-
-    /* FIXME: Should this function delete the path even if it failed? */
-
+Exit:
     PATH_UnlockPath(pPath);
+    PATH_Delete(pdc->dclevel.hPath);
+    pdc->dclevel.flPath &= ~DCPATH_ACTIVE;
+    pdc->dclevel.hPath = NULL;
+
     DC_UnlockDc(pdc);
 
     return success;

Modified: trunk/reactos/win32ss/gdi/ntgdi/dcstate.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcstate.…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/dcstate.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/dcstate.c	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -88,6 +88,18 @@
         DC_vUpdateTextBrush(dc);
     }
 
+    // Remove Path and reset flags.
+    if (dc->dclevel.hPath)
+    {
+        DPRINT("Clean DC Remove Path\n");
+        if (!PATH_Delete(dc->dclevel.hPath))
+        {
+           DPRINT1("Failed to remove Path\n");
+        }
+        dc->dclevel.hPath = 0;
+        dc->dclevel.flPath = 0;
+    }
+
     /* DC_vCopyState frees the Clip rgn and the Meta rgn. Take care of the other ones
      * There is no need to clear prgnVis, as UserGetDC updates it immediately. */
     if (dc->prgnRao)
@@ -175,13 +187,9 @@
             if (pdc->dctype == DCTYPE_MEMORY)
                 DC_vSelectSurface(pdc, pdcSave->dclevel.pSurface);
 
-            // Restore Path by removing it, if the Save flag is set.
-            // BeginPath will takecare of the rest.
-            if (pdc->dclevel.hPath && pdc->dclevel.flPath &
DCPATH_SAVE)
+            if (pdcSave->dclevel.hPath)
             {
-                PATH_Delete(pdc->dclevel.hPath);
-                pdc->dclevel.hPath = 0;
-                pdc->dclevel.flPath &= ~DCPATH_SAVE;
+               PATH_RestorePath( pdc, pdcSave );
             }
         }
 
@@ -294,10 +302,11 @@
         DC_vSelectSurface(pdcSave, pdc->dclevel.pSurface);
 
     /* Copy path */
-    /* FIXME: Why this way? */
-    pdcSave->dclevel.hPath = pdc->dclevel.hPath;
+    if (pdc->dclevel.hPath)
+    {
+       PATH_SavePath( pdcSave, pdc );
+    }
     pdcSave->dclevel.flPath = pdc->dclevel.flPath | DCPATH_SAVESTATE;
-    if (pdcSave->dclevel.hPath) pdcSave->dclevel.flPath |= DCPATH_SAVE;
 
     /* Set new dc as save dc */
     pdcSave->dclevel.hdcSave = pdc->dclevel.hdcSave;

Modified: trunk/reactos/win32ss/gdi/ntgdi/path.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/path.c?r…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/path.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/path.c	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -19,12 +19,57 @@
 #endif
 
 #define NUM_ENTRIES_INITIAL 16  /* Initial size of points / flags arrays  */
+
 #define GROW_FACTOR_NUMER    2  /* Numerator of grow factor for the array */
 #define GROW_FACTOR_DENOM    1  /* Denominator of grow factor             */
+
+#if DBG
+static int PathCount = 0;
+#endif
 
 /***********************************************************************
  * Internal functions
  */
+
+PPATH FASTCALL
+PATH_CreatePath(int count)
+{
+    PPATH pPath = PATH_AllocPathWithHandle();
+
+    if (!pPath)
+    {
+        EngSetLastError(ERROR_NOT_ENOUGH_MEMORY);
+        return NULL;
+    }
+
+    DPRINT("CreatePath p 0x%p\n", pPath);
+    // Path handles are shared. Also due to recursion with in the same thread.
+    GDIOBJ_vUnlockObject((POBJ)pPath);       // Unlock
+    pPath = PATH_LockPath(pPath->BaseObject.hHmgr); // Share Lock.
+
+    /* Make sure that path is empty */
+    PATH_EmptyPath(pPath);
+
+    count = max( NUM_ENTRIES_INITIAL, count );
+
+    pPath->numEntriesAllocated = count;
+
+    pPath->pPoints = (POINT *)ExAllocatePoolWithTag(PagedPool, count * sizeof(POINT),
TAG_PATH);
+    RtlZeroMemory( pPath->pPoints, count * sizeof(POINT));
+    pPath->pFlags  =  (BYTE *)ExAllocatePoolWithTag(PagedPool, count * sizeof(BYTE), 
TAG_PATH);
+    RtlZeroMemory( pPath->pFlags, count * sizeof(BYTE));
+
+    /* Initialize variables for new path */
+    pPath->numEntriesUsed = 0;
+    pPath->newStroke = TRUE;
+    pPath->state     = PATH_Open;
+    pPath->pos.x = pPath->pos.y = 0;
+#if DBG
+    PathCount++;
+    DPRINT("Create Path %d\n",PathCount);
+#endif
+    return pPath;
+}
 
 /* PATH_DestroyGdiPath
  *
@@ -50,6 +95,10 @@
     if (!pPath) return FALSE;
     PATH_DestroyGdiPath(pPath);
     GDIOBJ_vDeleteObject(&pPath->BaseObject);
+#if DBG
+    PathCount--;
+    DPRINT("Delete Path %d\n",PathCount);
+#endif
     return TRUE;
 }
 
@@ -82,97 +131,6 @@
 
     XFORMOBJ_vInit(&xo, &pdc->pdcattr->mxDeviceToWorld);
     return XFORMOBJ_bApplyXform(&xo, XF_LTOL, count, (PPOINTL)ppt, (PPOINTL)ppt);
-}
-
-/* PATH_FillPath
- *
- *
- */
-BOOL
-FASTCALL
-PATH_FillPath(
-    PDC dc,
-    PPATH pPath)
-{
-    //INT   mapMode, graphicsMode;
-    //SIZE  ptViewportExt, ptWindowExt;
-    //POINTL ptViewportOrg, ptWindowOrg;
-    XFORM xform;
-    PREGION  Rgn;
-    PDC_ATTR pdcattr = dc->pdcattr;
-
-    if (pPath->state != PATH_Closed)
-    {
-        EngSetLastError(ERROR_CAN_NOT_COMPLETE);
-        return FALSE;
-    }
-
-    /* Allocate a temporary region */
-    Rgn = IntSysCreateRectpRgn(0, 0, 0, 0);
-    if (!Rgn)
-    {
-        EngSetLastError(ERROR_NOT_ENOUGH_MEMORY);
-        return FALSE;
-    }
-
-    if (!PATH_PathToRegion(pPath, pdcattr->jFillMode, Rgn))
-    {
-        /* EngSetLastError ? */
-        REGION_Delete(Rgn);
-        return FALSE;
-    }
-
-    /* Since PaintRgn interprets the region as being in logical coordinates
-     * but the points we store for the path are already in device
-     * coordinates, we have to set the mapping mode to MM_TEXT temporarily.
-     * Using SaveDC to save information about the mapping mode / world
-     * transform would be easier but would require more overhead, especially
-     * now that SaveDC saves the current path.
-     */
-
-    /* Save the information about the old mapping mode */
-    //mapMode = pdcattr->iMapMode;
-    //ptViewportExt = pdcattr->szlViewportExt;
-    //ptViewportOrg = pdcattr->ptlViewportOrg;
-    //ptWindowExt   = pdcattr->szlWindowExt;
-    //ptWindowOrg   = pdcattr->ptlWindowOrg;
-
-    /* Save world transform
-     * NB: The Windows documentation on world transforms would lead one to
-     * believe that this has to be done only in GM_ADVANCED; however, my
-     * tests show that resetting the graphics mode to GM_COMPATIBLE does
-     * not reset the world transform.
-     */
-    MatrixS2XForm(&xform, &dc->pdcattr->mxWorldToPage);
-
-    /* Set MM_TEXT */
-//    IntGdiSetMapMode(dc, MM_TEXT);
-//    pdcattr->ptlViewportOrg.x = 0;
-//    pdcattr->ptlViewportOrg.y = 0;
-//    pdcattr->ptlWindowOrg.x = 0;
-//    pdcattr->ptlWindowOrg.y = 0;
-
-    // graphicsMode = pdcattr->iGraphicsMode;
-//    pdcattr->iGraphicsMode = GM_ADVANCED;
-//    IntGdiModifyWorldTransform(dc, &xform, MWT_IDENTITY);
-//    pdcattr->iGraphicsMode =  graphicsMode;
-
-    /* Paint the region */
-    IntGdiPaintRgn(dc, Rgn);
-    REGION_Delete(Rgn);
-    /* Restore the old mapping mode */
-//    IntGdiSetMapMode(dc, mapMode);
-//    pdcattr->szlViewportExt = ptViewportExt;
-//    pdcattr->ptlViewportOrg = ptViewportOrg;
-//    pdcattr->szlWindowExt   = ptWindowExt;
-//    pdcattr->ptlWindowOrg   = ptWindowOrg;
-
-    /* Go to GM_ADVANCED temporarily to restore the world transform */
-    //graphicsMode = pdcattr->iGraphicsMode;
-//    pdcattr->iGraphicsMode = GM_ADVANCED;
-//    IntGdiModifyWorldTransform(dc, &xform, MWT_MAX+1);
-//    pdcattr->iGraphicsMode = graphicsMode;
-    return TRUE;
 }
 
 /* PATH_InitGdiPath
@@ -216,14 +174,358 @@
         return FALSE;
 
     /* Perform the copy operation */
-    memcpy(pPathDest->pPoints, pPathSrc->pPoints,
-           sizeof(POINT)*pPathSrc->numEntriesUsed);
-    memcpy(pPathDest->pFlags, pPathSrc->pFlags,
-           sizeof(BYTE)*pPathSrc->numEntriesUsed);
-
+    memcpy(pPathDest->pPoints, pPathSrc->pPoints,
sizeof(POINT)*pPathSrc->numEntriesUsed);
+    memcpy(pPathDest->pFlags,  pPathSrc->pFlags, 
sizeof(BYTE)*pPathSrc->numEntriesUsed);
+
+    pPathDest->pos = pPathSrc->pos;
     pPathDest->state = pPathSrc->state;
     pPathDest->numEntriesUsed = pPathSrc->numEntriesUsed;
     pPathDest->newStroke = pPathSrc->newStroke;
+    return TRUE;
+}
+
+BOOL PATH_SavePath( DC *dst, DC *src )
+{
+    PPATH pdstPath, psrcPath = PATH_LockPath(src->dclevel.hPath);
+    DPRINT("PATH_SavePath\n");
+    if (psrcPath)
+    {
+       DPRINT("PATH_SavePath 1\n");
+
+       pdstPath = PATH_CreatePath(psrcPath->numEntriesAllocated);
+
+       dst->dclevel.flPath = src->dclevel.flPath;
+
+       dst->dclevel.hPath = pdstPath->BaseObject.hHmgr;
+
+       PATH_AssignGdiPath(pdstPath, psrcPath);
+
+       PATH_UnlockPath(pdstPath);
+       PATH_UnlockPath(psrcPath);
+    }
+    return TRUE;
+}
+
+BOOL PATH_RestorePath( DC *dst, DC *src )
+{
+    DPRINT("PATH_RestorePath\n");
+
+    if (dst->dclevel.hPath == NULL)
+    {
+       PPATH pdstPath, psrcPath = PATH_LockPath(src->dclevel.hPath);
+       DPRINT("PATH_RestorePath 1\n");
+       pdstPath = PATH_CreatePath(psrcPath->numEntriesAllocated);
+       dst->dclevel.flPath = src->dclevel.flPath;
+       dst->dclevel.hPath = pdstPath->BaseObject.hHmgr;
+
+       PATH_AssignGdiPath(pdstPath, psrcPath);
+
+       PATH_UnlockPath(pdstPath);
+       PATH_UnlockPath(psrcPath);       
+    }
+    else
+    {
+       PPATH pdstPath, psrcPath = PATH_LockPath(src->dclevel.hPath);
+       pdstPath = PATH_LockPath(dst->dclevel.hPath);
+       DPRINT("PATH_RestorePath 2\n");
+       dst->dclevel.flPath = src->dclevel.flPath &
(DCPATH_CLOCKWISE|DCPATH_ACTIVE);
+       PATH_AssignGdiPath(pdstPath, psrcPath);
+
+       PATH_UnlockPath(pdstPath);
+       PATH_UnlockPath(psrcPath);
+    }
+    return TRUE;
+}
+
+/* PATH_EmptyPath
+ *
+ * Removes all entries from the path and sets the path state to PATH_Null.
+ */
+VOID
+FASTCALL
+PATH_EmptyPath(PPATH pPath)
+{
+    ASSERT(pPath != NULL);
+
+    pPath->state = PATH_Null;
+    pPath->numEntriesUsed = 0;
+}
+
+/* PATH_AddEntry
+ *
+ * Adds an entry to the path. For "flags", pass either PT_MOVETO, PT_LINETO
+ * or PT_BEZIERTO, optionally ORed with PT_CLOSEFIGURE. Returns TRUE if
+ * successful, FALSE otherwise (e.g. if not enough memory was available).
+ */
+BOOL
+FASTCALL
+PATH_AddEntry(
+    PPATH pPath,
+    const POINT *pPoint,
+    BYTE flags)
+{
+    ASSERT(pPath != NULL);
+
+    /* FIXME: If newStroke is true, perhaps we want to check that we're
+     * getting a PT_MOVETO
+     */
+    DPRINT("(%d,%d) - %d\n", pPoint->x, pPoint->y, flags);
+
+    /* Reserve enough memory for an extra path entry */
+    if (!PATH_ReserveEntries(pPath, pPath->numEntriesUsed + 1))
+        return FALSE;
+
+    /* Store information in path entry */
+    pPath->pPoints[pPath->numEntriesUsed] = *pPoint;
+    pPath->pFlags[pPath->numEntriesUsed] = flags;
+
+    /* Increment entry count */
+    pPath->numEntriesUsed++;
+
+    return TRUE;
+}
+
+/* PATH_ReserveEntries
+ *
+ * Ensures that at least "numEntries" entries (for points and flags) have
+ * been allocated; allocates larger arrays and copies the existing entries
+ * to those arrays, if necessary. Returns TRUE if successful, else FALSE.
+ */
+BOOL
+FASTCALL
+PATH_ReserveEntries(
+    PPATH pPath,
+    INT numEntries)
+{
+    INT numEntriesToAllocate;
+    POINT *pPointsNew;
+    BYTE *pFlagsNew;
+
+    ASSERT(pPath != NULL);
+    ASSERT(numEntries >= 0);
+
+    /* Do we have to allocate more memory? */
+    if (numEntries > pPath->numEntriesAllocated)
+    {
+        /* Find number of entries to allocate. We let the size of the array
+         * grow exponentially, since that will guarantee linear time
+         * complexity. */
+        if (pPath->numEntriesAllocated)
+        {
+            numEntriesToAllocate = pPath->numEntriesAllocated;
+            while (numEntriesToAllocate < numEntries)
+                numEntriesToAllocate = numEntriesToAllocate * GROW_FACTOR_NUMER /
GROW_FACTOR_DENOM;
+        }
+        else
+            numEntriesToAllocate = numEntries;
+
+        /* Allocate new arrays */
+        pPointsNew = (POINT *)ExAllocatePoolWithTag(PagedPool, numEntriesToAllocate *
sizeof(POINT), TAG_PATH);
+        if (!pPointsNew)
+            return FALSE;
+
+        pFlagsNew = (BYTE *)ExAllocatePoolWithTag(PagedPool, numEntriesToAllocate *
sizeof(BYTE), TAG_PATH);
+        if (!pFlagsNew)
+        {
+            ExFreePoolWithTag(pPointsNew, TAG_PATH);
+            return FALSE;
+        }
+
+        /* Copy old arrays to new arrays and discard old arrays */
+        if (pPath->pPoints)
+        {
+            ASSERT(pPath->pFlags);
+
+            memcpy(pPointsNew, pPath->pPoints,
sizeof(POINT)*pPath->numEntriesUsed);
+            memcpy(pFlagsNew, pPath->pFlags, sizeof(BYTE)*pPath->numEntriesUsed);
+
+            ExFreePoolWithTag(pPath->pPoints, TAG_PATH);
+            ExFreePoolWithTag(pPath->pFlags, TAG_PATH);
+        }
+
+        pPath->pPoints = pPointsNew;
+        pPath->pFlags = pFlagsNew;
+        pPath->numEntriesAllocated = numEntriesToAllocate;
+    }
+
+    return TRUE;
+}
+
+/* PATH_ScaleNormalizedPoint
+ *
+ * Scales a normalized point (x, y) with respect to the box whose corners are
+ * passed in "corners". The point is stored in "*pPoint". The
normalized
+ * coordinates (-1.0, -1.0) correspond to corners[0], the coordinates
+ * (1.0, 1.0) correspond to corners[1].
+ */
+VOID
+FASTCALL
+PATH_ScaleNormalizedPoint(
+    FLOAT_POINT corners[],
+    double x,
+    double y,
+    POINT *pPoint)
+{
+    ASSERT(corners);
+    ASSERT(pPoint);
+
+    pPoint->x = GDI_ROUND((double)corners[0].x + (double)(corners[1].x - corners[0].x)
* 0.5 * (x + 1.0));
+    pPoint->y = GDI_ROUND((double)corners[0].y + (double)(corners[1].y - corners[0].y)
* 0.5 * (y + 1.0));
+}
+
+/* PATH_NormalizePoint
+ *
+ * Normalizes a point with respect to the box whose corners are passed in
+ * corners. The normalized coordinates are stored in *pX and *pY.
+ */
+VOID
+FASTCALL
+PATH_NormalizePoint(
+    FLOAT_POINT corners[],
+    const FLOAT_POINT *pPoint,
+    double *pX,
+    double *pY)
+{
+    ASSERT(corners);
+    ASSERT(pPoint);
+    ASSERT(pX);
+    ASSERT(pY);
+
+    *pX = (double)(pPoint->x - corners[0].x) / (double)(corners[1].x - corners[0].x) *
2.0 - 1.0;
+    *pY = (double)(pPoint->y - corners[0].y) / (double)(corners[1].y - corners[0].y) *
2.0 - 1.0;
+}
+
+/* PATH_CheckCorners
+ *
+ * Helper function for PATH_RoundRect() and PATH_Rectangle()
+ */
+BOOL
+PATH_CheckCorners(
+    DC *dc,
+    POINT corners[],
+    INT x1,
+    INT y1,
+    INT x2,
+    INT y2)
+{
+    INT temp;
+    PDC_ATTR pdcattr = dc->pdcattr;
+
+    /* Convert points to device coordinates */
+    corners[0].x = x1;
+    corners[0].y = y1;
+    corners[1].x = x2;
+    corners[1].y = y2;
+    IntLPtoDP(dc, corners, 2);
+
+    /* Make sure first corner is top left and second corner is bottom right */
+    if (corners[0].x > corners[1].x)
+    {
+        temp = corners[0].x;
+        corners[0].x = corners[1].x;
+        corners[1].x = temp;
+    }
+
+    if (corners[0].y > corners[1].y)
+    {
+        temp = corners[0].y;
+        corners[0].y = corners[1].y;
+        corners[1].y = temp;
+    }
+
+    /* In GM_COMPATIBLE, don't include bottom and right edges */
+    if (pdcattr->iGraphicsMode == GM_COMPATIBLE)
+    {
+        if (corners[0].x == corners[1].x) {DPRINT1("PATH_CheckCorners GM_COMPATIBLE
x\n"); return FALSE;}
+        if (corners[0].y == corners[1].y) {DPRINT1("PATH_CheckCorners GM_COMPATIBLE
y\n"); return FALSE;}
+        corners[1].x--;
+        corners[1].y--;
+    }
+    return TRUE;
+}
+
+/* add a number of points, converting them to device coords */
+/* return a pointer to the first type byte so it can be fixed up if necessary */
+static BYTE *add_log_points( DC *dc, PPATH path, const POINT *points,
+                             DWORD count, BYTE type )
+{
+    BYTE *ret;
+
+    if (!PATH_ReserveEntries( path, path->numEntriesUsed + count )) return NULL;
+
+    ret = &path->pFlags[path->numEntriesUsed];
+    memcpy( &path->pPoints[path->numEntriesUsed], points, count *
sizeof(*points) );
+    IntLPtoDP( dc, &path->pPoints[path->numEntriesUsed], count );
+    memset( ret, type, count );
+    path->numEntriesUsed += count;
+    return ret;
+}
+
+/* add a number of points that are already in device coords */
+/* return a pointer to the first type byte so it can be fixed up if necessary */
+static BYTE *add_points( PPATH path, const POINT *points, DWORD count, BYTE type )
+{
+    BYTE *ret;
+
+    if (!PATH_ReserveEntries( path, path->numEntriesUsed + count )) return NULL;
+
+    ret = &path->pFlags[path->numEntriesUsed];
+    memcpy( &path->pPoints[path->numEntriesUsed], points, count *
sizeof(*points) );
+    memset( ret, type, count );
+    path->numEntriesUsed += count;
+    return ret;
+}
+
+/* reverse the order of an array of points */
+static void reverse_points( POINT *points, UINT count )
+{
+    UINT i;
+    for (i = 0; i < count / 2; i++)
+    {
+        POINT pt = points[i];
+        points[i] = points[count - i - 1];
+        points[count - i - 1] = pt;
+    }
+}
+
+/* start a new path stroke if necessary */
+static BOOL start_new_stroke( PPATH path )
+{
+    if (!path->newStroke && path->numEntriesUsed &&
+        !(path->pFlags[path->numEntriesUsed - 1] & PT_CLOSEFIGURE) &&
+        path->pPoints[path->numEntriesUsed - 1].x == path->pos.x &&
+        path->pPoints[path->numEntriesUsed - 1].y == path->pos.y)
+        return TRUE;
+
+    path->newStroke = FALSE;
+    return add_points( path, &path->pos, 1, PT_MOVETO ) != NULL;
+}
+
+/* set current position to the last point that was added to the path */
+static void update_current_pos( PPATH path )
+{
+    assert( path->numEntriesUsed );
+    path->pos = path->pPoints[path->numEntriesUsed - 1];
+}
+
+/* close the current figure */
+static void close_figure( PPATH path )
+{
+    assert( path->numEntriesUsed );
+    path->pFlags[path->numEntriesUsed - 1] |= PT_CLOSEFIGURE;
+}
+
+/* add a number of points, starting a new stroke if necessary */
+static BOOL add_log_points_new_stroke( DC *dc, PPATH path, const POINT *points,
+                                       DWORD count, BYTE type )
+{
+    if (!start_new_stroke( path )) return FALSE;
+    if (!add_log_points( dc, path, points, count, type )) return FALSE;
+    update_current_pos( path );
+
+    DPRINT("ALPNS : Pos X %d Y %d\n",path->pos.x, path->pos.y);
+    IntGdiMoveToEx(dc, path->pos.x, path->pos.y, NULL);
+
     return TRUE;
 }
 
@@ -236,22 +538,23 @@
 BOOL
 FASTCALL
 PATH_MoveTo(
-    PDC dc)
-{
-    PPATH pPath = PATH_LockPath(dc->dclevel.hPath);
+    PDC dc,
+    PPATH pPath)
+{
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        /* FIXME: Do we have to call SetLastError? */
-        return FALSE;
-    }
-    /* Start a new stroke */
-    pPath->newStroke = TRUE;
-    PATH_UnlockPath(pPath);
-    return TRUE;
+    // GDI32 : Signal from user space of a change in position.
+    if (dc->pdcattr->ulDirty_ & DIRTY_STYLESTATE)
+    {
+       DPRINT("MoveTo has changed\n");
+       pPath->newStroke = TRUE;
+       // Set position and clear the signal flag.
+       IntGetCurrentPositionEx(dc, &pPath->pos);
+       IntLPtoDP( dc, &pPath->pos, 1 );
+       return TRUE;
+    }
+
+    return FALSE;
 }
 
 /* PATH_LineTo
@@ -275,33 +578,27 @@
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    /* Convert point to device coordinates */
     point.x = x;
     point.y = y;
-    CoordLPtoDP(dc, &point);
-
-    /* Add a PT_MOVETO if necessary */
-    if (pPath->newStroke)
-    {
-        pPath->newStroke = FALSE;
-        IntGetCurrentPositionEx(dc, &pointCurPos);
-        CoordLPtoDP(dc, &pointCurPos);
-        if (!PATH_AddEntry(pPath, &pointCurPos, PT_MOVETO))
-        {
-            PATH_UnlockPath(pPath);
-            return FALSE;
-        }
-    }
-
-    /* Add a PT_LINETO entry */
-    Ret = PATH_AddEntry(pPath, &point, PT_LINETO);
+
+    // Coalesce a MoveTo point.
+    if ( !PATH_MoveTo(dc, pPath) )
+    {
+       /* Add a PT_MOVETO if necessary */
+       if (pPath->newStroke)
+       {
+           DPRINT("Line To : New Stroke\n");
+           pPath->newStroke = FALSE;
+           IntGetCurrentPositionEx(dc, &pointCurPos);
+           CoordLPtoDP(dc, &pointCurPos);
+           if (!PATH_AddEntry(pPath, &pointCurPos, PT_MOVETO))
+           {
+               PATH_UnlockPath(pPath);
+               return FALSE;
+           }
+       }
+    }
+    Ret = add_log_points_new_stroke( dc, pPath, &point, 1, PT_LINETO );
     PATH_UnlockPath(pPath);
     return Ret;
 }
@@ -321,75 +618,33 @@
     INT y2)
 {
     PPATH pPath;
-    POINT corners[2], pointTemp;
-    INT   temp;
+    POINT corners[2], points[4];
+    BYTE *type;
 
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
+    if (!PATH_CheckCorners(dc, corners, x1, y1, x2, y2))
     {
         PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    /* Convert points to device coordinates */
-    corners[0].x = x1;
-    corners[0].y = y1;
-    corners[1].x = x2;
-    corners[1].y = y2;
-    IntLPtoDP(dc, corners, 2);
-
-    /* Make sure first corner is top left and second corner is bottom right */
-    if (corners[0].x > corners[1].x)
-    {
-        temp = corners[0].x;
-        corners[0].x = corners[1].x;
-        corners[1].x = temp;
-    }
-    if (corners[0].y > corners[1].y)
-    {
-        temp = corners[0].y;
-        corners[0].y = corners[1].y;
-        corners[1].y = temp;
-    }
-
-    /* In GM_COMPATIBLE, don't include bottom and right edges */
-    if (dc->pdcattr->iGraphicsMode == GM_COMPATIBLE)
-    {
-        corners[1].x--;
-        corners[1].y--;
-    }
-
-    /* Close any previous figure */
-    IntGdiCloseFigure(pPath);
-
-    /* Add four points to the path */
-    pointTemp.x = corners[1].x;
-    pointTemp.y = corners[0].y;
-    if (!PATH_AddEntry(pPath, &pointTemp, PT_MOVETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    if (!PATH_AddEntry(pPath, corners, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    pointTemp.x = corners[0].x;
-    pointTemp.y = corners[1].y;
-    if (!PATH_AddEntry(pPath, &pointTemp, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    if (!PATH_AddEntry(pPath, corners + 1, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
+        return TRUE;
+    }
+
+    points[0].x = corners[1].x;
+    points[0].y = corners[0].y;
+    points[1]   = corners[0];
+    points[2].x = corners[0].x;
+    points[2].y = corners[1].y;
+    points[3]   = corners[1];
+
+    if (dc->dclevel.flPath & DCPATH_CLOCKWISE) reverse_points( points, 4 );
+
+    if (!(type = add_points( pPath, points, 4, PT_LINETO )))
+    {
+       PATH_UnlockPath(pPath);
+       return FALSE;
+    }
+    type[0] = PT_MOVETO;
 
     /* Close the rectangle figure */
     IntGdiCloseFigure(pPath) ;
@@ -402,9 +657,6 @@
  * Should be called when a call to RoundRect is performed on a DC that has
  * an open path. Returns TRUE if successful, else FALSE.
  *
- * FIXME: It adds the same entries to the path as windows does, but there
- * is an error in the bezier drawing code so that there are small pixel-size
- * gaps when the resulting path is drawn by StrokePath()
  */
 BOOL
 FASTCALL
@@ -417,78 +669,81 @@
     INT ell_width,
     INT ell_height)
 {
+    const double factor = 0.55428475; /* 4 / 3 * (sqrt(2) - 1) */
     PPATH pPath;
-    POINT corners[2], pointTemp;
-    FLOAT_POINT ellCorners[2];
+    POINT corners[2], ellipse[2], points[16];
+    BYTE *type;
+    double width, height;
+
+    if (!ell_width || !ell_height) return PATH_Rectangle( dc, x1, y1, x2, y2 );
 
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
+    if (!PATH_CheckCorners(dc, corners, x1, y1, x2, y2))
     {
         PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    if (!PATH_CheckCorners(dc, corners, x1, y1, x2, y2))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    /* Add points to the roundrect path */
-    ellCorners[0].x = corners[1].x - ell_width;
-    ellCorners[0].y = corners[0].y;
-    ellCorners[1].x = corners[1].x;
-    ellCorners[1].y = corners[0].y + ell_height;
-    if (!PATH_DoArcPart(pPath, ellCorners, 0, -M_PI_2, PT_MOVETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    pointTemp.x = corners[0].x + ell_width / 2;
-    pointTemp.y = corners[0].y;
-    if (!PATH_AddEntry(pPath, &pointTemp, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    ellCorners[0].x = corners[0].x;
-    ellCorners[1].x = corners[0].x + ell_width;
-    if (!PATH_DoArcPart(pPath, ellCorners, -M_PI_2, -M_PI, FALSE))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    pointTemp.x = corners[0].x;
-    pointTemp.y = corners[1].y - ell_height / 2;
-    if (!PATH_AddEntry(pPath, &pointTemp, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    ellCorners[0].y = corners[1].y - ell_height;
-    ellCorners[1].y = corners[1].y;
-    if (!PATH_DoArcPart(pPath, ellCorners, M_PI, M_PI_2, FALSE))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    pointTemp.x = corners[1].x - ell_width / 2;
-    pointTemp.y = corners[1].y;
-    if (!PATH_AddEntry(pPath, &pointTemp, PT_LINETO))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    ellCorners[0].x = corners[1].x - ell_width;
-    ellCorners[1].x = corners[1].x;
-    if (!PATH_DoArcPart(pPath, ellCorners, M_PI_2, 0, FALSE))
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
+        return TRUE;
+    }
+
+    ellipse[0].x = ellipse[0].y = 0;
+    ellipse[1].x = ell_width;
+    ellipse[1].y = ell_height;
+    IntLPtoDP( dc, &ellipse, 2 );
+    ell_width  = min( abs( ellipse[1].x - ellipse[0].x ), corners[1].x - corners[0].x );
+    ell_height = min( abs( ellipse[1].y - ellipse[0].y ), corners[1].y - corners[0].y );
+    width  = ell_width / 2.0;
+    height = ell_height / 2.0;
+
+    /* starting point */
+    points[0].x  = corners[1].x;
+    points[0].y  = corners[0].y + GDI_ROUND( height );
+    /* first curve */
+    points[1].x  = corners[1].x;
+    points[1].y  = corners[0].y + GDI_ROUND( height * (1 - factor) );
+    points[2].x  = corners[1].x - GDI_ROUND( width * (1 - factor) );
+    points[2].y  = corners[0].y;
+    points[3].x  = corners[1].x - GDI_ROUND( width );
+    points[3].y  = corners[0].y;
+    /* horizontal line */
+    points[4].x  = corners[0].x + GDI_ROUND( width );
+    points[4].y  = corners[0].y;
+    /* second curve */
+    points[5].x  = corners[0].x + GDI_ROUND( width * (1 - factor) );
+    points[5].y  = corners[0].y;
+    points[6].x  = corners[0].x;
+    points[6].y  = corners[0].y + GDI_ROUND( height * (1 - factor) );
+    points[7].x  = corners[0].x;
+    points[7].y  = corners[0].y + GDI_ROUND( height );
+    /* vertical line */
+    points[8].x  = corners[0].x;
+    points[8].y  = corners[1].y - GDI_ROUND( height );
+    /* third curve */
+    points[9].x  = corners[0].x;
+    points[9].y  = corners[1].y - GDI_ROUND( height * (1 - factor) );
+    points[10].x = corners[0].x + GDI_ROUND( width * (1 - factor) );
+    points[10].y = corners[1].y;
+    points[11].x = corners[0].x + GDI_ROUND( width );
+    points[11].y = corners[1].y;
+    /* horizontal line */
+    points[12].x = corners[1].x - GDI_ROUND( width );
+    points[12].y = corners[1].y;
+    /* fourth curve */
+    points[13].x = corners[1].x - GDI_ROUND( width * (1 - factor) );
+    points[13].y = corners[1].y;
+    points[14].x = corners[1].x;
+    points[14].y = corners[1].y - GDI_ROUND( height * (1 - factor) );
+    points[15].x = corners[1].x;
+    points[15].y = corners[1].y - GDI_ROUND( height );
+
+    if (dc->dclevel.flPath & DCPATH_CLOCKWISE) reverse_points( points, 16 );
+    if (!(type = add_points( pPath, points, 16, PT_BEZIERTO )))
+    {
+       PATH_UnlockPath(pPath);
+       return FALSE;
+    }
+    type[0] = PT_MOVETO;
+    type[4] = type[8] = type[12] = PT_LINETO;
 
     IntGdiCloseFigure(pPath);
     PATH_UnlockPath(pPath);
@@ -497,9 +752,6 @@
 
 /* PATH_Ellipse
  *
- * Should be called when a call to Ellipse is performed on a DC that has
- * an open path. This adds four Bezier splines representing the ellipse
- * to the path. Returns TRUE if successful, else FALSE.
  */
 BOOL
 FASTCALL
@@ -510,18 +762,129 @@
     INT x2,
     INT y2)
 {
+    const double factor = 0.55428475; /* 4 / 3 * (sqrt(2) - 1) */
     PPATH pPath;
-    /* TODO: This should probably be revised to call PATH_AngleArc */
-    /* (once it exists) */
-    BOOL Ret = PATH_Arc(dc, x1, y1, x2, y2, x1, (y1 + y2) / 2, x1, (y1 + y2) / 2,
GdiTypeArc);
-    if (Ret)
-    {
-        pPath = PATH_LockPath(dc->dclevel.hPath);
-        if (!pPath) return FALSE;
-        IntGdiCloseFigure(pPath);
+    POINT corners[2], points[13];
+    BYTE *type;
+    double width, height;
+
+    pPath = PATH_LockPath(dc->dclevel.hPath);
+    if (!pPath) return FALSE;
+
+    if (!PATH_CheckCorners(dc, corners, x1, y1, x2, y2))
+    {
+        DPRINT1("PATH_Ellipse bad corners\n");
         PATH_UnlockPath(pPath);
-    }
-    return Ret;
+        return TRUE;
+    }
+
+    width  = (corners[1].x - corners[0].x) / 2.0;
+    height = (corners[1].y - corners[0].y) / 2.0;
+
+    /* starting point */
+    points[0].x = corners[1].x;
+    points[0].y = corners[0].y + GDI_ROUND( height );
+    /* first curve */
+    points[1].x = corners[1].x;
+    points[1].y = corners[0].y + GDI_ROUND( height * (1 - factor) );
+    points[2].x = corners[1].x - GDI_ROUND( width * (1 - factor) );
+    points[2].y = corners[0].y;
+    points[3].x = corners[0].x + GDI_ROUND( width );
+    points[3].y = corners[0].y;
+    /* second curve */
+    points[4].x = corners[0].x + GDI_ROUND( width * (1 - factor) );
+    points[4].y = corners[0].y;
+    points[5].x = corners[0].x;
+    points[5].y = corners[0].y + GDI_ROUND( height * (1 - factor) );
+    points[6].x = corners[0].x;
+    points[6].y = corners[0].y + GDI_ROUND( height );
+    /* third curve */
+    points[7].x = corners[0].x;
+    points[7].y = corners[1].y - GDI_ROUND( height * (1 - factor) );
+    points[8].x = corners[0].x + GDI_ROUND( width * (1 - factor) );
+    points[8].y = corners[1].y;
+    points[9].x = corners[0].x + GDI_ROUND( width );
+    points[9].y = corners[1].y;
+    /* fourth curve */
+    points[10].x = corners[1].x - GDI_ROUND( width * (1 - factor) );
+    points[10].y = corners[1].y;
+    points[11].x = corners[1].x;
+    points[11].y = corners[1].y - GDI_ROUND( height * (1 - factor) );
+    points[12].x = corners[1].x;
+    points[12].y = corners[1].y - GDI_ROUND( height );
+
+    if (dc->dclevel.flPath & DCPATH_CLOCKWISE) reverse_points( points, 13 );
+    if (!(type = add_points( pPath, points, 13, PT_BEZIERTO )))
+    {
+        DPRINT1("PATH_Ellipse No add\n");
+        PATH_UnlockPath(pPath);
+        return FALSE;
+    }
+    type[0] = PT_MOVETO;
+
+    IntGdiCloseFigure(pPath);
+    PATH_UnlockPath(pPath);
+    return TRUE;
+}
+
+/* PATH_DoArcPart
+ *
+ * Creates a Bezier spline that corresponds to part of an arc and appends the
+ * corresponding points to the path. The start and end angles are passed in
+ * "angleStart" and "angleEnd"; these angles should span a quarter
circle
+ * at most. If "startEntryType" is non-zero, an entry of that type for the
first
+ * control point is added to the path; otherwise, it is assumed that the current
+ * position is equal to the first control point.
+ */
+BOOL
+FASTCALL
+PATH_DoArcPart(
+    PPATH pPath,
+    FLOAT_POINT corners[],
+    double angleStart,
+    double angleEnd,
+    BYTE startEntryType)
+{
+    double  halfAngle, a;
+    double  xNorm[4], yNorm[4];
+    POINT points[4];
+    BYTE *type;
+    int i, start;
+
+    ASSERT(fabs(angleEnd - angleStart) <= M_PI_2);
+
+    /* FIXME: Is there an easier way of computing this? */
+
+    /* Compute control points */
+    halfAngle = (angleEnd - angleStart) / 2.0;
+    if (fabs(halfAngle) > 1e-8)
+    {
+        a = 4.0 / 3.0 * (1 - cos(halfAngle)) / sin(halfAngle);
+        xNorm[0] = cos(angleStart);
+        yNorm[0] = sin(angleStart);
+        xNorm[1] = xNorm[0] - a * yNorm[0];
+        yNorm[1] = yNorm[0] + a * xNorm[0];
+        xNorm[3] = cos(angleEnd);
+        yNorm[3] = sin(angleEnd);
+        xNorm[2] = xNorm[3] + a * yNorm[3];
+        yNorm[2] = yNorm[3] - a * xNorm[3];
+    }
+    else
+        for (i = 0; i < 4; i++)
+        {
+            xNorm[i] = cos(angleStart);
+            yNorm[i] = sin(angleStart);
+        }
+
+    /* Add starting point to path if desired */
+    start = !startEntryType;
+
+    /* Add remaining control points */
+    for (i = start; i < 4; i++) PATH_ScaleNormalizedPoint(corners, xNorm[i], yNorm[i],
&points[i]);
+    if (!(type = add_points( pPath, points + start, 4 - start, PT_BEZIERTO ))) return
FALSE;
+    if (!start) type[0] = startEntryType;
+
+    return TRUE;
 }
 
 /* PATH_Arc
@@ -546,6 +909,7 @@
     INT yStart,
     INT xEnd,
     INT yEnd,
+    INT direction,
     INT lines)
 {
     double  angleStart, angleEnd, angleStartQuadrant, angleEndQuadrant = 0.0;
@@ -566,14 +930,10 @@
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    clockwise = ((dc->dclevel.flPath & DCPATH_CLOCKWISE) != 0);
-
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        Ret = FALSE;
-        goto ArcExit;
-    }
+    if (direction)
+       clockwise = ((direction == AD_CLOCKWISE) !=0 );
+    else
+       clockwise = ((dc->dclevel.flPath & DCPATH_CLOCKWISE) != 0);
 
     /* Check for zero height / width */
     /* FIXME: Only in GM_COMPATIBLE? */
@@ -697,7 +1057,11 @@
     }
     while (!end);
 
-    /* chord: close figure. pie: add line and close figure */
+    if (lines == GdiTypeArcTo)
+    {
+       update_current_pos( pPath );
+    }
+    else /* chord: close figure. pie: add line and close figure */
     if (lines == GdiTypeChord) // 1
     {
         IntGdiCloseFigure(pPath);
@@ -721,9 +1085,8 @@
     const POINT *pts,
     DWORD cbPoints)
 {
-    POINT pt;
-    ULONG i;
     PPATH pPath;
+    BOOL ret;
 
     ASSERT(dc);
     ASSERT(pts);
@@ -732,35 +1095,10 @@
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    /* Add a PT_MOVETO if necessary */
-    if (pPath->newStroke)
-    {
-        pPath->newStroke = FALSE;
-        IntGetCurrentPositionEx(dc, &pt);
-        CoordLPtoDP(dc, &pt);
-        if (!PATH_AddEntry(pPath, &pt, PT_MOVETO))
-        {
-            PATH_UnlockPath(pPath);
-            return FALSE;
-        }
-    }
-
-    for (i = 0; i < cbPoints; i++)
-    {
-        pt = pts[i];
-        CoordLPtoDP(dc, &pt);
-        PATH_AddEntry(pPath, &pt, PT_BEZIERTO);
-    }
+    ret = add_log_points_new_stroke( dc, pPath, pts, cbPoints, PT_BEZIERTO );
 
     PATH_UnlockPath(pPath);
-    return TRUE;
+    return ret;
 }
 
 BOOL
@@ -770,9 +1108,8 @@
     const POINT *pts,
     DWORD cbPoints)
 {
-    POINT pt;
-    ULONG i;
     PPATH pPath;
+    BYTE *type;
 
     ASSERT(dc);
     ASSERT(pts);
@@ -781,19 +1118,10 @@
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    for (i = 0; i < cbPoints; i++)
-    {
-        pt = pts[i];
-        CoordLPtoDP(dc, &pt);
-        PATH_AddEntry(pPath, &pt, (i == 0) ? PT_MOVETO : PT_BEZIERTO);
-    }
+    type = add_log_points( dc, pPath, pts, cbPoints, PT_BEZIERTO );
+    if (!type) return FALSE;
+
+    type[0] = PT_MOVETO;
 
     PATH_UnlockPath(pPath);
     return TRUE;
@@ -808,10 +1136,8 @@
     DWORD cbPoints)
 {
     PPATH pPath;
-    POINT lastmove, orig_pos;
-    ULONG i;
-    PDC_ATTR pdcattr;
-    BOOL State = FALSE, Ret = FALSE;
+    POINT orig_pos, cur_pos;
+    ULONG i, lastmove = 0;
 
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
@@ -822,116 +1148,64 @@
         return FALSE;
     }
 
-    pdcattr = dc->pdcattr;
-
-    lastmove.x = orig_pos.x = pdcattr->ptlCurrent.x;
-    lastmove.y = orig_pos.y = pdcattr->ptlCurrent.y;
-
-    i = pPath->numEntriesUsed;
-
-    while (i != 0)
-    {
-        i--;
-        if (pPath->pFlags[i] == PT_MOVETO)
-        {
-            lastmove.x = pPath->pPoints[i].x;
-            lastmove.y = pPath->pPoints[i].y;
-            if (!GdiPathDPtoLP(dc, &lastmove, 1))
+    for (i = 0; i < pPath->numEntriesUsed; i++) if (pPath->pFlags[i] ==
PT_MOVETO) lastmove = i;
+    orig_pos = pPath->pos;
+
+    IntGetCurrentPositionEx(dc, &cur_pos);
+
+    DPRINT("PPD : Current pos X %d Y %d\n",pPath->pos.x, pPath->pos.y);
+    DPRINT("PPD : last %d pos X %d Y %d\n",lastmove,
pPath->pPoints[lastmove].x, pPath->pPoints[lastmove].y);
+
+
+    for(i = 0; i < cbPoints; i++)
+    {
+        switch (types[i])
+        {
+        case PT_MOVETO:
+            pPath->newStroke = TRUE;
+            pPath->pos = pts[i];
+            IntLPtoDP( dc, &pPath->pos, 1);
+            lastmove = pPath->numEntriesUsed;
+            break;
+        case PT_LINETO:
+        case PT_LINETO | PT_CLOSEFIGURE:
+            if (!add_log_points_new_stroke( dc, pPath, &pts[i], 1, PT_LINETO ))
             {
-                PATH_UnlockPath(pPath);
-                return FALSE;
+               PATH_UnlockPath(pPath);
+               return FALSE;
             }
             break;
-        }
-    }
-
-    for (i = 0; i < cbPoints; i++)
-    {
-        if (types[i] == PT_MOVETO)
-        {
-            pPath->newStroke = TRUE;
-            lastmove.x = pts[i].x;
-            lastmove.y = pts[i].y;
-        }
-        else if ((types[i] & ~PT_CLOSEFIGURE) == PT_LINETO)
-        {
-            PATH_LineTo(dc, pts[i].x, pts[i].y);
-        }
-        else if (types[i] == PT_BEZIERTO)
-        {
-            if (!((i + 2 < cbPoints) && (types[i + 1] == PT_BEZIERTO)
-                    && ((types[i + 2] & ~PT_CLOSEFIGURE) == PT_BEZIERTO)))
-                goto err;
-            PATH_PolyBezierTo(dc, &(pts[i]), 3);
-            i += 2;
-        }
-        else
-            goto err;
-
-        pdcattr->ptlCurrent.x = pts[i].x;
-        pdcattr->ptlCurrent.y = pts[i].y;
-        State = TRUE;
+        case PT_BEZIERTO:
+            if ((i + 2 < cbPoints) && (types[i + 1] == PT_BEZIERTO)
&&
+                (types[i + 2] & ~PT_CLOSEFIGURE) == PT_BEZIERTO)
+            {
+                if (!add_log_points_new_stroke( dc, pPath, &pts[i], 3, PT_BEZIERTO
))
+                {
+                   PATH_UnlockPath(pPath);
+                   return FALSE;
+                }
+                i += 2;
+                break;
+            }
+            /* fall through */
+        default:
+            /* restore original position */
+            pPath->pos = orig_pos;
+
+            DPRINT("PPD Bad   : pos X %d Y %d\n",pPath->pos.x,
pPath->pos.y);
+
+            IntGdiMoveToEx(dc, cur_pos.x, cur_pos.y, NULL);
+
+            PATH_UnlockPath(pPath);
+            return FALSE;
+        }
 
         if (types[i] & PT_CLOSEFIGURE)
         {
-            pPath->pFlags[pPath->numEntriesUsed - 1] |= PT_CLOSEFIGURE;
-            pPath->newStroke = TRUE;
-            pdcattr->ptlCurrent.x = lastmove.x;
-            pdcattr->ptlCurrent.y = lastmove.y;
-            State = TRUE;
-        }
-    }
-    Ret = TRUE;
-    goto Exit;
-
-err:
-    if ((pdcattr->ptlCurrent.x != orig_pos.x) || (pdcattr->ptlCurrent.y !=
orig_pos.y))
-    {
-        pPath->newStroke = TRUE;
-        pdcattr->ptlCurrent.x = orig_pos.x;
-        pdcattr->ptlCurrent.y = orig_pos.y;
-        State = TRUE;
-    }
-Exit:
-    if (State) // State change?
-    {
-        pdcattr->ptfxCurrent = pdcattr->ptlCurrent;
-        CoordLPtoDP(dc, &pdcattr->ptfxCurrent); // Update fx
-        pdcattr->ulDirty_ &= ~(DIRTY_PTLCURRENT | DIRTY_PTFXCURRENT |
DIRTY_STYLESTATE);
-    }
-    PATH_UnlockPath(pPath);
-    return Ret;
-}
-
-BOOL
-FASTCALL
-PATH_Polyline(
-    PDC dc,
-    const POINT *pts,
-    DWORD cbPoints)
-{
-    POINT pt;
-    ULONG i;
-    PPATH pPath;
-
-    ASSERT(dc);
-    ASSERT(pts);
-    ASSERT(cbPoints);
-
-    pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath) return FALSE;
-
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-    for (i = 0; i < cbPoints; i++)
-    {
-        pt = pts[i];
-        CoordLPtoDP(dc, &pt);
-        PATH_AddEntry(pPath, &pt, (i == 0) ? PT_MOVETO : PT_LINETO);
+            close_figure( pPath );
+            pPath->pos = pPath->pPoints[lastmove];
+            DPRINT("PPD close : pos X %d Y %d\n",pPath->pos.x,
pPath->pos.y);
+        }
     }
     PATH_UnlockPath(pPath);
     return TRUE;
@@ -944,83 +1218,21 @@
     const POINT *pts,
     DWORD cbPoints)
 {
-    POINT pt;
-    ULONG i;
     PPATH pPath;
+    BOOL ret;
 
     ASSERT(dc);
     ASSERT(pts);
     ASSERT(cbPoints);
 
+    if (cbPoints < 1) return FALSE;
+
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    /* Add a PT_MOVETO if necessary */
-    if (pPath->newStroke)
-    {
-        pPath->newStroke = FALSE;
-        IntGetCurrentPositionEx(dc, &pt);
-        CoordLPtoDP(dc, &pt);
-        if (!PATH_AddEntry(pPath, &pt, PT_MOVETO))
-        {
-            PATH_UnlockPath(pPath);
-            return FALSE;
-        }
-    }
-
-    for (i = 0; i < cbPoints; i++)
-    {
-        pt = pts[i];
-        CoordLPtoDP(dc, &pt);
-        PATH_AddEntry(pPath, &pt, PT_LINETO);
-    }
+    ret = add_log_points_new_stroke( dc, pPath, pts, cbPoints, PT_LINETO );
     PATH_UnlockPath(pPath);
-    return TRUE;
-}
-
-
-BOOL
-FASTCALL
-PATH_Polygon(
-    PDC dc,
-    const POINT *pts,
-    DWORD cbPoints)
-{
-    POINT pt;
-    ULONG i;
-    PPATH pPath;
-
-    ASSERT(dc);
-    ASSERT(pts);
-
-    pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath) return FALSE;
-
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    for (i = 0; i < cbPoints; i++)
-    {
-        pt = pts[i];
-        CoordLPtoDP(dc, &pt);
-        PATH_AddEntry(pPath, &pt, (i == 0) ? PT_MOVETO :
-                      ((i == cbPoints - 1) ? PT_LINETO | PT_CLOSEFIGURE :
-                       PT_LINETO));
-    }
-
-    PATH_UnlockPath(pPath);
-    return TRUE;
+    return ret;
 }
 
 BOOL
@@ -1031,8 +1243,8 @@
     const INT* counts,
     UINT polygons)
 {
-    POINT pt, startpt;
-    ULONG poly, point, i;
+    UINT poly, count;
+    BYTE *type;
     PPATH pPath;
 
     ASSERT(dc);
@@ -1040,30 +1252,35 @@
     ASSERT(counts);
     ASSERT(polygons);
 
+    if (!polygons) return FALSE;
+
     pPath = PATH_LockPath(dc->dclevel.hPath);
     if (!pPath) return FALSE;
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
-    }
-
-    for (i = 0, poly = 0; poly < polygons; poly++)
-    {
-        for (point = 0; point < (ULONG) counts[poly]; point++, i++)
-        {
-            pt = pts[i];
-            CoordLPtoDP(dc, &pt);
-            if (point == 0) startpt = pt;
-            PATH_AddEntry(pPath, &pt, (point == 0) ? PT_MOVETO : PT_LINETO);
-        }
-
-        /* Win98 adds an extra line to close the figure for some reason */
-        PATH_AddEntry(pPath, &startpt, PT_LINETO | PT_CLOSEFIGURE);
-    }
-
+
+    for (poly = count = 0; poly < polygons; poly++)
+    {
+        if (counts[poly] < 2)
+        {
+           PATH_UnlockPath(pPath);
+           return FALSE;
+        }
+        count += counts[poly];
+    }
+
+    type = add_log_points( dc, pPath, pts, count, PT_LINETO );
+    if (!type)
+    {
+       PATH_UnlockPath(pPath);
+       return FALSE;
+    }
+
+    /* make the first point of each polyline a PT_MOVETO, and close the last one */
+    for (poly = 0; poly < polygons; type += counts[poly++])
+    {
+        type[0] = PT_MOVETO;
+        type[counts[poly] - 1] = PT_LINETO | PT_CLOSEFIGURE;
+    }
     PATH_UnlockPath(pPath);
     return TRUE;
 }
@@ -1086,13 +1303,9 @@
     ASSERT(polylines);
 
     pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath) return FALSE;
-
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-    {
-        PATH_UnlockPath(pPath);
-        return FALSE;
+    if (!pPath)
+    {
+       return FALSE;
     }
 
     for (i = 0, poly = 0; poly < polylines; poly++)
@@ -1104,61 +1317,10 @@
             PATH_AddEntry(pPath, &pt, (point == 0) ? PT_MOVETO : PT_LINETO);
         }
     }
-
+    DPRINT("PATH_PolyPolyline end count %d\n",pPath->numEntriesUsed);
     PATH_UnlockPath(pPath);
     return TRUE;
 }
-
-
-/* PATH_CheckCorners
- *
- * Helper function for PATH_RoundRect() and PATH_Rectangle()
- */
-BOOL
-PATH_CheckCorners(
-    DC *dc,
-    POINT corners[],
-    INT x1,
-    INT y1,
-    INT x2,
-    INT y2)
-{
-    INT temp;
-    PDC_ATTR pdcattr = dc->pdcattr;
-
-    /* Convert points to device coordinates */
-    corners[0].x = x1;
-    corners[0].y = y1;
-    corners[1].x = x2;
-    corners[1].y = y2;
-    CoordLPtoDP(dc, &corners[0]);
-    CoordLPtoDP(dc, &corners[1]);
-
-    /* Make sure first corner is top left and second corner is bottom right */
-    if (corners[0].x > corners[1].x)
-    {
-        temp = corners[0].x;
-        corners[0].x = corners[1].x;
-        corners[1].x = temp;
-    }
-
-    if (corners[0].y > corners[1].y)
-    {
-        temp = corners[0].y;
-        corners[0].y = corners[1].y;
-        corners[1].y = temp;
-    }
-
-    /* In GM_COMPATIBLE, don't include bottom and right edges */
-    if (pdcattr->iGraphicsMode == GM_COMPATIBLE)
-    {
-        corners[1].x--;
-        corners[1].y--;
-    }
-
-    return TRUE;
-}
-
 
 /* PATH_AddFlatBezier
  *
@@ -1171,16 +1333,20 @@
     BOOL closed)
 {
     POINT *pts;
+    BOOL ret = FALSE;
     INT no, i;
 
     pts = GDI_Bezier(pt, 4, &no);
     if (!pts) return FALSE;
 
     for (i = 1; i < no; i++)
-        PATH_AddEntry(pPath, &pts[i],  (i == no - 1 && closed) ? PT_LINETO |
PT_CLOSEFIGURE : PT_LINETO);
+    {
+        if (!(ret = PATH_AddEntry(pPath, &pts[i],  (i == no - 1 && closed) ?
PT_LINETO | PT_CLOSEFIGURE : PT_LINETO)))
+           break;
+    }
 
     ExFreePoolWithTag(pts, TAG_BEZIER);
-    return TRUE;
+    return ret;
 }
 
 /* PATH_FlattenPath
@@ -1188,349 +1354,193 @@
  * Replaces Beziers with line segments
  *
  */
-BOOL
+PPATH
 FASTCALL
 PATH_FlattenPath(PPATH pPath)
 {
-    PATH newPath;
+    PPATH newPath;
     INT srcpt;
-
-    RtlZeroMemory(&newPath, sizeof(newPath));
-    newPath.state = PATH_Open;
+    DPRINT("PATH_FlattenPath\n");
+    if (!(newPath = PATH_CreatePath(pPath->numEntriesUsed))) return NULL;
+
     for (srcpt = 0; srcpt < pPath->numEntriesUsed; srcpt++)
     {
         switch(pPath->pFlags[srcpt] & ~PT_CLOSEFIGURE)
         {
             case PT_MOVETO:
             case PT_LINETO:
-                PATH_AddEntry(&newPath, &pPath->pPoints[srcpt],
pPath->pFlags[srcpt]);
+                if (!PATH_AddEntry(newPath, &pPath->pPoints[srcpt],
pPath->pFlags[srcpt]))
+                {
+                   PATH_UnlockPath(newPath);
+                   PATH_Delete(newPath->BaseObject.hHmgr);
+                   return NULL;
+                }
                 break;
             case PT_BEZIERTO:
-                PATH_AddFlatBezier(&newPath, &pPath->pPoints[srcpt - 1],
pPath->pFlags[srcpt + 2] & PT_CLOSEFIGURE);
+                if(!PATH_AddFlatBezier(newPath, &pPath->pPoints[srcpt - 1],
pPath->pFlags[srcpt + 2] & PT_CLOSEFIGURE))
+                {
+                   PATH_UnlockPath(newPath);
+                   PATH_Delete(newPath->BaseObject.hHmgr);
+                   return NULL;
+                }
                 srcpt += 2;
                 break;
         }
     }
-
-    newPath.state = PATH_Closed;
-    PATH_AssignGdiPath(pPath, &newPath);
-    PATH_EmptyPath(&newPath);
-    return TRUE;
-}
-
+    DPRINT("PATH_FlattenPath good\n");
+    newPath->state = pPath->state;
+    return newPath;
+}
 
 /* PATH_PathToRegion
  *
  * Creates a region from the specified path using the specified polygon
  * filling mode. The path is left unchanged. A handle to the region that
- * was created is stored in *pHrgn. If successful, TRUE is returned; if an
- * error occurs, SetLastError is called with the appropriate value and
- * FALSE is returned.
+ * was created is stored in *pHrgn.
  */
 BOOL
 FASTCALL
 PATH_PathToRegion(
     PPATH pPath,
-    INT nPolyFillMode,
+    INT Mode,
     PREGION Rgn)
 {
-    int    numStrokes, iStroke, i;
-    PULONG  pNumPointsInStroke;
-    BOOL Ret;
-
-    ASSERT(pPath != NULL);
-    ASSERT(Rgn != NULL);
-
-    PATH_FlattenPath(pPath);
-
-    /* First pass: Find out how many strokes there are in the path */
-    /* FIXME: We could eliminate this with some bookkeeping in GdiPath */
-    numStrokes = 0;
-    for (i = 0; i < pPath->numEntriesUsed; i++)
-        if ((pPath->pFlags[i] & ~PT_CLOSEFIGURE) == PT_MOVETO)
-            numStrokes++;
-
-    if (numStrokes == 0)
-    {
-        DPRINT1("numStrokes is 0\n");
-        return FALSE;
-    }
-
-    /* Allocate memory for number-of-points-in-stroke array */
-    pNumPointsInStroke = ExAllocatePoolWithTag(PagedPool, sizeof(ULONG) * numStrokes,
TAG_PATH);
-    if (!pNumPointsInStroke)
-    {
-        DPRINT1("Failed to allocate %lu strokes\n", numStrokes);
+    int i, pos, polygons; 
+    PULONG counts;
+    int Ret;
+
+    if (!pPath->numEntriesUsed) return FALSE;
+
+    counts = ExAllocatePoolWithTag(PagedPool, (pPath->numEntriesUsed / 2) *
sizeof(counts), TAG_PATH);
+    if (!counts)
+    {
+        DPRINT1("Failed to allocate %lu strokes\n", (pPath->numEntriesUsed /
2) * sizeof(*counts));
         EngSetLastError(ERROR_NOT_ENOUGH_MEMORY);
         return FALSE;
     }
 
-    /* Second pass: remember number of points in each polygon */
-    iStroke = -1; /* Will get incremented to 0 at beginning of first stroke */
-    for (i = 0; i < pPath->numEntriesUsed; i++)
-    {
-        /* Is this the beginning of a new stroke? */
-        if ((pPath->pFlags[i] & ~PT_CLOSEFIGURE) == PT_MOVETO)
-        {
-            iStroke++;
-            _PRAGMA_WARNING_SUPPRESS(__WARNING_WRITE_OVERRUN)
-            pNumPointsInStroke[iStroke] = 0;
-        }
-
-        _PRAGMA_WARNING_SUPPRESS(__WARNING_READ_OVERRUN)
-        pNumPointsInStroke[iStroke]++;
-    }
+    pos = polygons = 0;
+    ASSERT( pPath->pFlags[0] == PT_MOVETO );
+    for (i = 1; i < pPath->numEntriesUsed; i++)
+    {
+        if (pPath->pFlags[i] != PT_MOVETO) continue;
+        counts[polygons++] = i - pos;
+        pos = i;
+    }
+    if (i > pos + 1) counts[polygons++] = i - pos;
+
+    ASSERT( polygons <= pPath->numEntriesUsed / 2 );
 
     /* Fill the region with the strokes */
     Ret = REGION_SetPolyPolygonRgn(Rgn,
                                    pPath->pPoints,
-                                   pNumPointsInStroke,
-                                   numStrokes,
-                                   nPolyFillMode);
+                                   counts,
+                                   polygons,
+                                   Mode);
     if (!Ret)
     {
         DPRINT1("REGION_SetPolyPolygonRgn failed\n");
     }
 
-    /* Free memory for number-of-points-in-stroke array */
-    ExFreePoolWithTag(pNumPointsInStroke, TAG_PATH);
+    ExFreePoolWithTag(counts, TAG_PATH);
 
     /* Success! */
     return Ret;
 }
 
-/* PATH_EmptyPath
+/* PATH_FillPath
  *
- * Removes all entries from the path and sets the path state to PATH_Null.
+ * You can play with this as long as you like, but if you break Area.exe the purge will
Begain on Path!!!
+ *
  */
-VOID
-FASTCALL
-PATH_EmptyPath(PPATH pPath)
-{
-    ASSERT(pPath != NULL);
-
-    pPath->state = PATH_Null;
-    pPath->numEntriesUsed = 0;
-}
-
-/* PATH_AddEntry
- *
- * Adds an entry to the path. For "flags", pass either PT_MOVETO, PT_LINETO
- * or PT_BEZIERTO, optionally ORed with PT_CLOSEFIGURE. Returns TRUE if
- * successful, FALSE otherwise (e.g. if not enough memory was available).
- */
-BOOL
-FASTCALL
-PATH_AddEntry(
-    PPATH pPath,
-    const POINT *pPoint,
-    BYTE flags)
-{
-    ASSERT(pPath != NULL);
-
-    /* FIXME: If newStroke is true, perhaps we want to check that we're
-     * getting a PT_MOVETO
+BOOL
+FASTCALL
+PATH_FillPath(
+    PDC dc,
+    PPATH pPath)
+{
+    INT   mapMode, graphicsMode;
+    SIZE  ptViewportExt, ptWindowExt;
+    POINTL ptViewportOrg, ptWindowOrg;
+    XFORML xform;
+    PREGION  Rgn;
+    PDC_ATTR pdcattr = dc->pdcattr;
+
+    if (pPath->state != PATH_Closed)
+    {
+        DPRINT("PFP : path not closed\n");
+        EngSetLastError(ERROR_CAN_NOT_COMPLETE);
+        return FALSE;
+    }
+
+    /* Allocate a temporary region */
+    Rgn = IntSysCreateRectpRgn(0, 0, 0, 0);
+    if (!Rgn)
+    {
+        EngSetLastError(ERROR_NOT_ENOUGH_MEMORY);
+        return FALSE;
+    }
+
+    if (!PATH_PathToRegion(pPath, pdcattr->jFillMode, Rgn))
+    {
+        DPRINT("PFP : Fail P2R\n");
+        /* EngSetLastError ? */
+        REGION_Delete(Rgn);
+        return FALSE;
+    }
+
+    /* Since PaintRgn interprets the region as being in logical coordinates
+     * but the points we store for the path are already in device
+     * coordinates, we have to set the mapping mode to MM_TEXT temporarily.
+     * Using SaveDC to save information about the mapping mode / world
+     * transform would be easier but would require more overhead, especially
+     * now that SaveDC saves the current path.
      */
 
-    /* Check that path is open */
-    if (pPath->state != PATH_Open)
-        return FALSE;
-
-    /* Reserve enough memory for an extra path entry */
-    if (!PATH_ReserveEntries(pPath, pPath->numEntriesUsed + 1))
-        return FALSE;
-
-    /* Store information in path entry */
-    pPath->pPoints[pPath->numEntriesUsed] = *pPoint;
-    pPath->pFlags[pPath->numEntriesUsed] = flags;
-
-    /* If this is PT_CLOSEFIGURE, we have to start a new stroke next time */
-    if ((flags & PT_CLOSEFIGURE) == PT_CLOSEFIGURE)
-        pPath->newStroke = TRUE;
-
-    /* Increment entry count */
-    pPath->numEntriesUsed++;
-
+    /* Save the information about the old mapping mode */
+    mapMode = pdcattr->iMapMode;
+    ptViewportExt = pdcattr->szlViewportExt;
+    ptViewportOrg = pdcattr->ptlViewportOrg;
+    ptWindowExt   = pdcattr->szlWindowExt;
+    ptWindowOrg   = pdcattr->ptlWindowOrg;
+
+    /* Save world transform
+     * NB: The Windows documentation on world transforms would lead one to
+     * believe that this has to be done only in GM_ADVANCED; however, my
+     * tests show that resetting the graphics mode to GM_COMPATIBLE does
+     * not reset the world transform.
+     */
+    MatrixS2XForm(&xform, &dc->pdcattr->mxWorldToPage);
+
+    /* Set MM_TEXT */
+    IntGdiSetMapMode(dc, MM_TEXT);
+    pdcattr->ptlViewportOrg.x = 0;
+    pdcattr->ptlViewportOrg.y = 0;
+    pdcattr->ptlWindowOrg.x = 0;
+    pdcattr->ptlWindowOrg.y = 0;
+
+    graphicsMode = pdcattr->iGraphicsMode;
+    pdcattr->iGraphicsMode = GM_ADVANCED;
+    GreModifyWorldTransform(dc, &xform, MWT_IDENTITY);
+    pdcattr->iGraphicsMode =  graphicsMode;
+
+    /* Paint the region */
+    IntGdiPaintRgn(dc, Rgn);
+    REGION_Delete(Rgn);
+    /* Restore the old mapping mode */
+    IntGdiSetMapMode(dc, mapMode);
+    pdcattr->szlViewportExt = ptViewportExt;
+    pdcattr->ptlViewportOrg = ptViewportOrg;
+    pdcattr->szlWindowExt   = ptWindowExt;
+    pdcattr->ptlWindowOrg   = ptWindowOrg;
+
+    /* Go to GM_ADVANCED temporarily to restore the world transform */
+    graphicsMode = pdcattr->iGraphicsMode;
+    pdcattr->iGraphicsMode = GM_ADVANCED;
+    GreModifyWorldTransform(dc, &xform, MWT_MAX+1);
+    pdcattr->iGraphicsMode = graphicsMode;
     return TRUE;
 }
-
-/* PATH_ReserveEntries
- *
- * Ensures that at least "numEntries" entries (for points and flags) have
- * been allocated; allocates larger arrays and copies the existing entries
- * to those arrays, if necessary. Returns TRUE if successful, else FALSE.
- */
-BOOL
-FASTCALL
-PATH_ReserveEntries(
-    PPATH pPath,
-    INT numEntries)
-{
-    INT numEntriesToAllocate;
-    POINT *pPointsNew;
-    BYTE *pFlagsNew;
-
-    ASSERT(pPath != NULL);
-    ASSERT(numEntries >= 0);
-
-    /* Do we have to allocate more memory? */
-    if (numEntries > pPath->numEntriesAllocated)
-    {
-        /* Find number of entries to allocate. We let the size of the array
-         * grow exponentially, since that will guarantee linear time
-         * complexity. */
-        if (pPath->numEntriesAllocated)
-        {
-            numEntriesToAllocate = pPath->numEntriesAllocated;
-            while (numEntriesToAllocate < numEntries)
-                numEntriesToAllocate = numEntriesToAllocate * GROW_FACTOR_NUMER /
GROW_FACTOR_DENOM;
-        }
-        else
-            numEntriesToAllocate = numEntries;
-
-        /* Allocate new arrays */
-        pPointsNew = (POINT *)ExAllocatePoolWithTag(PagedPool, numEntriesToAllocate *
sizeof(POINT), TAG_PATH);
-        if (!pPointsNew)
-            return FALSE;
-
-        pFlagsNew = (BYTE *)ExAllocatePoolWithTag(PagedPool, numEntriesToAllocate *
sizeof(BYTE), TAG_PATH);
-        if (!pFlagsNew)
-        {
-            ExFreePoolWithTag(pPointsNew, TAG_PATH);
-            return FALSE;
-        }
-
-        /* Copy old arrays to new arrays and discard old arrays */
-        if (pPath->pPoints)
-        {
-            ASSERT(pPath->pFlags);
-
-            memcpy(pPointsNew, pPath->pPoints,
sizeof(POINT)*pPath->numEntriesUsed);
-            memcpy(pFlagsNew, pPath->pFlags, sizeof(BYTE)*pPath->numEntriesUsed);
-
-            ExFreePoolWithTag(pPath->pPoints, TAG_PATH);
-            ExFreePoolWithTag(pPath->pFlags, TAG_PATH);
-        }
-
-        pPath->pPoints = pPointsNew;
-        pPath->pFlags = pFlagsNew;
-        pPath->numEntriesAllocated = numEntriesToAllocate;
-    }
-
-    return TRUE;
-}
-
-/* PATH_DoArcPart
- *
- * Creates a Bezier spline that corresponds to part of an arc and appends the
- * corresponding points to the path. The start and end angles are passed in
- * "angleStart" and "angleEnd"; these angles should span a quarter
circle
- * at most. If "startEntryType" is non-zero, an entry of that type for the
first
- * control point is added to the path; otherwise, it is assumed that the current
- * position is equal to the first control point.
- */
-BOOL
-FASTCALL
-PATH_DoArcPart(
-    PPATH pPath,
-    FLOAT_POINT corners[],
-    double angleStart,
-    double angleEnd,
-    BYTE startEntryType)
-{
-    double  halfAngle, a;
-    double  xNorm[4], yNorm[4];
-    POINT point;
-    int i;
-
-    ASSERT(fabs(angleEnd - angleStart) <= M_PI_2);
-
-    /* FIXME: Is there an easier way of computing this? */
-
-    /* Compute control points */
-    halfAngle = (angleEnd - angleStart) / 2.0;
-    if (fabs(halfAngle) > 1e-8)
-    {
-        a = 4.0 / 3.0 * (1 - cos(halfAngle)) / sin(halfAngle);
-        xNorm[0] = cos(angleStart);
-        yNorm[0] = sin(angleStart);
-        xNorm[1] = xNorm[0] - a * yNorm[0];
-        yNorm[1] = yNorm[0] + a * xNorm[0];
-        xNorm[3] = cos(angleEnd);
-        yNorm[3] = sin(angleEnd);
-        xNorm[2] = xNorm[3] + a * yNorm[3];
-        yNorm[2] = yNorm[3] - a * xNorm[3];
-    }
-    else
-        for (i = 0; i < 4; i++)
-        {
-            xNorm[i] = cos(angleStart);
-            yNorm[i] = sin(angleStart);
-        }
-
-    /* Add starting point to path if desired */
-    if (startEntryType)
-    {
-        PATH_ScaleNormalizedPoint(corners, xNorm[0], yNorm[0], &point);
-        if (!PATH_AddEntry(pPath, &point, startEntryType))
-            return FALSE;
-    }
-
-    /* Add remaining control points */
-    for (i = 1; i < 4; i++)
-    {
-        PATH_ScaleNormalizedPoint(corners, xNorm[i], yNorm[i], &point);
-        if (!PATH_AddEntry(pPath, &point, PT_BEZIERTO))
-            return FALSE;
-    }
-
-    return TRUE;
-}
-
-/* PATH_ScaleNormalizedPoint
- *
- * Scales a normalized point (x, y) with respect to the box whose corners are
- * passed in "corners". The point is stored in "*pPoint". The
normalized
- * coordinates (-1.0, -1.0) correspond to corners[0], the coordinates
- * (1.0, 1.0) correspond to corners[1].
- */
-VOID
-FASTCALL
-PATH_ScaleNormalizedPoint(
-    FLOAT_POINT corners[],
-    double x,
-    double y,
-    POINT *pPoint)
-{
-    ASSERT(corners);
-    ASSERT(pPoint);
-
-    pPoint->x = GDI_ROUND((double)corners[0].x + (double)(corners[1].x - corners[0].x)
* 0.5 * (x + 1.0));
-    pPoint->y = GDI_ROUND((double)corners[0].y + (double)(corners[1].y - corners[0].y)
* 0.5 * (y + 1.0));
-}
-
-/* PATH_NormalizePoint
- *
- * Normalizes a point with respect to the box whose corners are passed in
- * corners. The normalized coordinates are stored in *pX and *pY.
- */
-VOID
-FASTCALL
-PATH_NormalizePoint(
-    FLOAT_POINT corners[],
-    const FLOAT_POINT *pPoint,
-    double *pX,
-    double *pY)
-{
-    ASSERT(corners);
-    ASSERT(pPoint);
-    ASSERT(pX);
-    ASSERT(pY);
-
-    *pX = (double)(pPoint->x - corners[0].x) / (double)(corners[1].x - corners[0].x) *
2.0 - 1.0;
-    *pY = (double)(pPoint->y - corners[0].y) / (double)(corners[1].y - corners[0].y) *
2.0 - 1.0;
-}
-
 
 BOOL
 FASTCALL
@@ -1711,32 +1721,36 @@
 #define round(x) ((int)((x)>0?(x)+0.5:(x)-0.5))
 
 static
-BOOL
+PPATH
 FASTCALL
 PATH_WidenPath(DC *dc)
 {
     INT i, j, numStrokes, numOldStrokes, penWidth, penWidthIn, penWidthOut, size,
penStyle;
-    BOOL ret = FALSE;
-    PPATH pPath, pNewPath, *pStrokes = NULL, *pOldStrokes, pUpPath, pDownPath;
+    PPATH pPath, flat_path, pNewPath, *pStrokes = NULL, *pOldStrokes, pUpPath,
pDownPath;
     EXTLOGPEN *elp;
+    BYTE *type;
     DWORD obj_type, joint, endcap, penType;
     PDC_ATTR pdcattr = dc->pdcattr;
 
     pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath) return FALSE;
-
-    if (pPath->state == PATH_Open)
-    {
+    if (!pPath)
+    {
+        EngSetLastError( ERROR_CAN_NOT_COMPLETE );
+        return NULL;
+    }
+
+    if (pPath->state != PATH_Closed)
+    {
+        DPRINT("PWP 1\n");
         PATH_UnlockPath(pPath);
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
-        return FALSE;
-    }
-
-    PATH_FlattenPath(pPath);
+        return NULL;
+    }
 
     size = GreGetObject(pdcattr->hpen, 0, NULL);
     if (!size)
     {
+        DPRINT("PWP 2\n");
         PATH_UnlockPath(pPath);
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
         return FALSE;
@@ -1745,6 +1759,7 @@
     elp = ExAllocatePoolWithTag(PagedPool, size, TAG_PATH);
     if (elp == NULL)
     {
+        DPRINT("PWP 3\n");
         PATH_UnlockPath(pPath);
         EngSetLastError(ERROR_OUTOFMEMORY);
         return FALSE;
@@ -1763,6 +1778,7 @@
     }
     else
     {
+        DPRINT("PWP 4\n");
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
         ExFreePoolWithTag(elp, TAG_PATH);
         PATH_UnlockPath(pPath);
@@ -1779,10 +1795,18 @@
     /* The function cannot apply to cosmetic pens */
     if (obj_type == GDI_OBJECT_TYPE_EXTPEN && penType == PS_COSMETIC)
     {
+        DPRINT("PWP 5\n");
         PATH_UnlockPath(pPath);
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
         return FALSE;
     }
+
+    if (!(flat_path = PATH_FlattenPath(pPath)))
+    {
+       PATH_UnlockPath(pPath);
+       return NULL;
+    }
+    PATH_UnlockPath(pPath);
 
     penWidthIn = penWidth / 2;
     penWidthOut = penWidth / 2;
@@ -1791,18 +1815,20 @@
 
     numStrokes = 0;
 
-    for (i = 0, j = 0; i < pPath->numEntriesUsed; i++, j++)
+    for (i = 0, j = 0; i < flat_path->numEntriesUsed; i++, j++)
     {
         POINT point;
-        if ((i == 0 || (pPath->pFlags[i - 1] & PT_CLOSEFIGURE)) &&
-                (pPath->pFlags[i] != PT_MOVETO))
+        if ((i == 0 || (flat_path->pFlags[i - 1] & PT_CLOSEFIGURE)) &&
+                (flat_path->pFlags[i] != PT_MOVETO))
         {
             DPRINT1("Expected PT_MOVETO %s, got path flag %c\n",
                     i == 0 ? "as first point" : "after
PT_CLOSEFIGURE",
-                    pPath->pFlags[i]);
+                    flat_path->pFlags[i]);
+            PATH_UnlockPath(flat_path);
+            PATH_Delete(flat_path->BaseObject.hHmgr);
             return FALSE;
         }
-        switch(pPath->pFlags[i])
+        switch(flat_path->pFlags[i])
         {
             case PT_MOVETO:
                 if (numStrokes > 0)
@@ -1813,47 +1839,52 @@
                 numStrokes++;
                 j = 0;
                 if (numStrokes == 1)
-                    pStrokes = ExAllocatePoolWithTag(PagedPool, numStrokes *
sizeof(PPATH), TAG_PATH);
+                    pStrokes = ExAllocatePoolWithTag(PagedPool, sizeof(*pStrokes),
TAG_PATH);
                 else
                 {
                     pOldStrokes = pStrokes; // Save old pointer.
-                    pStrokes = ExAllocatePoolWithTag(PagedPool, numStrokes *
sizeof(PPATH), TAG_PATH);
-                    if (!pStrokes) return FALSE;
+                    pStrokes = ExAllocatePoolWithTag(PagedPool, numStrokes *
sizeof(*pStrokes), TAG_PATH);
+                    if (!pStrokes)
+                    {
+                       PATH_UnlockPath(flat_path);
+                       PATH_Delete(flat_path->BaseObject.hHmgr);
+                       return FALSE;
+                    }
                     RtlCopyMemory(pStrokes, pOldStrokes, numOldStrokes * sizeof(PPATH));
                     ExFreePoolWithTag(pOldStrokes, TAG_PATH); // Free old pointer.
                 }
-                if (!pStrokes) return FALSE;
+                if (!pStrokes)
+                {
+                   PATH_UnlockPath(flat_path);
+                   PATH_Delete(flat_path->BaseObject.hHmgr);
+                   return FALSE;
+                }
                 pStrokes[numStrokes - 1] = ExAllocatePoolWithTag(PagedPool, sizeof(PATH),
TAG_PATH);
                 if (!pStrokes[numStrokes - 1])
                 {
                     ASSERT(FALSE); // FIXME
                 }
-
                 PATH_InitGdiPath(pStrokes[numStrokes - 1]);
                 pStrokes[numStrokes - 1]->state = PATH_Open;
             case PT_LINETO:
             case (PT_LINETO | PT_CLOSEFIGURE):
-                point.x = pPath->pPoints[i].x;
-                point.y = pPath->pPoints[i].y;
-                PATH_AddEntry(pStrokes[numStrokes - 1], &point,
pPath->pFlags[i]);
+                point.x = flat_path->pPoints[i].x;
+                point.y = flat_path->pPoints[i].y;
+                PATH_AddEntry(pStrokes[numStrokes - 1], &point,
flat_path->pFlags[i]);
                 break;
             case PT_BEZIERTO:
                 /* Should never happen because of the FlattenPath call */
                 DPRINT1("Should never happen\n");
                 break;
             default:
-                DPRINT1("Got path flag %c\n", pPath->pFlags[i]);
+                DPRINT1("Got path flag %c\n", flat_path->pFlags[i]);
+                PATH_UnlockPath(flat_path);
+                PATH_Delete(flat_path->BaseObject.hHmgr);
                 return FALSE;
         }
     }
 
-    pNewPath = ExAllocatePoolWithTag(PagedPool, sizeof(PATH), TAG_PATH);
-    if (!pNewPath)
-    {
-        ASSERT(FALSE); // FIXME
-    }
-    PATH_InitGdiPath(pNewPath);
-    pNewPath->state = PATH_Open;
+    pNewPath = PATH_CreatePath( flat_path->numEntriesUsed );
 
     for (i = 0; i < numStrokes; i++)
     {
@@ -2059,20 +2090,11 @@
                 }
             }
         }
-        for (j = 0; j < pUpPath->numEntriesUsed; j++)
-        {
-            POINT pt;
-            pt.x = pUpPath->pPoints[j].x;
-            pt.y = pUpPath->pPoints[j].y;
-            PATH_AddEntry(pNewPath, &pt, (j == 0 ? PT_MOVETO : PT_LINETO));
-        }
-        for (j = 0; j < pDownPath->numEntriesUsed; j++)
-        {
-            POINT pt;
-            pt.x = pDownPath->pPoints[pDownPath->numEntriesUsed - j - 1].x;
-            pt.y = pDownPath->pPoints[pDownPath->numEntriesUsed - j - 1].y;
-            PATH_AddEntry(pNewPath, &pt, ((j == 0 &&
(pStrokes[i]->pFlags[pStrokes[i]->numEntriesUsed - 1] & PT_CLOSEFIGURE)) ?
PT_MOVETO : PT_LINETO));
-        }
+        type = add_points( pNewPath, pUpPath->pPoints, pUpPath->numEntriesUsed,
PT_LINETO );
+        type[0] = PT_MOVETO;
+        reverse_points( pDownPath->pPoints, pDownPath->numEntriesUsed );
+        type = add_points( pNewPath, pDownPath->pPoints, pDownPath->numEntriesUsed,
PT_LINETO );
+        if (pStrokes[i]->pFlags[pStrokes[i]->numEntriesUsed - 1] &
PT_CLOSEFIGURE) type[0] = PT_MOVETO;
 
         PATH_DestroyGdiPath(pStrokes[i]);
         ExFreePoolWithTag(pStrokes[i], TAG_PATH);
@@ -2083,13 +2105,11 @@
     }
     if (pStrokes) ExFreePoolWithTag(pStrokes, TAG_PATH);
 
+    PATH_UnlockPath(flat_path);
+    PATH_Delete(flat_path->BaseObject.hHmgr);
     pNewPath->state = PATH_Closed;
-    if (!(ret = PATH_AssignGdiPath(pPath, pNewPath)))
-        DPRINT1("Assign path failed\n");
-    PATH_DestroyGdiPath(pNewPath);
-    ExFreePoolWithTag(pNewPath, TAG_PATH);
-    PATH_UnlockPath(pPath);
-    return ret;
+    PATH_UnlockPath(pNewPath);
+    return pNewPath;
 }
 
 static inline INT int_from_fixed(FIXED f)
@@ -2118,9 +2138,7 @@
     }
     else if (n == 3)
     {
-        PATH_AddEntry(pPath, &lppt[0], PT_BEZIERTO);
-        PATH_AddEntry(pPath, &lppt[1], PT_BEZIERTO);
-        PATH_AddEntry(pPath, &lppt[2], PT_BEZIERTO);
+        add_points( pPath, lppt, 3, PT_BEZIERTO );
     }
     else
     {
@@ -2136,7 +2154,7 @@
             pt[1] = lppt[i + 1];
             pt[2].x = (lppt[i + 2].x + lppt[i + 1].x) / 2;
             pt[2].y = (lppt[i + 2].y + lppt[i + 1].y) / 2;
-            PATH_BezierTo(pPath, pt, 3);
+            add_points( pPath, pt, 3, PT_BEZIERTO );
             n--;
             i++;
         }
@@ -2144,7 +2162,7 @@
         pt[0] = pt[2];
         pt[1] = lppt[i + 1];
         pt[2] = lppt[i + 2];
-        PATH_BezierTo(pPath, pt, 3);
+        add_points( pPath, pt, 3, PT_BEZIERTO );
     }
 }
 
@@ -2153,23 +2171,17 @@
 FASTCALL
 PATH_add_outline(
     PDC dc,
+    PPATH pPath,
     INT x,
     INT y,
     TTPOLYGONHEADER *header,
     DWORD size)
 {
-    PPATH pPath;
     TTPOLYGONHEADER *start;
     POINT pt;
     BOOL bResult = FALSE;
 
     start = header;
-
-    pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath)
-    {
-        return FALSE;
-    }
 
     while ((char *)header < (char *)start + size)
     {
@@ -2246,7 +2258,6 @@
 
 cleanup:
     IntGdiCloseFigure(pPath);
-    PATH_UnlockPath(pPath);
     return bResult;
 }
 
@@ -2265,10 +2276,24 @@
     UINT count,
     const INT *dx)
 {
-    unsigned int idx;
+    PPATH pPath;
+    unsigned int idx, ggo_flags = GGO_NATIVE;
     POINT offset = {0, 0};
 
+    pPath = PATH_LockPath(dc->dclevel.hPath);
+    if (!pPath)
+    {
+        return FALSE;
+    }
+
+    if (pPath->state != PATH_Open)
+    {
+        DPRINT1("PATH_ExtTextOut not open\n");
+        return FALSE;
+    }
+
     if (!count) return TRUE;
+    if (flags & ETO_GLYPH_INDEX) ggo_flags |= GGO_GLYPH_INDEX;
 
     for (idx = 0; idx < count; idx++)
     {
@@ -2279,30 +2304,38 @@
 
         dwSize = ftGdiGetGlyphOutline(dc,
                                       str[idx],
-                                      GGO_GLYPH_INDEX | GGO_NATIVE,
+                                      ggo_flags,
                                       &gm,
                                       0,
                                       NULL,
                                       &identity,
                                       TRUE);
-        if (dwSize == GDI_ERROR) return FALSE;
+        if (dwSize == GDI_ERROR)
+        {
+           PATH_UnlockPath(pPath);
+           return FALSE;
+        }
 
         /* Add outline only if char is printable */
         if (dwSize)
         {
             outline = ExAllocatePoolWithTag(PagedPool, dwSize, TAG_PATH);
-            if (!outline) return FALSE;
+            if (!outline)
+            {
+               PATH_UnlockPath(pPath);
+               return FALSE;
+            }
 
             ftGdiGetGlyphOutline(dc,
                                  str[idx],
-                                 GGO_GLYPH_INDEX | GGO_NATIVE,
+                                 ggo_flags,
                                  &gm,
                                  dwSize,
                                  outline,
                                  &identity,
                                  TRUE);
 
-            PATH_add_outline(dc, x + offset.x, y + offset.y, outline, dwSize);
+            PATH_add_outline(dc, pPath, x + offset.x, y + offset.y, outline, dwSize);
 
             ExFreePoolWithTag(outline, TAG_PATH);
         }
@@ -2323,6 +2356,7 @@
             offset.y += gm.gmCellIncY;
         }
     }
+    PATH_UnlockPath(pPath);
     return TRUE;
 }
 
@@ -2335,7 +2369,6 @@
 APIENTRY
 NtGdiAbortPath(HDC hDC)
 {
-    PPATH pPath;
     PDC dc = DC_LockDc(hDC);
     if (!dc)
     {
@@ -2343,16 +2376,19 @@
         return FALSE;
     }
 
-    pPath = PATH_LockPath(dc->dclevel.hPath);
-    if (!pPath)
-    {
-        DC_UnlockDc(dc);
-        return FALSE;
-    }
-
-    PATH_EmptyPath(pPath);
-
-    PATH_UnlockPath(pPath);
+    if (!dc->dclevel.hPath)
+    {
+       DC_UnlockDc(dc);
+       return TRUE;
+    }
+
+    if (!PATH_Delete(dc->dclevel.hPath))
+    {
+       DC_UnlockDc(dc);
+       return FALSE;
+    }
+
+    dc->dclevel.hPath = 0;
     dc->dclevel.flPath &= ~DCPATH_ACTIVE;
 
     DC_UnlockDc(dc);
@@ -2399,35 +2435,19 @@
             dc->dclevel.hPath = NULL;
         }
     }
-    pPath = PATH_AllocPathWithHandle();
-    if (!pPath)
-    {
-        EngSetLastError(ERROR_NOT_ENOUGH_MEMORY);
-        return FALSE;
-    }
+    pPath = PATH_CreatePath(NUM_ENTRIES_INITIAL);
     dc->dclevel.flPath |= DCPATH_ACTIVE; // Set active ASAP!
-
     dc->dclevel.hPath = pPath->BaseObject.hHmgr;
-
-    DPRINT("BeginPath 2 h 0x%p p 0x%p\n", dc->dclevel.hPath, pPath);
-    // Path handles are shared. Also due to recursion with in the same thread.
-    GDIOBJ_vUnlockObject((POBJ)pPath);       // Unlock
-    pPath = PATH_LockPath(dc->dclevel.hPath); // Share Lock.
-
-    /* Make sure that path is empty */
-    PATH_EmptyPath(pPath);
-
-    pPath->numEntriesAllocated = NUM_ENTRIES_INITIAL;
-
-    pPath->pPoints = (POINT *)ExAllocatePoolWithTag(PagedPool, NUM_ENTRIES_INITIAL *
sizeof(POINT), TAG_PATH);
-    pPath->pFlags  =  (BYTE *)ExAllocatePoolWithTag(PagedPool, NUM_ENTRIES_INITIAL *
sizeof(BYTE),  TAG_PATH);
-
-    /* Initialize variables for new path */
-    pPath->newStroke = TRUE;
-    pPath->state = PATH_Open;
-
+    IntGetCurrentPositionEx(dc, &pPath->pos);
+    IntLPtoDP( dc, &pPath->pos, 1 );
+    DPRINT("BP : Current pos X %d Y %d\n",pPath->pos.x, pPath->pos.y);
     PATH_UnlockPath(pPath);
     DC_UnlockDc(dc);
+
+    if (!pPath)
+    {
+        return FALSE;
+    }
     return TRUE;
 }
 
@@ -2462,7 +2482,6 @@
     }
     else
     {
-        // FIXME: Check if lasterror is set correctly
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
     }
 
@@ -2496,7 +2515,7 @@
     /* Check that path is currently being constructed */
     if ((pPath->state != PATH_Open) || !(dc->dclevel.flPath & DCPATH_ACTIVE))
     {
-        DPRINT1("EndPath ERROR! 0x%p\n", dc->dclevel.hPath);
+        DPRINT("EndPath ERROR! 0x%p\n", dc->dclevel.hPath);
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
         ret = FALSE;
     }
@@ -2518,7 +2537,7 @@
 NtGdiFillPath(HDC  hDC)
 {
     BOOL ret = FALSE;
-    PPATH pPath;
+    PPATH pPath, pNewPath;
     PDC_ATTR pdcattr;
     PDC dc;
 
@@ -2546,15 +2565,18 @@
     if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY))
         DC_vUpdateFillBrush(dc);
 
-    ret = PATH_FillPath(dc, pPath);
-    if (ret)
-    {
-        /* FIXME: Should the path be emptied even if conversion
-           failed? */
-        PATH_EmptyPath(pPath);
-    }
+    pNewPath = PATH_FlattenPath(pPath);
+
+    ret = PATH_FillPath(dc, pNewPath);
+
+    PATH_UnlockPath(pNewPath);
+    PATH_Delete(pNewPath->BaseObject.hHmgr);
 
     PATH_UnlockPath(pPath);
+    PATH_Delete(pPath->BaseObject.hHmgr);
+    dc->dclevel.hPath = 0;
+    dc->dclevel.flPath &= ~DCPATH_ACTIVE;
+
     DC_vFinishBlit(dc, NULL);
     DC_UnlockDc(dc);
     return ret;
@@ -2566,7 +2588,7 @@
 {
     BOOL Ret = FALSE;
     DC *pDc;
-    PPATH pPath;
+    PPATH pPath, pNewPath = NULL;
 
     DPRINT("Enter %s\n", __FUNCTION__);
 
@@ -2580,13 +2602,26 @@
     pPath = PATH_LockPath(pDc->dclevel.hPath);
     if (!pPath)
     {
+        EngSetLastError( ERROR_CAN_NOT_COMPLETE );
         DC_UnlockDc(pDc);
         return FALSE;
     }
-    if (pPath->state == PATH_Open)
-        Ret = PATH_FlattenPath(pPath);
+
+    if (pPath->state == PATH_Closed)
+    {
+        pNewPath = PATH_FlattenPath(pPath);
+    }
 
     PATH_UnlockPath(pPath);
+
+    if (pNewPath)
+    {
+       PATH_Delete(pDc->dclevel.hPath);
+       pDc->dclevel.hPath = pNewPath->BaseObject.hHmgr;
+       PATH_UnlockPath(pNewPath);
+       Ret = TRUE;
+    }
+
     DC_UnlockDc(pDc);
     return Ret;
 }
@@ -2636,6 +2671,7 @@
     PPATH pPath;
 
     DC *dc = DC_LockDc(hDC);
+    DPRINT("NtGdiGetPath start\n");
     if (!dc)
     {
         DPRINT1("Can't lock dc!\n");
@@ -2689,6 +2725,7 @@
     }
 
 done:
+    DPRINT("NtGdiGetPath exit %d\n",ret);
     PATH_UnlockPath(pPath);
     DC_UnlockDc(dc);
     return ret;
@@ -2698,8 +2735,9 @@
 APIENTRY
 NtGdiPathToRegion(HDC  hDC)
 {
-    PPATH pPath;
+    PPATH pPath, pNewPath;
     HRGN  hrgnRval = 0;
+    int Ret;
     PREGION Rgn;
     DC *pDc;
     PDC_ATTR pdcattr;
@@ -2709,7 +2747,7 @@
     pDc = DC_LockDc(hDC);
     if (!pDc)
     {
-        DPRINT1("Failed to lock DC %p\n", hDC);
+        DPRINT("Failed to lock DC %p\n", hDC);
         EngSetLastError(ERROR_INVALID_PARAMETER);
         return NULL;
     }
@@ -2719,7 +2757,7 @@
     pPath = PATH_LockPath(pDc->dclevel.hPath);
     if (!pPath)
     {
-        DPRINT1("Failed to lock DC path %p\n", pDc->dclevel.hPath);
+        DPRINT("Failed to lock DC path %p\n", pDc->dclevel.hPath);
         DC_UnlockDc(pDc);
         return NULL;
     }
@@ -2727,7 +2765,7 @@
     if (pPath->state != PATH_Closed)
     {
         // FIXME: Check that setlasterror is being called correctly
-        DPRINT1("Path is not closed!\n");
+        DPRINT("Path is not closed!\n");
         EngSetLastError(ERROR_CAN_NOT_COMPLETE);
     }
     else
@@ -2736,27 +2774,35 @@
         Rgn = REGION_AllocUserRgnWithHandle(1);
         if (!Rgn)
         {
-            DPRINT1("Failed to allocate a region\n");
+            DPRINT("Failed to allocate a region\n");
             PATH_UnlockPath(pPath);
             DC_UnlockDc(pDc);
             return NULL;
         }
         hrgnRval = Rgn->BaseObject.hHmgr;
-        /* FIXME: Should we empty the path even if conversion failed? */
-        if (PATH_PathToRegion(pPath, pdcattr->jFillMode, Rgn))
-        {
-            PATH_EmptyPath(pPath);
-            REGION_UnlockRgn(Rgn);
-        }
-        else
-        {
-            DPRINT1("PATH_PathToRegion failed\n");
+
+        pNewPath = PATH_FlattenPath(pPath);
+
+        Ret = PATH_PathToRegion(pNewPath, pdcattr->jFillMode, Rgn);
+
+        PATH_UnlockPath(pNewPath);
+        PATH_Delete(pNewPath->BaseObject.hHmgr);
+
+        if (!Ret)
+        {
+            DPRINT("PATH_PathToRegion failed\n");
             REGION_Delete(Rgn);
             hrgnRval = NULL;
         }
+        else
+            REGION_UnlockRgn(Rgn);
     }
 
     PATH_UnlockPath(pPath);
+    PATH_Delete(pDc->dclevel.hPath);
+    pDc->dclevel.hPath = NULL;
+    pDc->dclevel.flPath &= ~DCPATH_ACTIVE;
+
     DC_UnlockDc(pDc);
     return hrgnRval;
 }
@@ -2807,10 +2853,10 @@
 {
     DC *pDc;
     PDC_ATTR pdcattr;
-    PPATH pPath;
+    PPATH pPath, pNewPath;
     BOOL bRet = FALSE;
 
-    DPRINT1("Enter %s\n", __FUNCTION__);
+    DPRINT("Enter %s\n", __FUNCTION__);
 
     if (!(pDc = DC_LockDc(hDC)))
     {
@@ -2834,11 +2880,19 @@
     if (pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY))
         DC_vUpdateLineBrush(pDc);
 
-    bRet = PATH_FillPath(pDc, pPath);
-    if (bRet) bRet = PATH_StrokePath(pDc, pPath);
-    if (bRet) PATH_EmptyPath(pPath);
+    pNewPath = PATH_FlattenPath(pPath);
+
+    bRet = PATH_FillPath(pDc, pNewPath);
+    if (bRet) bRet = PATH_StrokePath(pDc, pNewPath);
+
+    PATH_UnlockPath(pNewPath);
+    PATH_Delete(pNewPath->BaseObject.hHmgr);
 
     PATH_UnlockPath(pPath);
+    PATH_Delete(pPath->BaseObject.hHmgr);
+    pDc->dclevel.hPath = 0;
+    pDc->dclevel.flPath &= ~DCPATH_ACTIVE;
+
     DC_vFinishBlit(pDc, NULL);
     DC_UnlockDc(pDc);
     return bRet;
@@ -2878,9 +2932,12 @@
     bRet = PATH_StrokePath(pDc, pPath);
 
     DC_vFinishBlit(pDc, NULL);
-    PATH_EmptyPath(pPath);
 
     PATH_UnlockPath(pPath);
+    PATH_Delete(pPath->BaseObject.hHmgr);
+    pDc->dclevel.hPath = 0;
+    pDc->dclevel.flPath &= ~DCPATH_ACTIVE;
+
     DC_UnlockDc(pDc);
     return bRet;
 }
@@ -2889,16 +2946,26 @@
 APIENTRY
 NtGdiWidenPath(HDC  hDC)
 {
-    BOOL Ret;
+    PPATH pPath;
+    BOOL Ret = FALSE;
     PDC pdc = DC_LockDc(hDC);
+    DPRINT("NtGdiWidenPat Enter\n");
     if (!pdc)
     {
         EngSetLastError(ERROR_INVALID_PARAMETER);
         return FALSE;
     }
 
-    Ret = PATH_WidenPath(pdc);
+    pPath = PATH_WidenPath(pdc);
+    if (pPath)
+    {
+       DPRINT("WindenPath New Path\n");
+       PATH_Delete(pdc->dclevel.hPath);
+       pdc->dclevel.hPath = pPath->BaseObject.hHmgr;
+       Ret = TRUE;
+    }
     DC_UnlockDc(pdc);
+    DPRINT("NtGdiWidenPat Ret %d\n",Ret);
     return Ret;
 }
 

Modified: trunk/reactos/win32ss/gdi/ntgdi/path.h
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/path.h?r…
==============================================================================
--- trunk/reactos/win32ss/gdi/ntgdi/path.h	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/path.h	[iso-8859-1] Thu Sep  8 22:37:41 2016
@@ -34,6 +34,7 @@
   int          numEntriesUsed;
   int          numEntriesAllocated;
   BOOL         newStroke;
+  POINT        pos;
 } PATH, *PPATH;
 
 typedef struct _EPATHOBJ
@@ -50,15 +51,13 @@
 
 #define PATH_IsPathOpen(dclevel) ( ((dclevel).hPath) && ((dclevel).flPath &
DCPATH_ACTIVE) )
 
-BOOL FASTCALL PATH_Arc (PDC dc, INT x1, INT y1, INT x2, INT y2, INT xStart, INT yStart,
INT xEnd, INT yEnd, INT lines);
+BOOL FASTCALL PATH_Arc (PDC dc, INT x1, INT y1, INT x2, INT y2, INT xStart, INT yStart,
INT xEnd, INT yEnd, INT direction, INT lines);
 BOOL FASTCALL PATH_Ellipse (PDC dc, INT x1, INT y1, INT x2, INT y2);
 VOID FASTCALL PATH_EmptyPath (PPATH pPath);
 BOOL FASTCALL PATH_LineTo (PDC dc, INT x, INT y);
 BOOL FASTCALL PATH_PolyBezier (PDC dc, const POINT *pts, DWORD cbPoints);
 BOOL FASTCALL PATH_PolyBezierTo (PDC dc, const POINT *pts, DWORD cbPoints);
 BOOL FASTCALL PATH_PolyDraw(PDC dc, const POINT *pts, const BYTE *types, DWORD
cbPoints);
-BOOL FASTCALL PATH_Polygon (PDC dc, const POINT *pts, DWORD cbPoints);
-BOOL FASTCALL PATH_Polyline (PDC dc, const POINT *pts, DWORD cbPoints);
 BOOL FASTCALL PATH_PolylineTo (PDC dc, const POINT *pts, DWORD cbPoints);
 BOOL FASTCALL PATH_PolyPolygon ( PDC dc, const POINT* pts, const INT* counts, UINT
polygons);
 BOOL FASTCALL PATH_PolyPolyline( PDC dc, const POINT* pts, const DWORD* counts, DWORD
polylines);
@@ -71,7 +70,7 @@
 BOOL FASTCALL PATH_AddFlatBezier (PPATH pPath, POINT *pt, BOOL closed);
 BOOL FASTCALL PATH_DoArcPart (PPATH pPath, FLOAT_POINT corners[], double angleStart,
double angleEnd, BYTE startEntryType);
 BOOL FASTCALL PATH_FillPath( PDC dc, PPATH pPath );
-BOOL FASTCALL PATH_FlattenPath (PPATH pPath);
+PPATH FASTCALL PATH_FlattenPath (PPATH pPath);
 VOID FASTCALL PATH_NormalizePoint (FLOAT_POINT corners[], const FLOAT_POINT *pPoint,
double *pX, double *pY);
 
 BOOL FASTCALL PATH_ReserveEntries (PPATH pPath, INT numEntries);
@@ -83,3 +82,6 @@
 BOOL FASTCALL PATH_Delete(HPATH hPath);
 
 VOID FASTCALL IntGetCurrentPositionEx(PDC dc, LPPOINT pt);
+
+BOOL PATH_RestorePath( DC *, DC *);
+BOOL PATH_SavePath( DC *, DC *);