🚀 go-pugleaf

Deepak Mishra wrote:

>The PRINTDLG structure size passed to PrintDlg is wrong.
>The correct one is 66 not 68 returned by sizeof(PRINTDLG) operator.
>
>Writing 66 instead of using sizeof might be a working solution, but I don't think that's a good idea. 
>
>Not able to find the reason.

At the top of COMMDLG.H is the following line
#pragma option push -b -a8 -pc -A- /*P_O_Push*/
which should set the packing to 8 bytes.
But, several lines later, is
#include <pshpack1.h>         /* Assume byte packing throughout */
which should result in byte packing.

Take a look at the many   #if defined    in pshpack1, and make sure
your application doesn't accidentally set incorrect #defines

The following program compiles and executes correctly.
PrintDlg.c:

#include <windows.h>
#include <commdlg.h>

#pragma	argsused
int _stdcall WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
				   LPSTR lpCmdLine, int nCmdShow)
{
	PRINTDLG pd;
	DOCINFO	lpdi;
	HDC	printerDC;

	pd.lStructSize = sizeof(PRINTDLG); 
	pd.hDevMode = (HANDLE) NULL; 
	pd.hDevNames = (HANDLE) NULL; 
	pd.Flags = PD_RETURNDC|PD_USEDEVMODECOPIESANDCOLLATE;
	pd.hwndOwner = 0; 
	pd.hDC = (HDC) NULL; 
	pd.nFromPage = 1; 
	pd.nToPage = 1; 
	pd.nMinPage = 0; 
	pd.nMaxPage = 0; 
	pd.nCopies = 1; 
	pd.hInstance = (HANDLE) NULL; 
	pd.lCustData = 0L; 
	pd.lpfnPrintHook = (LPPRINTHOOKPROC) NULL; 
	pd.lpfnSetupHook = (LPSETUPHOOKPROC) NULL; 
	pd.lpPrintTemplateName = (LPSTR) NULL; 
	pd.lpSetupTemplateName = (LPSTR)  NULL; 
	pd.hPrintTemplate = (HANDLE) NULL; 
	pd.hSetupTemplate = (HANDLE) NULL;

	if( IDOK == PrintDlg(&pd) )
	{	MessageBox( 0, "ID_OK", "Print Dialog",
MB_OK|MB_ICONINFORMATION|MB_TOPMOST );
	}else{
		MessageBox( 0, "Not ID_OK", "Print Dialog",
MB_OK|MB_ICONINFORMATION|MB_TOPMOST );
	};
	if( 0==(printerDC=pd.hDC) )
	{
		MessageBox( 0, "DC Not Ok", "Print Dialog",
MB_OK|MB_ICONINFORMATION|MB_TOPMOST );
	}else{
		MessageBox( 0, "DC Ok", "Print Dialog",
MB_OK|MB_ICONINFORMATION|MB_TOPMOST );
		DeleteDC(printerDC);
	};
	return 0;
}

Compiled with (might be more than needed, but it's what I use)
bcc32 -6 -a1 -C -d -g50 -K -O2 -Oi -Os -Ov -Q -VI- -tW  -vi -x-
-I$(INCLUDEPATH) -L$(LIBPATH) -P -M -v /c

Linked with
ilink32 /aa/v/x/c/C/Gn/P-/Twe/L$(LIBPATH) c0w32.obj+$(FILE).obj,
$(FILE), , import32.lib+cw32.lib