X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/34da0970a018b4b0828248028040ca339868ed4e..79b7b95aed76eb742deef93d046886c20452fb36:/src/msw/printwin.cpp

diff --git a/src/msw/printwin.cpp b/src/msw/printwin.cpp
index 31cf9641ce..310db919dd 100644
--- a/src/msw/printwin.cpp
+++ b/src/msw/printwin.cpp
@@ -1,341 +1,423 @@
 /////////////////////////////////////////////////////////////////////////////
-// Name:        printwin.cpp
+// Name:        src/msw/printwin.cpp
 // Purpose:     wxWindowsPrinter framework
 // Author:      Julian Smart
 // Modified by:
 // Created:     04/01/98
 // RCS-ID:      $Id$
-// Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:   	wxWindows license
+// Copyright:   (c) Julian Smart
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#ifdef __GNUG__
-#pragma implementation "printwin.h"
-#endif
+// ===========================================================================
+// declarations
+// ===========================================================================
+
+// ---------------------------------------------------------------------------
+// headers
+// ---------------------------------------------------------------------------
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
-#include "wx/defs.h"
-
-#define WINDOWS_PRINTING    (wxTheApp->GetPrintMode() == wxPRINT_WINDOWS)
+// Don't use the Windows printer if we're in wxUniv mode and using
+// the PostScript architecture
+#if wxUSE_PRINTING_ARCHITECTURE && (!defined(__WXUNIVERSAL__) || !wxUSE_POSTSCRIPT_ARCHITECTURE_IN_MSW)
 
 #ifndef WX_PRECOMP
-#include "wx/utils.h"
-#include "wx/dc.h"
-#include "wx/app.h"
-#include "wx/msgdlg.h"
+    #include "wx/msw/wrapcdlg.h"
+    #include "wx/window.h"
+    #include "wx/msw/private.h"
+    #include "wx/utils.h"
+    #include "wx/dc.h"
+    #include "wx/app.h"
+    #include "wx/msgdlg.h"
+    #include "wx/intl.h"
+    #include "wx/log.h"
+    #include "wx/dcprint.h"
 #endif
 
 #include "wx/msw/printwin.h"
-#include "wx/dcprint.h"
-#include "wx/printdlg.h"
+#include "wx/msw/printdlg.h"
 #include "wx/msw/private.h"
 
 #include <stdlib.h>
-#include <windows.h>
-#include <commdlg.h>
-
-// Clash with Windows header files
-#ifdef StartDoc
-#undef StartDoc
-#endif
 
 #ifndef __WIN32__
-#include <print.h>
+    #include <print.h>
 #endif
 
+// ---------------------------------------------------------------------------
+// private functions
+// ---------------------------------------------------------------------------
+
 LONG APIENTRY _EXPORT wxAbortProc(HDC hPr, int Code);
 
-#if !USE_SHARED_LIBRARY
-IMPLEMENT_DYNAMIC_CLASS(wxWindowsPrinter, wxPrinterBase)
-IMPLEMENT_CLASS(wxWindowsPrintPreview, wxPrintPreviewBase)
-#endif
+// ---------------------------------------------------------------------------
+// wxWin macros
+// ---------------------------------------------------------------------------
 
-/*
- * Printer
- */
- 
-wxWindowsPrinter::wxWindowsPrinter(wxPrintData *data):
-  wxPrinterBase(data)
+    IMPLEMENT_DYNAMIC_CLASS(wxWindowsPrinter, wxPrinterBase)
+    IMPLEMENT_CLASS(wxWindowsPrintPreview, wxPrintPreviewBase)
+
+// ===========================================================================
+// implementation
+// ===========================================================================
+
+// ---------------------------------------------------------------------------
+// Printer
+// ---------------------------------------------------------------------------
+
+wxWindowsPrinter::wxWindowsPrinter(wxPrintDialogData *data)
+                : wxPrinterBase(data)
 {
-    m_lpAbortProc = (WXFARPROC) MakeProcInstance((FARPROC) wxAbortProc, wxGetInstance());
+    m_lpAbortProc = (WXFARPROC)wxAbortProc;
 }
 
