gcc printf format warning fixed

[wxWidgets.git] / src / mac / carbon / printmac.cpp

diff --git a/src/mac/carbon/printmac.cpp b/src/mac/carbon/printmac.cpp
index 08a6bf77adc5cff8fed862dd499249361a3e0c25..9ce3354d4692bc6fc019c87aef19ff4a4321814b 100644 (file)
--- a/src/mac/carbon/printmac.cpp
+++ b/src/mac/carbon/printmac.cpp
@@ -1,51 +1,302 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        printwin.cpp
+// Name:        src/mac/carbon/printwin.cpp

 // Purpose:     wxMacPrinter framework
 // Author:      Julian Smart
 // Modified by:
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Purpose:     wxMacPrinter 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
-

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

+#if wxUSE_PRINTING_ARCHITECTURE
+

 #ifdef __BORLANDC__
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #endif
 

-#include "wx/defs.h"
-

 #ifndef WX_PRECOMP
 #ifndef WX_PRECOMP

-#include "wx/utils.h"
-#include "wx/dc.h"
-#include "wx/app.h"
-#include "wx/msgdlg.h"
+    #include "wx/utils.h"
+    #include "wx/dc.h"
+    #include "wx/app.h"
+    #include "wx/msgdlg.h"
+    #include "wx/dcprint.h"

 #endif
 
 #endif
 

+#include "wx/math.h"
+#include "wx/mac/uma.h"
+

 #include "wx/mac/printmac.h"
 #include "wx/mac/printmac.h"

-#include "wx/dcprint.h"
+#include "wx/mac/private/print.h"
+

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

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

 
 #include <stdlib.h>
 
 
 #include <stdlib.h>
 

-#if !USE_SHARED_LIBRARY
+IMPLEMENT_DYNAMIC_CLASS(wxMacCarbonPrintData, wxPrintNativeDataBase)

 IMPLEMENT_DYNAMIC_CLASS(wxMacPrinter, wxPrinterBase)
 IMPLEMENT_CLASS(wxMacPrintPreview, wxPrintPreviewBase)
 IMPLEMENT_DYNAMIC_CLASS(wxMacPrinter, wxPrinterBase)
 IMPLEMENT_CLASS(wxMacPrintPreview, wxPrintPreviewBase)

