[PATCH 0/4] MR1198: wineps.drv: Add DrvDocumentPropertySheets export.

[PATCH 1/4] wineps.drv: Store PSDRV_DEVMODE copy in PSDRV_DLGINFO structure.

[Piotr Caban]

From: Piotr Caban

---
 dlls/wineps.drv/driver.c | 44 +++++++++++++++++++---------------------
 1 file changed, 21 insertions(+), 23 deletions(-)

diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c
index 3b0300f3fd2..4d7dbdefab0 100644
--- a/dlls/wineps.drv/driver.c
+++ b/dlls/wineps.drv/driver.c
@@ -218,7 +218,7 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI
 typedef struct
 {
     PRINTERINFO *pi;
-    PSDRV_DEVMODE *dlgdm;
+    PSDRV_DEVMODE dlgdm;
 } PSDRV_DLGINFO;

/****************************************************************
@@ -339,17 +339,17 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
         }
         TRACE("Setting pagesize to item %d, WinPage %d (%s), PaperSize %.2fx%.2f\n", Cursel,
               ps->WinPage, ps->FullName, ps->PaperDimension->x, ps->PaperDimension->y);
-        di->dlgdm->dmPublic.dmPaperSize = ps->WinPage;
-        di->dlgdm->dmPublic.dmFields |= DM_PAPERSIZE;
+        di->dlgdm.dmPublic.dmPaperSize = ps->WinPage;
+        di->dlgdm.dmPublic.dmFields |= DM_PAPERSIZE;

-        di->dlgdm->dmPublic.dmPaperWidth  = paper_size_from_points(ps->PaperDimension->x);
-        di->dlgdm->dmPublic.dmPaperLength = paper_size_from_points(ps->PaperDimension->y);
-        di->dlgdm->dmPublic.dmFields |= DM_PAPERLENGTH | DM_PAPERWIDTH;
+        di->dlgdm.dmPublic.dmPaperWidth  = paper_size_from_points(ps->PaperDimension->x);
+        di->dlgdm.dmPublic.dmPaperLength = paper_size_from_points(ps->PaperDimension->y);
+        di->dlgdm.dmPublic.dmFields |= DM_PAPERLENGTH | DM_PAPERWIDTH;

-        if (di->dlgdm->dmPublic.dmSize >= FIELD_OFFSET(DEVMODEW, dmFormName) + CCHFORMNAME * sizeof(WCHAR))
+        if (di->dlgdm.dmPublic.dmSize >= FIELD_OFFSET(DEVMODEW, dmFormName) + CCHFORMNAME * sizeof(WCHAR))
         {
-            MultiByteToWideChar(CP_ACP, 0, ps->FullName, -1, di->dlgdm->dmPublic.dmFormName, CCHFORMNAME);
-            di->dlgdm->dmPublic.dmFields |= DM_FORMNAME;
+            MultiByteToWideChar(CP_ACP, 0, ps->FullName, -1, di->dlgdm.dmPublic.dmFormName, CCHFORMNAME);
+            di->dlgdm.dmPublic.dmFields |= DM_FORMNAME;
         }
         SendMessageW(GetParent(hwnd), PSM_CHANGED, 0, 0);
       }
@@ -358,9 +358,9 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
     case IDD_ORIENT_LANDSCAPE:
       TRACE("Setting orientation to %s\n", wParam == IDD_ORIENT_PORTRAIT ?
             "portrait" : "landscape");
-      di->dlgdm->dmPublic.dmOrientation = wParam == IDD_ORIENT_PORTRAIT ?
+      di->dlgdm.dmPublic.dmOrientation = wParam == IDD_ORIENT_PORTRAIT ?
         DMORIENT_PORTRAIT : DMORIENT_LANDSCAPE;
-      di->dlgdm->dmPublic.dmFields |= DM_ORIENTATION;
+      di->dlgdm.dmPublic.dmFields |= DM_ORIENTATION;
       SendMessageW(GetParent(hwnd), PSM_CHANGED, 0, 0);
       break;
     case IDD_DUPLEX:
@@ -373,8 +373,8 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
             i++;
         }
         TRACE("Setting duplex to item %d Winduplex = %d\n", Cursel, duplex->WinDuplex);
-        di->dlgdm->dmPublic.dmDuplex = duplex->WinDuplex;
-        di->dlgdm->dmPublic.dmFields |= DM_DUPLEX;
+        di->dlgdm.dmPublic.dmDuplex = duplex->WinDuplex;
+        di->dlgdm.dmPublic.dmFields |= DM_DUPLEX;
         SendMessageW(GetParent(hwnd), PSM_CHANGED, 0, 0);
       }
       break;
@@ -392,16 +392,16 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
         resy = HIWORD(data);
         TRACE("Setting resolution to %dx%d\n", resx, resy);

-        di->dlgdm->dmPublic.dmPrintQuality = resx;
-        di->dlgdm->dmPublic.dmFields |= DM_PRINTQUALITY;
+        di->dlgdm.dmPublic.dmPrintQuality = resx;
+        di->dlgdm.dmPublic.dmFields |= DM_PRINTQUALITY;

-        di->dlgdm->dmPublic.dmYResolution = resy;
-        di->dlgdm->dmPublic.dmFields |= DM_YRESOLUTION;
+        di->dlgdm.dmPublic.dmYResolution = resy;
+        di->dlgdm.dmPublic.dmFields |= DM_YRESOLUTION;

if (di->pi->Devmode->dmPublic.dmFields & DM_LOGPIXELS)
         {
-            di->dlgdm->dmPublic.dmLogPixels = resx;
-            di->dlgdm->dmPublic.dmFields |= DM_LOGPIXELS;
+            di->dlgdm.dmPublic.dmLogPixels = resx;
+            di->dlgdm.dmPublic.dmFields |= DM_LOGPIXELS;
         }

SendMessageW(GetParent(hwnd), PSM_CHANGED, 0, 0);
@@ -416,7 +416,7 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
     di = (PSDRV_DLGINFO *)GetWindowLongPtrW(hwnd, DWLP_USER);
     switch(nmhdr->code) {
     case PSN_APPLY:
-      *di->pi->Devmode = *di->dlgdm;
+      *di->pi->Devmode = di->dlgdm;
       SetWindowLongPtrW(hwnd, DWLP_MSGRESULT, PSNRET_NOERROR);
       return TRUE;

@@ -479,7 +479,6 @@ INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *outpu
     PROPSHEETPAGEW psp;
     PROPSHEETHEADERW psh;
     PSDRV_DLGINFO di;
-    PSDRV_DEVMODE dlgdm;
     WCHAR SetupW[64];

LoadStringW(PSDRV_hInstance, IDS_SETUP, SetupW, ARRAY_SIZE(SetupW));
@@ -488,9 +487,8 @@ INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *outpu
    					    "CreatePropertySheetPageW");
     pPropertySheet = (void*)GetProcAddress(hinstComctl32, "PropertySheetW");
     memset(&psp,0,sizeof(psp));
-    dlgdm = *pi->Devmode;
     di.pi = pi;
-    di.dlgdm = &dlgdm;
+    di.dlgdm = *pi->Devmode;
     psp.dwSize = sizeof(psp);
     psp.hInstance = PSDRV_hInstance;
     psp.pszTemplate = L"PAPER";

[PATCH 2/4] wineps.drv: Add DrvDocumentPropertySheets implementation.

[Piotr Caban]

From: Piotr Caban

---
 dlls/wineps.drv/driver.c        | 91 +++++++++++++++++++++++++++++++++
 dlls/wineps.drv/wineps.drv.spec |  1 +
 2 files changed, 92 insertions(+)

diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c
index 4d7dbdefab0..a3489b3b9d9 100644
--- a/dlls/wineps.drv/driver.c
+++ b/dlls/wineps.drv/driver.c
@@ -23,6 +23,7 @@
  */