-wxWindowsPrinter::~wxWindowsPrinter(void)
+wxWindowsPrinter::~wxWindowsPrinter()
 {
+    // avoids warning about statement with no effect (FreeProcInstance
+    // doesn't do anything under Win32)
+#if !defined(__WIN32__) && !defined(__NT__)
     FreeProcInstance((FARPROC) m_lpAbortProc);
+#endif
 }
 
 bool wxWindowsPrinter::Print(wxWindow *parent, wxPrintout *printout, bool prompt)
 {
-  sm_abortIt = FALSE;
-  sm_abortWindow = NULL;
-
-  if (!printout)
-    return FALSE;
-    
-  printout->SetIsPreview(FALSE);
-  printout->OnPreparePrinting();
-
-  // Get some parameters from the printout, if defined
-  int fromPage, toPage;
-  int minPage, maxPage;
-  printout->GetPageInfo(&minPage, &maxPage, &fromPage, &toPage);
-
-  if (maxPage == 0)
-    return FALSE;
-
-  m_printData.SetMinPage(minPage);
-  m_printData.SetMaxPage(maxPage);
-  if (fromPage != 0)
-    m_printData.SetFromPage(fromPage);
-  if (toPage != 0)
-    m_printData.SetToPage(toPage);
-
-  if (minPage != 0)
-  {
-    m_printData.EnablePageNumbers(TRUE);
-    if (m_printData.GetFromPage() < m_printData.GetMinPage())
-      m_printData.SetFromPage(m_printData.GetMinPage());
-    else if (m_printData.GetFromPage() > m_printData.GetMaxPage())
-      m_printData.SetFromPage(m_printData.GetMaxPage());
-    if (m_printData.GetToPage() > m_printData.GetMaxPage())
-      m_printData.SetToPage(m_printData.GetMaxPage());
-    else if (m_printData.GetToPage() < m_printData.GetMinPage())
-      m_printData.SetToPage(m_printData.GetMinPage());
-  }
-  else
-    m_printData.EnablePageNumbers(FALSE);
-
-  // Create a suitable device context  
-  wxDC *dc = NULL;
-  if (prompt)
-  {
-        wxPrintDialog dialog(parent, & m_printData);
-        if (dialog.ShowModal() == wxID_OK)
-        {
-          dc = dialog.GetPrintDC();
-          m_printData = dialog.GetPrintData();
-        }
-  }
-  else
-  {
-      dc = new wxPrinterDC("", "", "", FALSE, m_printData.GetOrientation());
-  }
-
-  // May have pressed cancel.
-  if (!dc || !dc->Ok())
-  {
-    if (dc) delete dc;
-    return FALSE;
-  }
-  
-  int logPPIScreenX = 0;
-  int logPPIScreenY = 0;
-  int logPPIPrinterX = 0;
-  int logPPIPrinterY = 0;
-
-  HDC hdc = ::GetDC(NULL);
-  logPPIScreenX = ::GetDeviceCaps(hdc, LOGPIXELSX);
-  logPPIScreenY = ::GetDeviceCaps(hdc, LOGPIXELSY);
-  ::ReleaseDC(NULL, hdc);
-
-  logPPIPrinterX = ::GetDeviceCaps((HDC) dc->GetHDC(), LOGPIXELSX);
-  logPPIPrinterY = ::GetDeviceCaps((HDC) dc->GetHDC(), LOGPIXELSY);
-  if (logPPIPrinterX == 0 || logPPIPrinterY == 0)
-  {
-    delete dc;
-    return FALSE;
-  }
+    sm_abortIt = false;
+    sm_abortWindow = NULL;
 
-  printout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
-  printout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
+    if (!printout)
+    {
+        sm_lastError = wxPRINTER_ERROR;
+        return false;
+    }
 
-  // Set printout parameters  
-  printout->SetDC(dc);
+    printout->SetIsPreview(false);
 
-  int w, h;
-  long ww, hh;
-  dc->GetSize(&w, &h);
-  printout->SetPageSizePixels((int)w, (int)h);
-  dc->GetSizeMM(&ww, &hh);
-  printout->SetPageSizeMM((int)ww, (int)hh);
+    if (m_printDialogData.GetMinPage() < 1)
+        m_printDialogData.SetMinPage(1);
+    if (m_printDialogData.GetMaxPage() < 1)
+        m_printDialogData.SetMaxPage(9999);
 
-  // Create an abort window
-  wxBeginBusyCursor();
+    // Create a suitable device context
+    wxPrinterDC *dc wxDUMMY_INITIALIZE(NULL);
+    if (prompt)
+    {
+        dc = wxDynamicCast(PrintDialog(parent), wxPrinterDC);
+        if (!dc)
+            return false;
+    }
+    else
+    {
+        dc = new wxPrinterDC(m_printDialogData.GetPrintData());
+    }
 
-  wxWindow *win = CreateAbortWindow(parent, printout);
-  wxYield();
+    // May have pressed cancel.
+    if (!dc || !dc->Ok())
+    {
+        if (dc) delete dc;
+        return false;
+    }
+
+    HDC hdc = ::GetDC(NULL);
+    int logPPIScreenX = ::GetDeviceCaps(hdc, LOGPIXELSX);
+    int logPPIScreenY = ::GetDeviceCaps(hdc, LOGPIXELSY);
+    ::ReleaseDC(NULL, hdc);
+
+    int logPPIPrinterX = ::GetDeviceCaps((HDC) dc->GetHDC(), LOGPIXELSX);
+    int logPPIPrinterY = ::GetDeviceCaps((HDC) dc->GetHDC(), LOGPIXELSY);
+    if (logPPIPrinterX == 0 || logPPIPrinterY == 0)
+    {
+        delete dc;
+        sm_lastError = wxPRINTER_ERROR;
+        return false;
+    }
+
+    printout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
+    printout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
+
+    // Set printout parameters
+    printout->SetDC(dc);
+
+    int w, h;
+    dc->GetSize(&w, &h);
+    printout->SetPageSizePixels((int)w, (int)h);
+    printout->SetPaperRectPixels(dc->GetPaperRect());
+
+    dc->GetSizeMM(&w, &h);
+    printout->SetPageSizeMM((int)w, (int)h);
+
+    // Create an abort window
+    wxBusyCursor busyCursor;
 
-#if defined(__BORLANDC__) || defined(__GNUWIN32__) || !defined(__WIN32__)
-  ::SetAbortProc((HDC) dc->GetHDC(), (FARPROC) m_lpAbortProc);
+    printout->OnPreparePrinting();
+
+    // Get some parameters from the printout, if defined
+    int fromPage, toPage;
+    int minPage, maxPage;
+    printout->GetPageInfo(&minPage, &maxPage, &fromPage, &toPage);
+
+    if (maxPage == 0)
+    {
+        sm_lastError = wxPRINTER_ERROR;
+        return false;
+    }
+
+    // Only set min and max, because from and to have been
+    // set by the user
+    m_printDialogData.SetMinPage(minPage);
+    m_printDialogData.SetMaxPage(maxPage);
+
+    wxWindow *win = CreateAbortWindow(parent, printout);
+    wxYield();
+
+#if defined(__WATCOMC__) || defined(__BORLANDC__) || defined(__GNUWIN32__) || defined(__SALFORDC__) || !defined(__WIN32__)
+#ifdef STRICT
+    ::SetAbortProc((HDC) dc->GetHDC(), (ABORTPROC) m_lpAbortProc);
+#else
+    ::SetAbortProc((HDC) dc->GetHDC(), (FARPROC) m_lpAbortProc);
+#endif
 #else
-  ::SetAbortProc((HDC) dc->GetHDC(), (int (_stdcall *)
-    // cast it to right type only if required
-    // @@@ it's really cdecl and we're casting it to stdcall - either there is
-    //     something I don't understand or it will crash at first usage
-  #ifdef STRICT
-    (HDC, int)
-  #else
-    ()
-  #endif
-    )m_lpAbortProc);
+    ::SetAbortProc((HDC) dc->GetHDC(), (int (_stdcall *)
+        // cast it to right type only if required
+        // FIXME it's really cdecl and we're casting it to stdcall - either there is
+        //       something I don't understand or it will crash at first usage
+#ifdef STRICT
+        (HDC, int)
+#else
+        ()
+#endif
+        )m_lpAbortProc);
 #endif
 
-  if (!win)
-  {
-    wxEndBusyCursor();
-    wxMessageBox("Sorry, could not create an abort dialog.", "Print Error", wxOK, parent);
-    delete dc;
-  }
-  sm_abortWindow = win;
-  sm_abortWindow->Show(TRUE);
-  wxYield();
-
-  printout->OnBeginPrinting();
-  
-  bool keepGoing = TRUE;
-
-  int copyCount;
-  for (copyCount = 1; copyCount <= m_printData.GetNoCopies(); copyCount ++)
-  {
-    if (!printout->OnBeginDocument(m_printData.GetFromPage(), m_printData.GetToPage()))
+    if (!win)
     {
-      wxEndBusyCursor();
-      wxMessageBox("Could not start printing.", "Print Error", wxOK, parent);
-      break;
+        wxLogDebug(wxT("Could not create an abort dialog."));
+        sm_lastError = wxPRINTER_ERROR;
+
+        delete dc;
+        return false;
     }
-    if (sm_abortIt)
-      break;
+    sm_abortWindow = win;
+    sm_abortWindow->Show();
+    wxSafeYield();
+
+    printout->OnBeginPrinting();
 
-    int pn;
-    for (pn = m_printData.GetFromPage(); keepGoing && (pn <= m_printData.GetToPage()) && printout->HasPage(pn);
-         pn++)
+    sm_lastError = wxPRINTER_NO_ERROR;
+
+    int minPageNum = minPage, maxPageNum = maxPage;
+
+    if ( !m_printDialogData.GetAllPages() )
     {
-      if (sm_abortIt)
-      {
-        keepGoing = FALSE;
-        break;
-      }
-      else
-      {
-//        int dcID = ::SaveDC(dc->GetHDC());
-        dc->StartPage();
-        printout->OnPrintPage(pn);
-        dc->EndPage();
-//        ::RestoreDC(dc->GetHDC(), dcID);
-      }
+        minPageNum = m_printDialogData.GetFromPage();
+        maxPageNum = m_printDialogData.GetToPage();
     }
-    printout->OnEndDocument();
-  }
 
-  printout->OnEndPrinting();
+    int copyCount;
+    for ( copyCount = 1;
+          copyCount <= m_printDialogData.GetNoCopies();
+          copyCount++ )
+    {
+        if ( !printout->OnBeginDocument(minPageNum, maxPageNum) )
+        {
+            wxLogError(_("Could not start printing."));
+            sm_lastError = wxPRINTER_ERROR;
+            break;
+        }
+        if (sm_abortIt)
+        {
+            sm_lastError = wxPRINTER_CANCELLED;
+            break;
+        }
 
-  if (sm_abortWindow)
-  {
-    sm_abortWindow->Show(FALSE);
-    delete sm_abortWindow;
-    sm_abortWindow = NULL;
-  }
+        int pn;
 
-  wxEndBusyCursor();
+        for ( pn = minPageNum;
+              pn <= maxPageNum && printout->HasPage(pn);
+              pn++ )
+        {
+            if ( sm_abortIt )
+            {
+                sm_lastError = wxPRINTER_CANCELLED;
+                break;
+            }
+
+            dc->StartPage();
+            bool cont = printout->OnPrintPage(pn);
+            dc->EndPage();
+
+            if ( !cont )
+            {
+                sm_lastError = wxPRINTER_CANCELLED;
+                break;
+            }
+        }
+
+        printout->OnEndDocument();
+    }
 
-  delete dc;
-  
-  return TRUE;
+    printout->OnEndPrinting();
+
+    if (sm_abortWindow)
+    {
+        sm_abortWindow->Show(false);
+        delete sm_abortWindow;
+        sm_abortWindow = NULL;
+    }
+
+    delete dc;
+
+    return sm_lastError == wxPRINTER_NO_ERROR;
 }
 
