Fixed wxPalette memory leak when loading BMP files

[wxWidgets.git] / src / msw / printwin.cpp

diff --git a/src/msw/printwin.cpp b/src/msw/printwin.cpp
index f60b5747257b436dd4e5f0df09f4de06e44529dd..f65d3e74317b21e637434964fc082468b9e804fb 100644 (file)
--- a/src/msw/printwin.cpp
+++ b/src/msw/printwin.cpp
@@ -22,8 +22,6 @@
 
 #include "wx/defs.h"
 
 
 #include "wx/defs.h"
 

-#define WINDOWS_PRINTING    (wxTheApp->GetPrintMode() == wxPRINT_WINDOWS)
-

 #ifndef WX_PRECOMP
 #include "wx/utils.h"
 #include "wx/dc.h"
 #ifndef WX_PRECOMP
 #include "wx/utils.h"
 #include "wx/dc.h"


@@ -57,211 +55,234 @@ IMPLEMENT_CLASS(wxWindowsPrintPreview, wxPrintPreviewBase)
 #endif
 
 /*
 #endif
 
 /*

- * Printer
- */
- 
-wxWindowsPrinter::wxWindowsPrinter(wxPrintData *data):
-  wxPrinterBase(data)
+* Printer
+*/
+
+wxWindowsPrinter::wxWindowsPrinter(wxPrintDialogData *data):
+wxPrinterBase(data)

 {
 {

-  lpAbortProc = (WXFARPROC) MakeProcInstance((FARPROC) wxAbortProc, wxGetInstance());
+    m_lpAbortProc = (WXFARPROC) MakeProcInstance((FARPROC) wxAbortProc, wxGetInstance());

 }
 
 wxWindowsPrinter::~wxWindowsPrinter(void)
 {
 }
 
 wxWindowsPrinter::~wxWindowsPrinter(void)
 {

-  FreeProcInstance((FARPROC) lpAbortProc);
+    FreeProcInstance((FARPROC) m_lpAbortProc);

 }
 
 bool wxWindowsPrinter::Print(wxWindow *parent, wxPrintout *printout, bool prompt)
 {
 }
 
 bool wxWindowsPrinter::Print(wxWindow *parent, wxPrintout *printout, bool prompt)
 {

-  abortIt = FALSE;
-  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;
-
-  printData.SetMinPage(minPage);
-  printData.SetMaxPage(maxPage);
-  if (fromPage != 0)
-    printData.SetFromPage(fromPage);
-  if (toPage != 0)
-    printData.SetToPage(toPage);
-
-  if (minPage != 0)
-  {
-    printData.EnablePageNumbers(TRUE);
-    if (printData.GetFromPage() < printData.GetMinPage())
-      printData.SetFromPage(printData.GetMinPage());
-    else if (printData.GetFromPage() > printData.GetMaxPage())
-      printData.SetFromPage(printData.GetMaxPage());
-    if (printData.GetToPage() > printData.GetMaxPage())
-      printData.SetToPage(printData.GetMaxPage());
-    else if (printData.GetToPage() < printData.GetMinPage())
-      printData.SetToPage(printData.GetMinPage());
-  }
-  else
-    printData.EnablePageNumbers(FALSE);
-  
-  // Create a suitable device context  
-  wxDC *dc = NULL;
-  if (prompt)
-  {
-        wxPrintDialog dialog(parent, & printData);
-        if (dialog.ShowModal() == wxID_OK)
-        {
-          dc = dialog.GetPrintDC();
-          printData = dialog.GetPrintData();
-        }
-  }
-  else
-  {
-      dc = new wxPrinterDC("", "", "", FALSE, 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;
-  }
-
-  printout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
-  printout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
-
-  // Set printout parameters  
-  printout->SetDC(dc);
-
-  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);
-
-  // Create an abort window
-  wxBeginBusyCursor();
+    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_printDialogData.SetMinPage(minPage);
+    m_printDialogData.SetMaxPage(maxPage);
+    if (fromPage != 0)
+        m_printDialogData.SetFromPage(fromPage);
+    if (toPage != 0)
+        m_printDialogData.SetToPage(toPage);
+    
+    if (minPage != 0)
+    {
+        m_printDialogData.EnablePageNumbers(TRUE);
+        if (m_printDialogData.GetFromPage() < m_printDialogData.GetMinPage())
+            m_printDialogData.SetFromPage(m_printDialogData.GetMinPage());
+        else if (m_printDialogData.GetFromPage() > m_printDialogData.GetMaxPage())
+            m_printDialogData.SetFromPage(m_printDialogData.GetMaxPage());
+        if (m_printDialogData.GetToPage() > m_printDialogData.GetMaxPage())
+            m_printDialogData.SetToPage(m_printDialogData.GetMaxPage());
+        else if (m_printDialogData.GetToPage() < m_printDialogData.GetMinPage())
+            m_printDialogData.SetToPage(m_printDialogData.GetMinPage());
+    }
+    else
+        m_printDialogData.EnablePageNumbers(FALSE);

 
 

-  wxWindow *win = CreateAbortWindow(parent, printout);
-  wxYield();
+    // Create a suitable device context  
+    wxDC *dc = NULL;
+    if (prompt)
+    {
+        dc = PrintDialog(parent);
+        if (!dc)
+            return FALSE;
+    }
+    else
+    {
+        //      dc = new wxPrinterDC("", "", "", FALSE, m_printData.GetOrientation());
+        dc = new wxPrinterDC(m_printDialogData.GetPrintData());
+    }
+    
+    // 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;
+    }
+    
+    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);

 
 