#include <stdarg.h>
+#include <stdlib.h>

#include "windef.h"
 #include "winbase.h"
@@ -39,6 +40,17 @@

WINE_DEFAULT_DEBUG_CHANNEL(psdrv);

+typedef struct {
+    WORD cbSize;
+    WORD Reserved;
+    HANDLE hPrinter;
+    LPCWSTR pszPrinterName;
+    PDEVMODEW pdmIn;
+    PDEVMODEW pdmOut;
+    DWORD cbOut;
+    DWORD fMode;
+} DOCUMENTPROPERTYHEADERW;
+
 /* convert points to paper size units (10th of a millimeter) */
 static inline int paper_size_from_points( float size )
 {
@@ -519,6 +531,85 @@ INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *outpu
   return IDOK;
 }

+/******************************************************************************
+ *           DrvDocumentPropertySheets (wineps.drv.@)
+ */
+LONG WINAPI DrvDocumentPropertySheets(PROPSHEETUI_INFO *info, LPARAM lparam)
+{
+    DOCUMENTPROPERTYHEADERW *dph;
+    PRINTERINFO *pi;
+
+    TRACE("(info=%p, lparam=%ld)\n", info, lparam);
+
+    dph = (DOCUMENTPROPERTYHEADERW *)(info ? info->lParamInit : lparam);
+
+    if (!(pi = PSDRV_FindPrinterInfo(dph->pszPrinterName))) return ERR_CPSUI_GETLASTERROR;
+
+    if (!info || info->Reason == PROPSHEETUI_REASON_INIT)
+    {
+        /* If dph->fMode == 0, return size of DEVMODE structure */
+        if (!dph->fMode || !dph->pdmOut)
+        {
+            dph->cbOut = pi->Devmode->dmPublic.dmSize + pi->Devmode->dmPublic.dmDriverExtra;
+            return dph->cbOut;
+        }
+
+        /* If DM_MODIFY is set, change settings in accordance with pdmIn */
+        if ((dph->fMode & DM_MODIFY) && dph->pdmIn)
+        {
+            TRACE("DM_MODIFY set. devIn->dmFields = %08lx\n", dph->pdmIn->dmFields);
+            PSDRV_MergeDevmodes(pi->Devmode, (PSDRV_DEVMODE *)dph->pdmIn, pi);
+        }
+
+        /* If DM_PROMPT is set, present modal dialog box */
+        if (dph->fMode & DM_PROMPT) {
+            PROPSHEETPAGEW psp;
+            PSDRV_DLGINFO *di;
+
+            if (!info)
+            {
+                ERR("DM_PROMPT passed with NULL PROPSHEETUI_INFO\n");
+                return ERR_CPSUI_GETLASTERROR;
+            }
+
+            di = malloc(sizeof(*di));
+            if (!di)
+                return ERR_CPSUI_ALLOCMEM_FAILED;
+            info->UserData = (ULONG_PTR)di;
+
+            memset(&psp,0,sizeof(psp));
+            di->pi = pi;
+            di->dlgdm = *pi->Devmode;
+            psp.dwSize = sizeof(psp);
+            psp.hInstance = PSDRV_hInstance;
+            psp.pszTemplate = L"PAPER";
+            psp.pszIcon = NULL;
+            psp.pfnDlgProc = PSDRV_PaperDlgProc;
+            psp.lParam = (LPARAM)di;
+
+            if (!info->pfnComPropSheet(info->hComPropSheet, CPSFUNC_ADD_PROPSHEETPAGEW, (LPARAM)&psp, 0))
+                return ERR_CPSUI_GETLASTERROR;
+        }
+    }
+
+    if (info && info->Reason == PROPSHEETUI_REASON_DESTROY)
+        free((void *)info->UserData);
+
+    if (!info || (info->Reason == PROPSHEETUI_REASON_DESTROY && lparam))
+    {
+        /* If DM_UPDATE is set, should write settings to environment and initialization file */
+        if (dph->fMode & DM_UPDATE)
+            FIXME("Mode DM_UPDATE.  Just do the same as DM_COPY\n");
+
+        /* If DM_COPY is set, should write settings to pdmOut */
+        if (dph->pdmOut && (dph->fMode & (DM_COPY | DM_UPDATE)))
+            memcpy(dph->pdmOut, &pi->Devmode->dmPublic,
+                    pi->Devmode->dmPublic.dmSize + pi->Devmode->dmPublic.dmDriverExtra);
+    }
+
+    return CPSUI_OK;
+}
+
 /******************************************************************************
  *           DrvDeviceCapabilities    (wineps.drv.@)
  */