+
+bool wxMacCarbonPrintData::Ok() const
+{
+    return (m_macPageFormat != kPMNoPageFormat) && (m_macPrintSettings != kPMNoPrintSettings) && (m_macPrintSession != kPMNoReference);
+}
+wxMacCarbonPrintData::wxMacCarbonPrintData()
+{
+    m_macPageFormat = kPMNoPageFormat;
+    m_macPrintSettings = kPMNoPrintSettings;
+    m_macPrintSession = kPMNoReference ;
+    ValidateOrCreate() ;
+}
+
+wxMacCarbonPrintData::~wxMacCarbonPrintData()
+{
+    if (m_macPageFormat != kPMNoPageFormat)
+    {
+        (void)PMRelease(m_macPageFormat);
+        m_macPageFormat = kPMNoPageFormat;
+    }
+
+    if (m_macPrintSettings != kPMNoPrintSettings)
+    {
+        (void)PMRelease(m_macPrintSettings);
+        m_macPrintSettings = kPMNoPrintSettings;
+    }
+
+    if ( m_macPrintSession != kPMNoReference )
+    {
+        (void)PMRelease(m_macPrintSession);
+        m_macPrintSession = kPMNoReference;
+    }
+}
+
+void wxMacCarbonPrintData::ValidateOrCreate()
+{
+    OSStatus err = noErr ;
+    if ( m_macPrintSession == kPMNoReference )
+    {
+        err = PMCreateSession( (PMPrintSession *) &m_macPrintSession ) ;
+    }
+    //  Set up a valid PageFormat object.
+    if ( m_macPageFormat == kPMNoPageFormat)
+    {
+        err = PMCreatePageFormat((PMPageFormat *) &m_macPageFormat);
+
+        //  Note that PMPageFormat is not session-specific, but calling
+        //  PMSessionDefaultPageFormat assigns values specific to the printer
+        //  associated with the current printing session.
+        if ((err == noErr) &&
+            ( m_macPageFormat != kPMNoPageFormat))
+        {
+            err = PMSessionDefaultPageFormat((PMPrintSession) m_macPrintSession,
+                (PMPageFormat) m_macPageFormat);
+        }
+    }
+    else
+    {
+        err = PMSessionValidatePageFormat((PMPrintSession) m_macPrintSession,
+            (PMPageFormat) m_macPageFormat,
+            kPMDontWantBoolean);
+    }
+
+    //  Set up a valid PrintSettings object.
+    if ( m_macPrintSettings == kPMNoPrintSettings)
+    {
+        err = PMCreatePrintSettings((PMPrintSettings *) &m_macPrintSettings);
+
+        //  Note that PMPrintSettings is not session-specific, but calling
+        //  PMSessionDefaultPrintSettings assigns values specific to the printer
+        //  associated with the current printing session.
+        if ((err == noErr) &&
+            ( m_macPrintSettings != kPMNoPrintSettings))
+        {
+            err = PMSessionDefaultPrintSettings((PMPrintSession) m_macPrintSession,
+                (PMPrintSettings) m_macPrintSettings);
+        }
+    }
+    else
+    {
+        err = PMSessionValidatePrintSettings((PMPrintSession) m_macPrintSession,
+            (PMPrintSettings) m_macPrintSettings,
+            kPMDontWantBoolean);
+    }
+}
+
+bool wxMacCarbonPrintData::TransferFrom( const wxPrintData &data )
+{
+    ValidateOrCreate() ;
+    PMSetCopies( (PMPrintSettings) m_macPrintSettings , data.GetNoCopies() , false ) ;
+    PMSetOrientation( (PMPageFormat) m_macPageFormat , ( data.GetOrientation() == wxLANDSCAPE ) ?
+        kPMLandscape : kPMPortrait , false ) ;
+    // collate cannot be set
+#if 0 // not yet tested
+    if ( !m_printerName.empty() )
+        PMSessionSetCurrentPrinter( (PMPrintSession) m_macPrintSession , wxMacCFStringHolder( m_printerName , wxFont::GetDefaultEncoding() ) ) ;

 #endif
 #endif

+    PMColorMode color ;
+    PMGetColorMode(  (PMPrintSettings) m_macPrintSettings, &color ) ;
+    if ( data.GetColour() )
+    {
+        if ( color == kPMBlackAndWhite )
+            PMSetColorMode( (PMPrintSettings) m_macPrintSettings, kPMColor ) ;
+    }
+    else
+        PMSetColorMode( (PMPrintSettings) m_macPrintSettings, kPMBlackAndWhite ) ;
+
+    // PMDuplexMode not yet accessible via API
+    // PMQualityMode not yet accessible via API
+    // todo paperSize
+    PMResolution res;
+    PMPrinter printer;
+    PMTag tag = kPMMaxSquareResolution;
+    PMSessionGetCurrentPrinter(m_macPrintSession, &printer);
+    PMPrinterGetPrinterResolution(printer, tag, &res);
+    PMSetResolution((PMPageFormat) m_macPageFormat, &res);
+
+    return true ;
+}
+
+bool wxMacCarbonPrintData::TransferTo( wxPrintData &data )
+{
+    OSStatus err = noErr ;
+
+    UInt32 copies ;
+    err = PMGetCopies( m_macPrintSettings , &copies ) ;
+    if ( err == noErr )
+        data.SetNoCopies( copies ) ;
+
+    PMOrientation orientation ;
+    err = PMGetOrientation(  m_macPageFormat , &orientation ) ;
+    if ( err == noErr )
+    {
+        if ( orientation == kPMPortrait || orientation == kPMReversePortrait )
+            data.SetOrientation( wxPORTRAIT  );
+        else
+            data.SetOrientation( wxLANDSCAPE );
+    }
+
+    // collate cannot be set
+#if 0
+    {
+        wxMacCFStringHolder name ;
+        PMPrinter printer ;
+        PMSessionGetCurrentPrinter( m_macPrintSession ,
+            &printer ) ;
+        m_printerName = name.AsString() ;
+    }
+#endif
+
+    PMColorMode color ;
+    err = PMGetColorMode( m_macPrintSettings, &color ) ;
+    if ( err == noErr )
+        data.SetColour( !(color == kPMBlackAndWhite) ) ;
+
+    // PMDuplexMode not yet accessible via API
+    // PMQualityMode not yet accessible via API
+    // todo paperSize
+    PMRect rPaper;
+    err = PMGetUnadjustedPaperRect( m_macPageFormat, &rPaper);
+    if ( err == noErr )
+    {
+        data.SetPaperSize( wxSize (
+            (int)(( rPaper.right - rPaper.left ) * pt2mm + 0.5 ) ,
+             (int)(( rPaper.bottom - rPaper.top ) * pt2mm + 0.5 ) ) );
+    }
+    return true ;
+}
+
+void wxMacCarbonPrintData::TransferFrom( wxPageSetupData *data )
+{
+    // should we setup the page rect here ?
+    // since MacOS sometimes has two same paper rects with different
+    // page rects we could make it roundtrip safe perhaps
+}
+
+void wxMacCarbonPrintData::TransferTo( wxPageSetupData* data )
+{
+    PMRect rPaper;
+    OSStatus err = PMGetUnadjustedPaperRect(m_macPageFormat, &rPaper);
+    if ( err == noErr )
+    {
+        PMRect rPage ;
+        err = PMGetUnadjustedPageRect(m_macPageFormat , &rPage ) ;
+        if ( err == noErr )
+        {
+            data->SetMinMarginTopLeft( wxPoint (
+                (int)(((double) rPage.left - rPaper.left ) * pt2mm) ,
+                (int)(((double) rPage.top - rPaper.top ) * pt2mm) ) ) ;
+
+            data->SetMinMarginBottomRight( wxPoint (
+                (wxCoord)(((double) rPaper.right - rPage.right ) * pt2mm),
+                (wxCoord)(((double) rPaper.bottom - rPage.bottom ) * pt2mm)) ) ;
+
+            if ( data->GetMarginTopLeft().x < data->GetMinMarginTopLeft().x )
+                data->SetMarginTopLeft( wxPoint( data->GetMinMarginTopLeft().x ,
+                    data->GetMarginTopLeft().y ) ) ;
+
+            if ( data->GetMarginBottomRight().x < data->GetMinMarginBottomRight().x )
+                data->SetMarginBottomRight( wxPoint( data->GetMinMarginBottomRight().x ,
+                    data->GetMarginBottomRight().y ) );
+
+            if ( data->GetMarginTopLeft().y < data->GetMinMarginTopLeft().y )
+                data->SetMarginTopLeft( wxPoint( data->GetMarginTopLeft().x , data->GetMinMarginTopLeft().y ) );
+
+            if ( data->GetMarginBottomRight().y < data->GetMinMarginBottomRight().y )
+                data->SetMarginBottomRight( wxPoint( data->GetMarginBottomRight().x ,
+                    data->GetMinMarginBottomRight().y) );
+        }
+    }
+}
+
+void wxMacCarbonPrintData::TransferTo( wxPrintDialogData* data )
+{
+    UInt32 minPage , maxPage ;
+    PMGetPageRange( m_macPrintSettings , &minPage , &maxPage ) ;
+    data->SetMinPage( minPage ) ;
+    data->SetMaxPage( maxPage ) ;
+    UInt32 copies ;
+    PMGetCopies( m_macPrintSettings , &copies ) ;
+    data->SetNoCopies( copies ) ;
+    UInt32 from , to ;
+    PMGetFirstPage( m_macPrintSettings , &from ) ;
+    PMGetLastPage( m_macPrintSettings , &to ) ;
+    if ( to >= 0x7FFFFFFF ) //  due to an OS Bug we don't get back kPMPrintAllPages
+    {
+        data->SetAllPages( true ) ;
+        // This means all pages, more or less
+        data->SetFromPage(1);
+        data->SetToPage(32000);
+    }
+    else
+    {
+        data->SetFromPage( from ) ;
+        data->SetToPage( to ) ;
+        data->SetAllPages( false );
+    }
+}
+
+void wxMacCarbonPrintData::TransferFrom( wxPrintDialogData* data )
+{
+    PMSetPageRange( m_macPrintSettings , data->GetMinPage() , data->GetMaxPage() ) ;
+    PMSetCopies( m_macPrintSettings , data->GetNoCopies() , false ) ;
+    PMSetFirstPage( m_macPrintSettings , data->GetFromPage() , false ) ;
+
+    if (data->GetAllPages() || data->GetFromPage() == 0)
+        PMSetLastPage( m_macPrintSettings , (UInt32) kPMPrintAllPages, true ) ;
+    else
+        PMSetLastPage( m_macPrintSettings , (UInt32) data->GetToPage() , false ) ;
+}

 
 /*
 
 /*

- * Printer
- */
- 
+* Printer
+*/
+

 wxMacPrinter::wxMacPrinter(wxPrintDialogData *data):
 wxMacPrinter::wxMacPrinter(wxPrintDialogData *data):