-#if defined(__BORLANDC__) || defined(__GNUWIN32__)
-  ::SetAbortProc((HDC) dc->GetHDC(), (FARPROC) lpAbortProc);
+    dc->GetSizeMM(&w, &h);
+    printout->SetPageSizeMM((int)w, (int)h);
+    
+    // Create an abort window
+    wxBeginBusyCursor();
+    
+    wxWindow *win = CreateAbortWindow(parent, printout);
+    wxYield();
+    
+#if defined(__BORLANDC__) || defined(__GNUWIN32__) || defined(__SALFORDC__) || !defined(__WIN32__)
+    ::SetAbortProc((HDC) dc->GetHDC(), (FARPROC) m_lpAbortProc);

 #else
 #else

-  ::SetAbortProc((HDC) dc->GetHDC(), (int (_stdcall *)(HDC, int)) lpAbortProc);
+    ::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
 #endif

-
-  if (!win)
-  {
-    wxEndBusyCursor();
-    wxMessageBox("Sorry, could not create an abort dialog.", "Print Error", wxOK, parent);
-    delete dc;
-  }
-  abortWindow = win;
-  abortWindow->Show(TRUE);
-  wxYield();
-
-  printout->OnBeginPrinting();
-  
-  bool keepGoing = TRUE;
-
-  int copyCount;
-  for (copyCount = 1; copyCount <= printData.GetNoCopies(); copyCount ++)
-  {
-    if (!printout->OnBeginDocument(printData.GetFromPage(), printData.GetToPage()))
+        )m_lpAbortProc);
+#endif
+    
+    if (!win)

     {
     {

-      wxEndBusyCursor();
-      wxMessageBox("Could not start printing.", "Print Error", wxOK, parent);
-      break;
+        wxEndBusyCursor();
+        wxMessageBox("Sorry, could not create an abort dialog.", "Print Error", wxOK, parent);
+        delete dc;

     }
     }

-    if (abortIt)
-      break;
-
-    int pn;
-    for (pn = printData.GetFromPage(); keepGoing && (pn <= printData.GetToPage()) && printout->HasPage(pn);
-         pn++)
+    sm_abortWindow = win;
+    sm_abortWindow->Show(TRUE);
+    wxYield();
+    
+    printout->OnBeginPrinting();
+    
+    bool keepGoing = TRUE;
+    
+    int copyCount;
+    for (copyCount = 1; copyCount <= m_printDialogData.GetNoCopies(); copyCount ++)

     {
     {

-      if (abortIt)
-      {
-        keepGoing = FALSE;
-        break;
-      }
-      else
-      {
-//        int dcID = ::SaveDC(dc->GetHDC());
-        dc->StartPage();
-        printout->OnPrintPage(pn);
-        dc->EndPage();
-//        ::RestoreDC(dc->GetHDC(), dcID);
-      }
+        if (!printout->OnBeginDocument(m_printDialogData.GetFromPage(), m_printDialogData.GetToPage()))
+        {
+            wxEndBusyCursor();
+            wxMessageBox("Could not start printing.", "Print Error", wxOK, parent);
+            break;
+        }
+        if (sm_abortIt)
+            break;
+        
+        int pn;
+        for (pn = m_printDialogData.GetFromPage(); keepGoing && (pn <= m_printDialogData.GetToPage()) && printout->HasPage(pn);
+        pn++)
+        {
+            if (sm_abortIt)
+            {
+                keepGoing = FALSE;
+                break;
+            }
+            else
+            {
+                dc->StartPage();
+                printout->OnPrintPage(pn);
+                dc->EndPage();
+            }
+        }
+        printout->OnEndDocument();

     }
     }