diff --git a/dlls/wineps.drv/wineps.drv.spec b/dlls/wineps.drv/wineps.drv.spec
index 470bbd8e5c6..c5775b0328b 100644
--- a/dlls/wineps.drv/wineps.drv.spec
+++ b/dlls/wineps.drv/wineps.drv.spec
@@ -3,3 +3,4 @@
 # Printer driver config exports
 @ stdcall DrvDeviceCapabilities(ptr wstr long ptr ptr)
 @ stdcall DrvDocumentProperties(long wstr ptr ptr long)
+@ stdcall DrvDocumentPropertySheets(ptr long)

[PATCH 3/4] winspool.drv: Use DrvDocumentPropertySheets driver entry in DocumentProperties.

[Piotr Caban]

From: Piotr Caban

---
 dlls/winspool.drv/Makefile.in |  2 +-
 dlls/winspool.drv/info.c      | 64 +++++++++++++++++++++++++++++------
 2 files changed, 54 insertions(+), 12 deletions(-)

diff --git a/dlls/winspool.drv/Makefile.in b/dlls/winspool.drv/Makefile.in
index b4948ccb5e8..3f6d32cb44d 100644
--- a/dlls/winspool.drv/Makefile.in
+++ b/dlls/winspool.drv/Makefile.in
@@ -1,7 +1,7 @@
 MODULE    = winspool.drv
 UNIXLIB   = winspool.so
 IMPORTLIB = winspool
