🚀 go-pugleaf

Keith wrote:

>typedef struct
>{
>     WORD wSize;
>     BYTE bData[2];                  // placeholder
>} PASSTHROUGHSTRUCT, FAR *LPPTS;
>
>LPPTS m_lpPTS;

Don't rely on "magic".
You rely on m_lpPTS being NULL, so set it.
	LPPTS  m_lpPTS = NULL;

>if (!m_lpPTS)
>  {
>  m_lpPTS = (LPPTS)GlobalLock(GlobalAlloc(GPTR, sizeof(WORD) +
>                                     2048));

Incorrect.
GPTR includes GMEM_FIXED, which is already locked, AND it returns the
Address, not a Handle. Also not sure why you want is Zero Initialized.

	m_lpPTS = (LPPTS)GlobalAlloc(GMEM_FIXED, 2050 );

>char *PrintThis="Now is the time for all good men to come to the aid of
>his country and Liz need not be offended.\r\n"
>m_lpPTS->wSize = strlen(PrintThis);
>
>memcpy (m_lpPTS->bData, PrintThis, strlen(PrintThis));
>
>int rv=1;
>for (int i=0;rv && i<30;++i)
>  {
>  rv=Escape(m_dcPrint,PASSTHROUGH,sizeof(m_lpPTS->wSize) +
>m_lpPTS->wSize,
>         (LPSTR)m_lpPTS,0);
>  }

You don't check for failure of the Escape.
You only check if it is implimented (zero return).
Are you sure the printer wants to see the size field?
I would think this would be more appropriate...

static  char  PrintThis[] = "Now is the time for all good men to come
to the aid of his country and Liz need not be offended.\r\n";

  rv=Escape( m_dcPrint ,PASSTHROUGH, sizeof(PrintThis)-1, PrintThis);
  if( rv <= 0 )
  {
	/* handle error */
	break;
   };

>EndPage(m_dcPrint);
>EndDoc(m_dcPrint);
>GlobalUnlock(m_lpPTS);

You forgot GlobalFree()
Also, since your Lock was incorrect, the Unlock is too...

	EndPage(m_dcPrint);
	EndDoc(m_dcPrint);
	GlobalFree(m_lpPTS);
	m_lpPTS = NULL;

I also didn't see where you actually Opened the Printer, or Closed it.