fix unused variable warning when wxUSE_PALETTE==0

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

diff --git a/src/msw/printwin.cpp b/src/msw/printwin.cpp
index cd496d3291d7627229183ab42abadcb3336253c8..6856dd94620f9272bdb268f3cbe26b131f956c9a 100644 (file)
--- a/src/msw/printwin.cpp
+++ b/src/msw/printwin.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        printwin.cpp
+// Name:        src/msw/printwin.cpp

 // Purpose:     wxWindowsPrinter framework
 // Author:      Julian Smart
 // Modified by:
 // Purpose:     wxWindowsPrinter framework
 // Author:      Julian Smart
 // Modified by:


@@ -17,10 +17,6 @@
 // headers
 // ---------------------------------------------------------------------------
 
 // headers
 // ---------------------------------------------------------------------------
 

-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-    #pragma implementation "printwin.h"
-#endif
-

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 


@@ -28,13 +24,12 @@
     #pragma hdrstop
 #endif
 
     #pragma hdrstop
 #endif
 

-#include "wx/defs.h"
-

 // 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
 // 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/msw/wrapcdlg.h"

     #include "wx/window.h"
     #include "wx/msw/private.h"
     #include "wx/utils.h"
     #include "wx/window.h"
     #include "wx/msw/private.h"
     #include "wx/utils.h"


@@ -42,20 +37,16 @@
     #include "wx/app.h"
     #include "wx/msgdlg.h"
     #include "wx/intl.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"
 #endif
 
 #include "wx/msw/printwin.h"

-#include "wx/dcprint.h"

 #include "wx/printdlg.h"
 #include "wx/printdlg.h"

-#include "wx/log.h"

 #include "wx/msw/private.h"
 
 #include <stdlib.h>
 
 #include "wx/msw/private.h"
 
 #include <stdlib.h>
 

-#include "wx/msw/private.h"
-
-#include <commdlg.h>
-

 #ifndef __WIN32__
     #include <print.h>
 #endif
 #ifndef __WIN32__
     #include <print.h>
 #endif


@@ -361,28 +352,27 @@ bool wxWindowsPrintPreview::Print(bool interactive)
 
 void wxWindowsPrintPreview::DetermineScaling()
 {
 
 void wxWindowsPrintPreview::DetermineScaling()
 {

-    HDC dc = ::GetDC(NULL);
+    ScreenHDC dc;

     int screenWidth = ::GetDeviceCaps(dc, HORZSIZE);
     int screenWidth = ::GetDeviceCaps(dc, HORZSIZE);

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

     int logPPIScreenX = ::GetDeviceCaps(dc, LOGPIXELSX);
     int logPPIScreenY = ::GetDeviceCaps(dc, LOGPIXELSY);
     m_previewPrintout->SetPPIScreen(logPPIScreenX, logPPIScreenY);
 
     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(m_printDialogData.GetPrintData());
 
     int printerWidth = 150;
 
     // Get a device context for the currently selected printer
     wxPrinterDC printerDC(m_printDialogData.GetPrintData());
 
     int printerWidth = 150;

-    int printerHeight wxDUMMY_INITIALIZE(250);

     int printerXRes = 1500;
     int printerYRes = 2500;
 
     int printerXRes = 1500;
     int printerYRes = 2500;
 

-    dc = GetHdcOf(printerDC);
-    if ( dc )
+    if ( printerDC.Ok() )

     {
     {

+        HDC dc = GetHdcOf(printerDC);
+

         printerWidth = ::GetDeviceCaps(dc, HORZSIZE);
         printerWidth = ::GetDeviceCaps(dc, HORZSIZE);

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

         printerXRes = ::GetDeviceCaps(dc, HORZRES);
         printerYRes = ::GetDeviceCaps(dc, VERTRES);
 
         printerXRes = ::GetDeviceCaps(dc, HORZRES);
         printerYRes = ::GetDeviceCaps(dc, VERTRES);
 


@@ -392,18 +382,29 @@ void wxWindowsPrintPreview::DetermineScaling()
         m_previewPrintout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
         m_previewPrintout->SetPageSizeMM(printerWidth, printerHeight);
 
         m_previewPrintout->SetPPIPrinter(logPPIPrinterX, logPPIPrinterY);
         m_previewPrintout->SetPageSizeMM(printerWidth, printerHeight);
 

-        if (logPPIPrinterX == 0 || logPPIPrinterY == 0 || printerWidth == 0 || printerHeight == 0)
+        if ( logPPIPrinterX == 0 ||
+                logPPIPrinterY == 0 ||
+                    printerWidth == 0 ||
+                        printerHeight == 0 )
+        {

             m_isOk = false;
             m_isOk = false;

+        }

     }
     else
     }
     else

+    {

         m_isOk = false;
         m_isOk = false;

+    }

 
     m_pageWidth = printerXRes;
     m_pageHeight = printerYRes;
 
     // At 100%, the page should look about page-size on the screen.
 
     m_pageWidth = printerXRes;
     m_pageHeight = printerYRes;
 
     // 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)screenYRes/(float)printerYRes);
+    //
+    // TODO: the scale could be different in vertical and horizontal directions
+    float screenDPI = (float)screenXRes / screenWidth;
+    float printerDPI = (float)printerXRes / printerWidth;
+
+    m_previewScale = screenDPI / printerDPI;

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