[PATCH] comdlg32: Remove unused code for the Button psh1

Detlef Riekenberg wine.dev at web.de
Wed Jan 6 16:52:11 CST 2010


This Button is not present in the 32bit print dialog
I do not think, that we need to support this feature in commdlg.dll16

--
By by ... Detlef
---
 dlls/comdlg32/printdlg.c |   23 -----------------------
 1 files changed, 0 insertions(+), 23 deletions(-)

diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c
index 149694a..5bf776f 100644
--- a/dlls/comdlg32/printdlg.c
+++ b/dlls/comdlg32/printdlg.c
@@ -1561,24 +1561,6 @@ static LRESULT PRINTDLG_WMCommandA(HWND hDlg, WPARAM wParam,
 	}
 	break;
 
-#if 0
-     case psh1:                       /* Print Setup */
-	{
-	    PRINTDLG16	pdlg;
-
-	    if (!PrintStructures->dlg.lpPrintDlg16) {
-		FIXME("The 32bit print dialog does not have this button!?\n");
-		break;
-	    }
-
-	    memcpy(&pdlg,PrintStructures->dlg.lpPrintDlg16,sizeof(pdlg));
-	    pdlg.Flags |= PD_PRINTSETUP;
-	    pdlg.hwndOwner = HWND_16(hDlg);
-	    if (!PrintDlg16(&pdlg))
-		break;
-	}
-	break;
-#endif
      case psh2:                       /* Properties button */
        {
          HANDLE hPrinter;
@@ -1733,11 +1715,6 @@ static LRESULT PRINTDLG_WMCommandW(HWND hDlg, WPARAM wParam,
         }
         break;
 
-     case psh1:                       /* Print Setup */
-	{
-		ERR("psh1 is called from 16bit code only, we should not get here.\n");
-	}
-	break;
      case psh2:                       /* Properties button */
        {
          HANDLE hPrinter;
-- 
1.6.5




More information about the wine-patches mailing list