-    printout->OnEndDocument();
-  }
-
-  printout->OnEndPrinting();
+    
+    printout->OnEndPrinting();
+    
+    if (sm_abortWindow)
+    {
+        sm_abortWindow->Show(FALSE);
+        delete sm_abortWindow;
+        sm_abortWindow = NULL;
+    }
+    
+    wxEndBusyCursor();
+    
+    delete dc;
+    
+    return TRUE;
+}

 
 

-  if (abortWindow)
-  {
-    abortWindow->Show(FALSE);
-    delete abortWindow;
-    abortWindow = NULL;
-  }
+wxDC* wxWindowsPrinter::PrintDialog(wxWindow *parent)
+{
+    wxDC* dc = (wxDC*) NULL;

 
 

-  wxEndBusyCursor();
+    wxPrintDialog dialog(parent, & m_printDialogData);
+    int ret = dialog.ShowModal();

 
 

-  delete dc;
-  
-  return TRUE;
-}
+    if (ret == wxID_OK)
+    {
+        dc = dialog.GetPrintDC();
+        m_printDialogData = dialog.GetPrintDialogData();
+    }

 
 

-bool wxWindowsPrinter::PrintDialog(wxWindow *parent)
-{
-  wxPrintDialog dialog(parent, & printData);
-  return (dialog.ShowModal() == wxID_OK);
+    return dc;

 }
 
 bool wxWindowsPrinter::Setup(wxWindow *parent)
 {
 }
 
 bool wxWindowsPrinter::Setup(wxWindow *parent)
 {

-  wxPrintDialog dialog(parent, & printData);
-  dialog.GetPrintData().SetSetupDialog(TRUE);
-  return (dialog.ShowModal() == wxID_OK);
+    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);

 }
 
 /*
 }
 
 /*

- * 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(void)


@@ -270,85 +291,85 @@ wxWindowsPrintPreview::~wxWindowsPrintPreview(void)
 
 bool wxWindowsPrintPreview::Print(bool interactive)
 {
 
 bool wxWindowsPrintPreview::Print(bool interactive)
 {

-  if (!printPrintout)
-    return FALSE;
-  wxWindowsPrinter printer(&printData);
-  return printer.Print(previewFrame, printPrintout, interactive);
+    if (!m_printPrintout)
+        return FALSE;
+    wxWindowsPrinter printer(&m_printDialogData);
+    return printer.Print(m_previewFrame, m_printPrintout, interactive);

 }
 
 void wxWindowsPrintPreview::DetermineScaling(void)
 {
     HDC dc = ::GetDC(NULL);
     int screenWidth = ::GetDeviceCaps(dc, HORZSIZE);
 }
 
 void wxWindowsPrintPreview::DetermineScaling(void)
 {
     HDC dc = ::GetDC(NULL);
     int screenWidth = ::GetDeviceCaps(dc, HORZSIZE);

-//    int screenHeight = ::GetDeviceCaps(dc, VERTSIZE);
+    //    int screenHeight = ::GetDeviceCaps(dc, VERTSIZE);

     int screenXRes = ::GetDeviceCaps(dc, HORZRES);
     int screenXRes = ::GetDeviceCaps(dc, HORZRES);

-//    int screenYRes = ::GetDeviceCaps(dc, VERTRES);
+    //    int screenYRes = ::GetDeviceCaps(dc, VERTRES);

     int logPPIScreenX = ::GetDeviceCaps(dc, LOGPIXELSX);
     int logPPIScreenY = ::GetDeviceCaps(dc, LOGPIXELSY);
     int logPPIScreenX = ::GetDeviceCaps(dc, LOGPIXELSX);
     int logPPIScreenY = ::GetDeviceCaps(dc, LOGPIXELSY);

-    previewPrintout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
-
+    m_previewPrintout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
+    

     ::ReleaseDC(NULL, dc);
     ::ReleaseDC(NULL, dc);

-
+    

     // Get a device context for the currently selected printer
     // Get a device context for the currently selected printer

-    wxPrinterDC printerDC("", "", "", FALSE, printData.GetOrientation());
-
+    wxPrinterDC printerDC(m_printDialogData.GetPrintData());
+    

     int printerWidth = 150;
     int printerHeight = 250;
     int printerXRes = 1500;
     int printerYRes = 2500;
     int printerWidth = 150;
     int printerHeight = 250;
     int printerXRes = 1500;
     int printerYRes = 2500;

-
+    

     if (printerDC.GetHDC())
     {
     if (printerDC.GetHDC())
     {

-      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);
-
-      previewPrintout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
-      previewPrintout->SetPageSizeMM(printerWidth, printerHeight);
-      
-      if (logPPIPrinterX == 0 || logPPIPrinterY == 0 || printerWidth == 0 || printerHeight == 0)
-        isOk = FALSE;
+        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;

     }
     else
     }
     else

-      isOk = FALSE;
-
-    pageWidth = printerXRes;
-    pageHeight = printerYRes;
-
+        m_isOk = FALSE;
+    
+    m_pageWidth = printerXRes;
+    m_pageHeight = printerYRes;
+    

     // At 100%, the page should look about page-size on the screen.
     // At 100%, the page should look about page-size on the screen.

-    previewScale = (float)((float)screenWidth/(float)printerWidth);
-    previewScale = previewScale * (float)((float)screenXRes/(float)printerYRes);
+    m_previewScale = (float)((float)screenWidth/(float)printerWidth);
+    m_previewScale = m_previewScale * (float)((float)screenXRes/(float)printerYRes);

 }
 
 /****************************************************************************
 
 }
 
 /****************************************************************************
 

-    FUNCTION: wxAbortProc()
-
+  FUNCTION: wxAbortProc()
+  

     PURPOSE:  Processes messages for the Abort Dialog box
     PURPOSE:  Processes messages for the Abort Dialog box

-
+    

 ****************************************************************************/
 
 LONG APIENTRY _EXPORT wxAbortProc(HDC WXUNUSED(hPr), int WXUNUSED(Code))
 {
     MSG msg;
 ****************************************************************************/
 
 LONG APIENTRY _EXPORT wxAbortProc(HDC WXUNUSED(hPr), int WXUNUSED(Code))
 {
     MSG msg;

-
-    if (!wxPrinterBase::abortWindow)              /* If the abort dialog isn't up yet */
+    
+    if (!wxPrinterBase::sm_abortWindow)              /* If the abort dialog isn't up yet */

         return(TRUE);
         return(TRUE);

-
+    

     /* Process messages intended for the abort dialog box */
     /* Process messages intended for the abort dialog box */

-
-    while (!wxPrinterBase::abortIt && PeekMessage(&msg, 0, 0, 0, TRUE))
-        if (!IsDialogMessage((HWND) wxPrinterBase::abortWindow->GetHWND(), &msg)) {
+    
+    while (!wxPrinterBase::sm_abortIt && PeekMessage(&msg, 0, 0, 0, TRUE))
+        if (!IsDialogMessage((HWND) wxPrinterBase::sm_abortWindow->GetHWND(), &msg)) {

             TranslateMessage(&msg);
             DispatchMessage(&msg);
         }
             TranslateMessage(&msg);
             DispatchMessage(&msg);
         }

-
-    /* bAbort is TRUE (return is FALSE) if the user has aborted */
-
-    return (!wxPrinterBase::abortIt);
+        
+        /* bAbort is TRUE (return is FALSE) if the user has aborted */
+        
+        return (!wxPrinterBase::sm_abortIt);

 }
 
 }