-  wxPrinterBase(data)
+wxPrinterBase(data)

 {
 }
 
 {
 }
 


@@ -55,151 +306,156 @@ wxMacPrinter::~wxMacPrinter(void)
 
 bool wxMacPrinter::Print(wxWindow *parent, wxPrintout *printout, bool prompt)
 {
 
 bool wxMacPrinter::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_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);
-
- // Create a suitable device context  
-  wxDC *dc = NULL;
-  if (prompt)
-  {
+    sm_abortIt = false;
+    sm_abortWindow = NULL;
+
+    if (!printout)
+        return false;
+
+    printout->SetIsPreview(false);
+    if (m_printDialogData.GetMinPage() < 1)
+        m_printDialogData.SetMinPage(1);
+    if (m_printDialogData.GetMaxPage() < 1)
+        m_printDialogData.SetMaxPage(9999);
+
+    // Create a suitable device context
+    wxDC *dc = NULL;
+    if (prompt)
+    {

         wxPrintDialog dialog(parent, & m_printDialogData);
         if (dialog.ShowModal() == wxID_OK)
         wxPrintDialog dialog(parent, & m_printDialogData);
         if (dialog.ShowModal() == wxID_OK)

-               {
-                    dc = dialog.GetPrintDC();
-                    m_printDialogData = dialog.GetPrintData();
-               }
-  }
-  else
-  {
-               dc = new wxPrinterDC( m_printDialogData.GetPrintData() ) ;
-  }
-
-
-  // May have pressed cancel.
-  if (!dc || !dc->Ok())
-  {
-    if (dc) delete dc;
-    return FALSE;
-  }
-  
-       // on the mac we have always pixels as addressing mode with 72 dpi
-
-  printout->SetPPIScreen(72, 72);
-  printout->SetPPIPrinter(72, 72);
-
-  // Set printout parameters  
-  printout->SetDC(dc);
-
-  int w, h;
-  wxCoord 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();
+        {
+            dc = dialog.GetPrintDC();
+            m_printDialogData = dialog.GetPrintDialogData();
+        }
+    }
+    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;
+    }

 
 