-bool wxWindowsPrinter::PrintDialog(wxWindow *parent)
+wxDC *wxWindowsPrinter::PrintDialog(wxWindow *parent)
 {
-  wxPrintDialog dialog(parent, & m_printData);
-  return (dialog.ShowModal() == wxID_OK);
+    wxDC *dc = (wxPrinterDC*) NULL;
+
+    wxWindowsPrintDialog dialog(parent, & m_printDialogData);
+    int ret = dialog.ShowModal();
+
+    if (ret == wxID_OK)
+    {
+        dc = dialog.GetPrintDC();
+        m_printDialogData = dialog.GetPrintDialogData();
+        if (dc == NULL)
+            sm_lastError = wxPRINTER_ERROR;
+        else
+            sm_lastError = wxPRINTER_NO_ERROR;
+    }
+    else
+        sm_lastError = wxPRINTER_CANCELLED;
+
+    return dc;
 }
 
-bool wxWindowsPrinter::Setup(wxWindow *parent)
+bool wxWindowsPrinter::Setup(wxWindow *WXUNUSED(parent))
 {
-  wxPrintDialog dialog(parent, & m_printData);
-  dialog.GetPrintData().SetSetupDialog(TRUE);
-  return (dialog.ShowModal() == wxID_OK);
+#if 0
+    // We no longer expose that dialog
+    wxPrintDialog dialog(parent, & m_printDialogData);
+    dialog.GetPrintDialogData().SetSetupDialog(true);
+
+    int ret = dialog.ShowModal();
+
+    if (ret == wxID_OK)
+    {
+        m_printDialogData = dialog.GetPrintDialogData();
+    }
+
+    return (ret == wxID_OK);
+#else
+    return false;
+#endif
 }
 
 /*
- * Print preview
- */
+* Print preview
+*/
 