-IMPORTS   = user32 gdi32 advapi32
+IMPORTS   = user32 gdi32 advapi32 compstui
 UNIX_CFLAGS  = $(CUPS_CFLAGS)
 UNIX_LIBS    = $(APPLICATIONSERVICES_LIBS)

diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c
index 3806384ee08..43ff1e459a2 100644
--- a/dlls/winspool.drv/info.c
+++ b/dlls/winspool.drv/info.c
@@ -51,6 +51,7 @@
 #include "wine/heap.h"
 #include <wine/unixlib.h>

+#include "ddk/compstui.h"
 #include "ddk/winsplp.h"
 #include "wspool.h"

@@ -115,11 +116,27 @@ typedef struct

/* entry points */
     DWORD (WINAPI *pDrvDeviceCapabilities)(HANDLE, const WCHAR *, WORD, void *, const DEVMODEW *);
-    INT   (WINAPI *pDrvDocumentProperties)(HWND, const WCHAR *, DEVMODEW *, DEVMODEW *, DWORD);
+    LONG  (WINAPI *pDrvDocumentPropertySheets)(PROPSHEETUI_INFO*, LPARAM);

WCHAR name[1];
 } config_module_t;

+typedef struct {
+    WORD cbSize;
+    WORD Reserved;
+    HANDLE hPrinter;
+    LPCWSTR pszPrinterName;
+    PDEVMODEW pdmIn;
+    PDEVMODEW pdmOut;
+    DWORD cbOut;
+    DWORD fMode;
+} DOCUMENTPROPERTYHEADERW;
+
+typedef struct {
+    DOCUMENTPROPERTYHEADERW dph;
+    config_module_t *config;
+} document_property_t;
+
 /* ############################### */

static opened_printer_t **printer_handles;
@@ -502,7 +519,7 @@ static config_module_t *get_config_module(const WCHAR *device, BOOL grab)
     ret->ref = 2; /* one for config_module and one for the caller */
     ret->module = driver_module;
     ret->pDrvDeviceCapabilities = (void *)GetProcAddress(driver_module, "DrvDeviceCapabilities");
-    ret->pDrvDocumentProperties = (void *)GetProcAddress(driver_module, "DrvDocumentProperties");
+    ret->pDrvDocumentPropertySheets = (void *)GetProcAddress(driver_module, "DrvDocumentPropertySheets");
     wcscpy( ret->name, device );