-  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();
-*/
+    // on the mac we have always pixels as addressing mode with 72 dpi
+    printout->SetPPIScreen(72, 72);
+    PMResolution res;
+    wxMacCarbonPrintData* nativeData = (wxMacCarbonPrintData*)
+          (m_printDialogData.GetPrintData().GetNativeData());
+    PMGetResolution((PMPageFormat) (nativeData->m_macPageFormat), &res);
+    printout->SetPPIPrinter(int(res.hRes), int(res.vRes));
+
+    // Set printout parameters
+    printout->SetDC(dc);
+
+    int w, h;
+    wxCoord 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();
+
+    printout->OnPreparePrinting();

 
 

-  printout->OnBeginPrinting();
-  
-  bool keepGoing = TRUE;
+    // Get some parameters from the printout, if defined
+    int fromPage, toPage;
+    int minPage, maxPage;
+    printout->GetPageInfo(&minPage, &maxPage, &fromPage, &toPage);

 
 

-  int copyCount;
-  for (copyCount = 1; copyCount <= m_printDialogData.GetNoCopies(); copyCount ++)
-  {
-    if (!printout->OnBeginDocument(m_printDialogData.GetFromPage(), m_printDialogData.GetToPage()))
+    if (maxPage == 0)

     {
     {

-      wxEndBusyCursor();
-      wxMessageBox("Could not start printing.", "Print Error", wxOK, parent);
-      break;
+        wxEndBusyCursor();
+        return false;

     }
     }

-    if (sm_abortIt)
-      break;

 
 

-    int pn;
-    for (pn = m_printDialogData.GetFromPage(); keepGoing && (pn <= m_printDialogData.GetToPage()) && printout->HasPage(pn);
-         pn++)
+    // 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);
+    wxSafeYield(win,true);
+
+    if (!win)

     {
     {

-      if (sm_abortIt)
-      {
-        keepGoing = FALSE;
-        break;
-      }
-      else
-      {
-        dc->StartPage();
-        keepGoing = printout->OnPrintPage(pn);
-        dc->EndPage();
-      }
+        wxEndBusyCursor();
+        wxMessageBox(wxT("Sorry, could not create an abort dialog."), wxT("Print Error"), wxOK, parent);
+        delete dc;
+
+        return false;

     }
     }

-    printout->OnEndDocument();
-  }

 
 