-wxWindowsPrintPreview::wxWindowsPrintPreview(wxPrintout *printout, wxPrintout *printoutForPrinting, wxPrintData *data):
-  wxPrintPreviewBase(printout, printoutForPrinting, data)
+wxWindowsPrintPreview::wxWindowsPrintPreview(wxPrintout *printout,
+                                             wxPrintout *printoutForPrinting,
+                                             wxPrintDialogData *data)
+                     : wxPrintPreviewBase(printout, printoutForPrinting, data)
 {
-  DetermineScaling();
+    DetermineScaling();
 }
 
-wxWindowsPrintPreview::~wxWindowsPrintPreview(void)
+wxWindowsPrintPreview::wxWindowsPrintPreview(wxPrintout *printout,
+                                             wxPrintout *printoutForPrinting,
+                                             wxPrintData *data)
+                     : wxPrintPreviewBase(printout, printoutForPrinting, data)
+{
+    DetermineScaling();
+}
+
+wxWindowsPrintPreview::~wxWindowsPrintPreview()
 {
 }
 
 bool wxWindowsPrintPreview::Print(bool interactive)
 {
-  if (!m_printPrintout)
-    return FALSE;
-  wxWindowsPrinter printer(&m_printData);
-  return printer.Print(m_previewFrame, m_printPrintout, interactive);
+    if (!m_printPrintout)
+        return false;
+    wxWindowsPrinter printer(&m_printDialogData);
+    return printer.Print(m_previewFrame, m_printPrintout, interactive);
 }
 