wine_rb_put(&config_modules, ret->name, &ret->entry);
@@ -1896,6 +1913,18 @@ LONG WINAPI DocumentPropertiesA(HWND hwnd, HANDLE printer, char *device_name, DE
     return ret;
 }

+static LONG WINAPI document_callback(PROPSHEETUI_INFO *info, LPARAM lparam)
+{
+    if (info->Reason == PROPSHEETUI_REASON_INIT)
+    {
+        document_property_t *dp = (document_property_t *)info->lParamInit;
+
+        if(!info->pfnComPropSheet(info->hComPropSheet, CPSFUNC_ADD_PFNPROPSHEETUIW,
+                    (LPARAM)dp->config->pDrvDocumentPropertySheets, (LPARAM)&dp->dph))
+            return ERR_CPSUI_GETLASTERROR;
+    }
+    return CPSUI_OK;
+}

/*****************************************************************************
  *          DocumentPropertiesW (WINSPOOL.@)
@@ -1905,8 +1934,8 @@ LONG WINAPI DocumentPropertiesW(HWND hWnd, HANDLE hPrinter,
    			LPDEVMODEW pDevModeOutput,
    			LPDEVMODEW pDevModeInput, DWORD fMode)
 {
-    config_module_t *config = NULL;
-    const WCHAR *device = NULL;
+    document_property_t dp;
+    const WCHAR *device;
     LONG ret;

TRACE("(%p,%p,%s,%p,%p,%ld)\n",
@@ -1918,17 +1947,30 @@ LONG WINAPI DocumentPropertiesW(HWND hWnd, HANDLE hPrinter,
         return -1;
     }

-    config = get_config_module(device, TRUE);
-    if (!config) {
+    dp.dph.cbSize = sizeof(dp.dph);
+    dp.dph.Reserved = 0;
+    dp.dph.hPrinter = hPrinter;
+    dp.dph.pszPrinterName = device;
+    dp.dph.pdmIn = pDevModeInput;
+    dp.dph.pdmOut = pDevModeOutput;
+    dp.dph.cbOut = dp.dph.pdmOut ? dp.dph.pdmOut->dmSize : 0;
+    dp.dph.fMode = fMode;
+    dp.config = get_config_module(device, TRUE);
+    if (!dp.config) {
         ERR("Could not load config module for %s\n", debugstr_w(device));
         return -1;
     }

-    /* FIXME: This uses Wine-specific config file entry point.
-     * We should use DrvDevicePropertySheets instead (requires CPSUI support first).
-     */
-    ret = config->pDrvDocumentProperties(hWnd, device, pDevModeOutput, pDevModeInput, fMode);
-    release_config_module(config);
+    if (!(fMode & ~(DM_IN_BUFFER | DM_OUT_BUFFER | DM_OUT_DEFAULT))) {
+        ret = dp.config->pDrvDocumentPropertySheets(NULL, (LPARAM)&dp.dph);
+
+        if ((!fMode || !pDevModeOutput) && dp.dph.cbOut != ret)
+            FIXME("size mismatch: ret = %ld cbOut = %ld\n", ret, dp.dph.cbOut);
+    } else {
+        ret = CommonPropertySheetUIW(hWnd, document_callback, (LPARAM)&dp, NULL);
+    }
+
+    release_config_module(dp.config);
     return ret;
 }

[PATCH 4/4] wineps.drv: Remove DrvDocumentProperties entry point.

[Piotr Caban]

From: Piotr Caban

---
 dlls/wineps.drv/driver.c        | 87 ---------------------------------
 dlls/wineps.drv/psdlg.h         |  1 -
 dlls/wineps.drv/wineps.drv.spec |  1 -
 dlls/wineps.drv/wineps.rc       |  1 -
 4 files changed, 90 deletions(-)

diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c
index a3489b3b9d9..95259e9eff3 100644
--- a/dlls/wineps.drv/driver.c
+++ b/dlls/wineps.drv/driver.c
@@ -444,93 +444,6 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg,
   return TRUE;
 }