-  printout->OnEndPrinting();
+    sm_abortWindow = win;
+    sm_abortWindow->Show(true);
+    wxSafeYield(win,true);

 
 

-  if (sm_abortWindow)
-  {
-    sm_abortWindow->Show(FALSE);
-    delete sm_abortWindow;
-    sm_abortWindow = NULL;
-  }
+    printout->OnBeginPrinting();
+
+    bool keepGoing = true;
+
+    int copyCount;
+    for (copyCount = 1; copyCount <= m_printDialogData.GetNoCopies(); copyCount ++)
+    {
+        if (!printout->OnBeginDocument(m_printDialogData.GetFromPage(), m_printDialogData.GetToPage()))
+        {
+            wxEndBusyCursor();
+            wxMessageBox(wxT("Could not start printing."), wxT("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
+            {
+#if TARGET_CARBON
+                if ( UMAGetSystemVersion() >= 0x1000 )
+#endif
+                {
+                    GrafPtr thePort ;
+                    GetPort( &thePort ) ;
+                    wxSafeYield(win,true);
+                    SetPort( thePort ) ;
+                }
+                dc->StartPage();
+                keepGoing = printout->OnPrintPage(pn);
+                dc->EndPage();
+            }
+        }
+        printout->OnEndDocument();
+    }
+
+    printout->OnEndPrinting();
+
+    if (sm_abortWindow)
+    {
+        sm_abortWindow->Show(false);
+        delete sm_abortWindow;
+        sm_abortWindow = NULL;
+    }
+
+    wxEndBusyCursor();

 
 

-  wxEndBusyCursor();
+    delete dc;

 
 

-  delete dc;
-  
-  return TRUE;
+    return true;

 }
 
 wxDC* wxMacPrinter::PrintDialog(wxWindow *parent)
 }
 
 wxDC* wxMacPrinter::PrintDialog(wxWindow *parent)