-void wxWindowsPrintPreview::DetermineScaling(void)
+void wxWindowsPrintPreview::DetermineScaling()
 {
-    HDC dc = ::GetDC(NULL);
-    int screenWidth = ::GetDeviceCaps(dc, HORZSIZE);
-//    int screenHeight = ::GetDeviceCaps(dc, VERTSIZE);
-    int screenXRes = ::GetDeviceCaps(dc, HORZRES);
-//    int screenYRes = ::GetDeviceCaps(dc, VERTRES);
+    ScreenHDC dc;
     int logPPIScreenX = ::GetDeviceCaps(dc, LOGPIXELSX);
     int logPPIScreenY = ::GetDeviceCaps(dc, LOGPIXELSY);
     m_previewPrintout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
 
-    ::ReleaseDC(NULL, dc);
-
     // Get a device context for the currently selected printer
-    wxPrinterDC printerDC("", "", "", FALSE, m_printData.GetOrientation());
+    wxPrinterDC printerDC(m_printDialogData.GetPrintData());
+
+    int printerWidthMM;
+    int printerHeightMM;
+    int printerXRes;
+    int printerYRes;
+    int logPPIPrinterX;
+    int logPPIPrinterY;
 
-    int printerWidth = 150;
-    int printerHeight = 250;
-    int printerXRes = 1500;
-    int printerYRes = 2500;
+    wxRect paperRect;
 
-    if (printerDC.GetHDC())
+    if ( printerDC.Ok() )
     {
-      printerWidth = ::GetDeviceCaps((HDC) printerDC.GetHDC(), HORZSIZE);
-      printerHeight = ::GetDeviceCaps((HDC) printerDC.GetHDC(), VERTSIZE);
-      printerXRes = ::GetDeviceCaps((HDC) printerDC.GetHDC(), HORZRES);
-      printerYRes = ::GetDeviceCaps((HDC) printerDC.GetHDC(), VERTRES);
-
-      int logPPIPrinterX = ::GetDeviceCaps((HDC) printerDC.GetHDC(), LOGPIXELSX);
-      int logPPIPrinterY = ::GetDeviceCaps((HDC) printerDC.GetHDC(), LOGPIXELSY);
-
-      m_previewPrintout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
-      m_previewPrintout->SetPageSizeMM(printerWidth, printerHeight);
-      
-      if (logPPIPrinterX == 0 || logPPIPrinterY == 0 || printerWidth == 0 || printerHeight == 0)
-        m_isOk = FALSE;
+        HDC dc = GetHdcOf(printerDC);
+        printerWidthMM = ::GetDeviceCaps(dc, HORZSIZE);
+        printerHeightMM = ::GetDeviceCaps(dc, VERTSIZE);
+        printerXRes = ::GetDeviceCaps(dc, HORZRES);
+        printerYRes = ::GetDeviceCaps(dc, VERTRES);
+        logPPIPrinterX = ::GetDeviceCaps(dc, LOGPIXELSX);
+        logPPIPrinterY = ::GetDeviceCaps(dc, LOGPIXELSY);
+
+        paperRect = printerDC.GetPaperRect();
+
+        if ( logPPIPrinterX == 0 ||
+                logPPIPrinterY == 0 ||
+                    printerWidthMM == 0 ||
+                        printerHeightMM == 0 )
+        {
+            m_isOk = false;
+        }
     }
     else
-      m_isOk = FALSE;
-
+    {
+        // use some defaults
+        printerWidthMM = 150;
+        printerHeightMM = 250;
+        printerXRes = 1500;
+        printerYRes = 2500;
+        logPPIPrinterX = 600;
+        logPPIPrinterY = 600;
+
+        paperRect = wxRect(0, 0, printerXRes, printerYRes);
+        m_isOk = false;
+    }
     m_pageWidth = printerXRes;
     m_pageHeight = printerYRes;
+    m_previewPrintout->SetPageSizePixels(printerXRes, printerYRes);
+    m_previewPrintout->SetPageSizeMM(printerWidthMM, printerHeightMM);
+    m_previewPrintout->SetPaperRectPixels(paperRect);
+    m_previewPrintout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
 
     // At 100%, the page should look about page-size on the screen.
-    m_previewScale = (float)((float)screenWidth/(float)printerWidth);
-    m_previewScale = m_previewScale * (float)((float)screenXRes/(float)printerYRes);
+    m_previewScaleX = float(logPPIScreenX) / logPPIPrinterX;
+    m_previewScaleY = float(logPPIScreenY) / logPPIPrinterY;
 }
 
 /****************************************************************************
 
-    FUNCTION: wxAbortProc()
+  FUNCTION: wxAbortProc()
 
     PURPOSE:  Processes messages for the Abort Dialog box
 
@@ -350,7 +432,7 @@ LONG APIENTRY _EXPORT wxAbortProc(HDC WXUNUSED(hPr), int WXUNUSED(Code))
 
     /* Process messages intended for the abort dialog box */
 
-    while (!wxPrinterBase::sm_abortIt && PeekMessage(&msg, 0, 0, 0, TRUE))
+    while (!wxPrinterBase::sm_abortIt && ::PeekMessage(&msg, 0, 0, 0, TRUE))
         if (!IsDialogMessage((HWND) wxPrinterBase::sm_abortWindow->GetHWND(), &msg)) {
             TranslateMessage(&msg);
             DispatchMessage(&msg);
@@ -358,6 +440,8 @@ LONG APIENTRY _EXPORT wxAbortProc(HDC WXUNUSED(hPr), int WXUNUSED(Code))
 
     /* bAbort is TRUE (return is FALSE) if the user has aborted */
 
-    return (!wxPrinterBase::sm_abortIt);
+    return !wxPrinterBase::sm_abortIt;
 }
 
+#endif
+    // wxUSE_PRINTING_ARCHITECTURE