-
-static HPROPSHEETPAGE (WINAPI *pCreatePropertySheetPage) (LPCPROPSHEETPAGEW);
-static int (WINAPI *pPropertySheet) (LPCPROPSHEETHEADERW);
-
-/******************************************************************************
- *           DrvDocumentProperties    (wineps.drv.@)
- *
- *  Retrieves or modifies device-initialization information for the PostScript
- *  driver, or displays a driver-supplied dialog box for configuring the driver.
- *
- * RETURNS
- *  Returns size of DEVMODE structure if wMode is 0.  Otherwise, IDOK is returned for success
- *  for both dialog and non-dialog operations.  IDCANCEL is returned if the dialog box was cancelled.
- *  A return value less than zero is returned if a non-dialog operation fails.
- *
- * BUGS
- *
- * Just returns default devmode at the moment.  No use of initialization file.
- */
-INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *output,
-                                 DEVMODEW *input, DWORD mode)
-{
-  PRINTERINFO *pi;
-
-  TRACE("(hwnd=%p, Device='%s', devOut=%p, devIn=%p, Mode=%04lx)\n",
-        hwnd, debugstr_w(device), output, input, mode);
-
-  if (!(pi = PSDRV_FindPrinterInfo(device))) return -1;
-
-  /* If mode == 0, return size of DEVMODE structure */
-  if (!mode)
-      return pi->Devmode->dmPublic.dmSize + pi->Devmode->dmPublic.dmDriverExtra;
-
-  /* If DM_MODIFY is set, change settings in accordance with lpdmInput */
-  if ((mode & DM_MODIFY) && input)
-  {
-    TRACE("DM_MODIFY set. devIn->dmFields = %08lx\n", input->dmFields);
-    PSDRV_MergeDevmodes(pi->Devmode, (PSDRV_DEVMODE *)input, pi);
-  }
-
-  /* If DM_PROMPT is set, present modal dialog box */
-  if (mode & DM_PROMPT) {
-    HINSTANCE hinstComctl32;
-    HPROPSHEETPAGE hpsp[1];
-    PROPSHEETPAGEW psp;
-    PROPSHEETHEADERW psh;
-    PSDRV_DLGINFO di;
-    WCHAR SetupW[64];
-
-    LoadStringW(PSDRV_hInstance, IDS_SETUP, SetupW, ARRAY_SIZE(SetupW));
-    hinstComctl32 = LoadLibraryA("comctl32.dll");
-    pCreatePropertySheetPage = (void*)GetProcAddress(hinstComctl32,
-						    "CreatePropertySheetPageW");
-    pPropertySheet = (void*)GetProcAddress(hinstComctl32, "PropertySheetW");
-    memset(&psp,0,sizeof(psp));
-    di.pi = pi;
-    di.dlgdm = *pi->Devmode;
-    psp.dwSize = sizeof(psp);
-    psp.hInstance = PSDRV_hInstance;
-    psp.pszTemplate = L"PAPER";
-    psp.pszIcon = NULL;
-    psp.pfnDlgProc = PSDRV_PaperDlgProc;
-    psp.lParam = (LPARAM)&di;
-    hpsp[0] = pCreatePropertySheetPage(&psp);
-
-    memset(&psh, 0, sizeof(psh));
-    psh.dwSize = sizeof(psh);
-    psh.pszCaption = SetupW;
-    psh.nPages = 1;
-    psh.hwndParent = hwnd;
-    psh.phpage = hpsp;
-
-    pPropertySheet(&psh);
-
-  }
-
-  /* If DM_UPDATE is set, should write settings to environment and initialization file */
-  if (mode & DM_UPDATE)
-    FIXME("Mode DM_UPDATE.  Just do the same as DM_COPY\n");
-
-  /* If DM_COPY is set, should write settings to lpdmOutput */
-  if (output && (mode & (DM_COPY | DM_UPDATE)))
-    memcpy( output, &pi->Devmode->dmPublic,
-            pi->Devmode->dmPublic.dmSize + pi->Devmode->dmPublic.dmDriverExtra );
-  return IDOK;
-}
-
 /******************************************************************************
  *           DrvDocumentPropertySheets (wineps.drv.@)
  */