@@ -220,35 +476,37 @@ wxDC* wxMacPrinter::PrintDialog(wxWindow *parent)
 
 bool wxMacPrinter::Setup(wxWindow *parent)
 {
 
 bool wxMacPrinter::Setup(wxWindow *parent)
 {

+#if 0

     wxPrintDialog dialog(parent, & m_printDialogData);
     wxPrintDialog dialog(parent, & m_printDialogData);

-    dialog.GetPrintDialogData().SetSetupDialog(TRUE);
+    dialog.GetPrintDialogData().SetSetupDialog(true);

 
     int ret = dialog.ShowModal();
 
     if (ret == wxID_OK)
 
     int ret = dialog.ShowModal();
 
     if (ret == wxID_OK)

-    {

         m_printDialogData = dialog.GetPrintDialogData();
         m_printDialogData = dialog.GetPrintDialogData();

-    }

 
     return (ret == wxID_OK);
 
     return (ret == wxID_OK);

+#endif
+
+    return wxID_CANCEL;

 }
 
 /*
 }
 
 /*

- * Print preview
- */
+* Print preview
+*/

 
 wxMacPrintPreview::wxMacPrintPreview(wxPrintout *printout,
 
 wxMacPrintPreview::wxMacPrintPreview(wxPrintout *printout,

-                                             wxPrintout *printoutForPrinting,
-                                             wxPrintDialogData *data)
-                     : wxPrintPreviewBase(printout, printoutForPrinting, data)
+                                     wxPrintout *printoutForPrinting,
+                                     wxPrintDialogData *data)
+                                     : wxPrintPreviewBase(printout, printoutForPrinting, data)

 {
     DetermineScaling();
 }
 
 wxMacPrintPreview::wxMacPrintPreview(wxPrintout *printout, wxPrintout *printoutForPrinting, wxPrintData *data):
 {
     DetermineScaling();
 }
 
 wxMacPrintPreview::wxMacPrintPreview(wxPrintout *printout, wxPrintout *printoutForPrinting, wxPrintData *data):

-  wxPrintPreviewBase(printout, printoutForPrinting, data)
+wxPrintPreviewBase(printout, printoutForPrinting, data)

 {
 {

-  DetermineScaling();
+    DetermineScaling();

 }
 
 wxMacPrintPreview::~wxMacPrintPreview(void)
 }
 
 wxMacPrintPreview::~wxMacPrintPreview(void)


@@ -257,21 +515,22 @@ wxMacPrintPreview::~wxMacPrintPreview(void)
 
 bool wxMacPrintPreview::Print(bool interactive)
 {
 
 bool wxMacPrintPreview::Print(bool interactive)
 {

-  if (!m_printPrintout)
-    return FALSE;
-  wxMacPrinter printer(&m_printDialogData);
-  return printer.Print(m_previewFrame, m_printPrintout, interactive);
+    if (!m_printPrintout)
+        return false;
+
+    wxMacPrinter printer(&m_printDialogData);
+    return printer.Print(m_previewFrame, m_printPrintout, interactive);

 }
 
 void wxMacPrintPreview::DetermineScaling(void)
 {
 }
 
 void wxMacPrintPreview::DetermineScaling(void)
 {

-       int screenWidth , screenHeight ;
-       wxDisplaySize( &screenWidth , &screenHeight ) ;
-       
-       m_previewPrintout->SetPPIScreen( 72 , 72 ) ;
-       m_previewPrintout->SetPPIPrinter( 72 , 72 ) ;
-       m_previewPrintout->SetPageSizeMM( 8 * 25.6 , 11 * 25.6 ) ;
-       m_previewPrintout->SetPageSizePixels( 8 * 72 , 11 * 72 ) ;
+    int screenWidth , screenHeight ;
+    wxDisplaySize( &screenWidth , &screenHeight ) ;
+
+    m_previewPrintout->SetPPIScreen( 72 , 72 ) ;
+    m_previewPrintout->SetPPIPrinter( 72 , 72 ) ;
+    m_previewPrintout->SetPageSizeMM( (int) (8.0 * 25.6), (int) (11.0 * 25.6) );
+    m_previewPrintout->SetPageSizePixels( 8 * 72 , 11 * 72 ) ;

     m_pageWidth = 8 * 72 ;
     m_pageHeight = 11 * 72 ;
     m_previewScale = 1 ;
     m_pageWidth = 8 * 72 ;
     m_pageHeight = 11 * 72 ;
     m_previewScale = 1 ;


@@ -280,20 +539,26 @@ void wxMacPrintPreview::DetermineScaling(void)
     wxPrinterDC printerDC(m_printDialogData.GetPrintData());
     if (printerDC.Ok())
     {
     wxPrinterDC printerDC(m_printDialogData.GetPrintData());
     if (printerDC.Ok())
     {

-               int x , y ;
-               wxCoord ww, hh;
-               printerDC.GetSizeMM(&ww, &hh);
-               printerDC.GetSize( &x , &y ) ;
-               m_previewPrintout->SetPageSizeMM((int)ww, (int)hh);
-               m_previewPrintout->SetPageSizePixels( x , y) ;
-               m_pageWidth = x ;
-               m_pageHeight =  y ;
-               m_isOk = true ;
-
-       }
+        int x , y ;
+        wxCoord ww, hh;
+        printerDC.GetSizeMM(&ww, &hh);
+        printerDC.GetSize( &x , &y ) ;
+        m_previewPrintout->SetPageSizeMM((int)ww, (int)hh);
+        m_previewPrintout->SetPageSizePixels( x , y) ;
+        m_pageWidth = x ;
+        m_pageHeight =  y ;
+        m_isOk = true ;
+    }
+    else
+    {
+        m_isOk = false ;
+    }
+

     // 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)printerXRes);
     // 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)printerXRes);

-    
+

     m_previewScale = 1 ;
 }
     m_previewScale = 1 ;
 }

+
+#endif