diff --git a/dlls/wineps.drv/psdlg.h b/dlls/wineps.drv/psdlg.h
index b8f4bc3eed8..09ce9012d50 100644
--- a/dlls/wineps.drv/psdlg.h
+++ b/dlls/wineps.drv/psdlg.h
@@ -32,6 +32,5 @@
 #define IDD_QUALITY 112

#define IDS_DPI 1000
-#define IDS_SETUP 1001

#endif /* __WINE_PSDLG_H */
diff --git a/dlls/wineps.drv/wineps.drv.spec b/dlls/wineps.drv/wineps.drv.spec
index c5775b0328b..365965c69cf 100644
--- a/dlls/wineps.drv/wineps.drv.spec
+++ b/dlls/wineps.drv/wineps.drv.spec
@@ -2,5 +2,4 @@

# Printer driver config exports
 @ stdcall DrvDeviceCapabilities(ptr wstr long ptr ptr)
-@ stdcall DrvDocumentProperties(long wstr ptr ptr long)
 @ stdcall DrvDocumentPropertySheets(ptr long)
diff --git a/dlls/wineps.drv/wineps.rc b/dlls/wineps.drv/wineps.rc
index 7a657c833e6..35b3e5eb167 100644
--- a/dlls/wineps.drv/wineps.rc
+++ b/dlls/wineps.drv/wineps.rc
@@ -46,7 +46,6 @@ FONT 8, "MS Shell Dlg"
 STRINGTABLE
 {
  IDS_DPI "#msgctxt#unit: dots/inch#dpi"
- IDS_SETUP "Setup"
 }

LANGUAGE LANG_NEUTRAL, SUBLANG_NEUTRAL

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/wineps.drv/driver.c:
> typedef struct
>  {
>      PRINTERINFO *pi;
> - PSDRV_DEVMODE *dlgdm;
> - PSDRV_DEVMODE dlgdm;

Do we actually need to do this?  This would prevent PSDRV_DEVMODE being of variable length (see the `numInstalledOptions` member (not that we use it at the moment).

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/wineps.drv/driver.c:
> WINE_DEFAULT_DEBUG_CHANNEL(psdrv);
>
> +typedef struct {

It's not very standard in this file but:
```suggestion:-0+0
typedef struct
{
```

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/wineps.drv/driver.c:
> -
>    /* If dph->fMode == 0, return size of DEVMODE structure */
> -
>    if (!dph->fMode || !dph->pdmOut)
> -
>    {
> -
>        dph->cbOut = pi->Devmode->dmPublic.dmSize + pi->Devmode->dmPublic.dmDriverExtra;
> -
>        return dph->cbOut;
> -
>    }
> -
> -
>    /* If DM_MODIFY is set, change settings in accordance with pdmIn */
> -
>    if ((dph->fMode & DM_MODIFY) && dph->pdmIn)
> -
>    {
> -
>        TRACE("DM_MODIFY set. devIn->dmFields = %08lx\n", dph->pdmIn->dmFields);
> -
>        PSDRV_MergeDevmodes(pi->Devmode, (PSDRV_DEVMODE *)dph->pdmIn, pi);
> -
>    }
> -
> -
>    /* If DM_PROMPT is set, present modal dialog box */
> -
>    if (dph->fMode & DM_PROMPT) {

```suggestion:-0+0
        if (dph->fMode & DM_PROMPT)
        {
```

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/winspool.drv/info.c:
>  return ret;
>
> }
>
> +static LONG WINAPI document_callback(PROPSHEETUI_INFO *info, LPARAM lparam)
> +{
> - if (info->Reason == PROPSHEETUI_REASON_INIT)
> - {
> -
>    document_property_t *dp = (document_property_t *)info->lParamInit;
> -
> -
>    if(!info->pfnComPropSheet(info->hComPropSheet, CPSFUNC_ADD_PFNPROPSHEETUIW,

```suggestion:-0+0
        if (!info->pfnComPropSheet(info->hComPropSheet, CPSFUNC_ADD_PFNPROPSHEETUIW,
```