]> git.saurik.com Git - wxWidgets.git/blobdiff - samples/printing/printing.cpp
correcting text alignment flag
[wxWidgets.git] / samples / printing / printing.cpp
index f76894eb4f9ceabddcf513b035bb790cf0e991f8..ace34b5d7f1c35b80d6e24da3e3bb4fcb45ce63c 100644 (file)
@@ -1,31 +1,28 @@
-/*
-* File:        printing.cc
-* Purpose:     Printing demo for wxWindows class library
-* Author:      Julian Smart
-* Created:     1995
-* Updated:     
-* Copyright:   (c) 1995, AIAI, University of Edinburgh
-*/
-
-/* static const char sccsid[] = "%W% %G%"; */
-
-#ifdef __GNUG__
-#pragma implementation
-#endif
+/////////////////////////////////////////////////////////////////////////////
+// Name:        samples/printing.cpp
+// Purpose:     Printing demo for wxWidgets
+// Author:      Julian Smart
+// Modified by: Francesco Montorsi
+// Created:     1995
+// RCS-ID:      $Id$
+// Copyright:   (c) Julian Smart
+// Licence:     wxWindows licence
+/////////////////////////////////////////////////////////////////////////////
 
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP
-#include "wx/wx.h"
+    #include "wx/wx.h"
+    #include "wx/log.h"
 #endif
 
 #if !wxUSE_PRINTING_ARCHITECTURE
 #endif
 
 #if !wxUSE_PRINTING_ARCHITECTURE
-#error You must set wxUSE_PRINTING_ARCHITECTURE to 1 in setup.h to compile this demo.
+    #error "You must set wxUSE_PRINTING_ARCHITECTURE to 1 in setup.h, and recompile the library."
 #endif
 
 // Set this to 1 if you want to test PostScript printing under MSW.
 #endif
 
 // Set this to 1 if you want to test PostScript printing under MSW.
 #include "wx/metafile.h"
 #include "wx/print.h"
 #include "wx/printdlg.h"
 #include "wx/metafile.h"
 #include "wx/print.h"
 #include "wx/printdlg.h"
-
+#include "wx/image.h"
 #include "wx/accel.h"
 
 #if wxTEST_POSTSCRIPT_IN_MSW
 #include "wx/accel.h"
 
 #if wxTEST_POSTSCRIPT_IN_MSW
-#include "wx/generic/printps.h"
-#include "wx/generic/prntdlgg.h"
+    #include "wx/generic/printps.h"
+    #include "wx/generic/prntdlgg.h"
 #endif
 
 #endif
 
-#include "printing.h"
+#if wxUSE_GRAPHICS_CONTEXT
+    #include "wx/graphics.h"
+#endif
 
 
-#ifndef __WXMSW__
-#include "mondrian.xpm"
+#ifdef __WXMAC__
+    #include "wx/osx/printdlg.h"
 #endif
 
 #endif
 
-// Declare a frame
-MyFrame   *frame = (MyFrame *) NULL;
-// int orientation = wxPORTRAIT;
+#include "printing.h"
+
+#ifndef wxHAS_IMAGES_IN_RESOURCES
+    #include "../sample.xpm"
+#endif
 
 // Global print data, to remember settings during the session
 
 // Global print data, to remember settings during the session
-wxPrintData *g_printData = (wxPrintData*) NULL ;
+wxPrintData *g_printData = NULL;
 
 // Global page setup data
 
 // Global page setup data
-wxPageSetupData* g_pageSetupData = (wxPageSetupData*) NULL;
+wxPageSetupDialogData* g_pageSetupData = NULL;
 
 
-// Main proc
-IMPLEMENT_APP(MyApp)
 
 
-// Writes a header on a page. Margin units are in millimetres.
-bool WritePageHeader(wxPrintout *printout, wxDC *dc, char *text, float mmToLogical);
 
 
-MyApp::MyApp()
-{
-}
+// ----------------------------------------------------------------------------
+// MyApp
+// ----------------------------------------------------------------------------
+
+IMPLEMENT_APP(MyApp)
 
 
-// The `main program' equivalent, creating the windows and returning the
-// main frame
 bool MyApp::OnInit(void)
 {
 bool MyApp::OnInit(void)
 {
-    m_testFont = new wxFont(10, wxSWISS, wxNORMAL, wxNORMAL);
-    
+    if ( !wxApp::OnInit() )
+        return false;
+
+    wxInitAllImageHandlers();
+
+
+    // init global objects
+    // -------------------
+
     g_printData = new wxPrintData;
     g_printData = new wxPrintData;
+
+    // You could set an initial paper size here
+#if 0
+    g_printData->SetPaperId(wxPAPER_LETTER); // for Americans
+    g_printData->SetPaperId(wxPAPER_A4);    // for everyone else
+#endif
+
     g_pageSetupData = new wxPageSetupDialogData;
 
     g_pageSetupData = new wxPageSetupDialogData;
 
-    // Compatibility with old system. In fact, we might keep wxThePrintSetupData
-    // just for useful default values which we can optionally assign to our
-    // own print data object.
+    // copy over initial paper size from print record
+    (*g_pageSetupData) = *g_printData;
+
+    // Set some initial page margins in mm.
+    g_pageSetupData->SetMarginTopLeft(wxPoint(15, 15));
+    g_pageSetupData->SetMarginBottomRight(wxPoint(15, 15));
+
 
 
-#if defined(__WXGTK__) || defined(__WXMOTIF__)
-    (*g_printData) = * wxThePrintSetupData;
+    // init local GUI objects
+    // ----------------------
+
+#if 0
+    wxImage image( wxT("test.jpg") );
+    image.SetAlpha();
+    int i,j;
+    for (i = 0; i < image.GetWidth(); i++)
+       for (j = 0; j < image.GetHeight(); j++)
+          image.SetAlpha( i, j, 50 );
+    m_bitmap = image;
 #endif
 #endif
-    
+    m_angle = 30;
+    m_testFont.Create(10, wxFONTFAMILY_SWISS, wxFONTSTYLE_NORMAL, wxFONTWEIGHT_NORMAL);
+
+
     // Create the main frame window
     // Create the main frame window
-    frame = new MyFrame((wxFrame *) NULL, (char *) "wxWindows Printing Demo", wxPoint(0, 0), wxSize(400, 400));
-    
-    // Give it a status line
-    frame->CreateStatusBar(2);
-    
+    // ----------------------------
+
+    MyFrame* frame = new MyFrame((wxFrame *) NULL, wxT("wxWidgets Printing Demo"),
+                                 wxPoint(0, 0), wxSize(400, 400));
+
+    frame->Centre(wxBOTH);
+    frame->Show();
+
+    return true;
+}
+
+int MyApp::OnExit()
+{
+    delete g_printData;
+    delete g_pageSetupData;
+
+    return wxApp::OnExit();
+}
+
+void MyApp::Draw(wxDC&dc)
+{
+    // This routine just draws a bunch of random stuff on the screen so that we
+    // can check that different types of object are being drawn consistently
+    // between the screen image, the print preview image (at various zoom
+    // levels), and the printed page.
+    dc.SetBackground(*wxWHITE_BRUSH);
+    // dc.Clear();
+    dc.SetFont(m_testFont);
+
+    // dc.SetBackgroundMode(wxTRANSPARENT);
+
+    dc.SetPen(*wxBLACK_PEN);
+    dc.SetBrush(*wxLIGHT_GREY_BRUSH);
+
+    dc.DrawRectangle(0, 0, 230, 350);
+    dc.DrawLine(0, 0, 229, 349);
+    dc.DrawLine(229, 0, 0, 349);
+    dc.SetBrush(*wxTRANSPARENT_BRUSH);
+
+    dc.SetBrush(*wxCYAN_BRUSH);
+    dc.SetPen(*wxRED_PEN);
+
+    dc.DrawRoundedRectangle(0, 20, 200, 80, 20);
+
+    dc.DrawText( wxT("Rectangle 200 by 80"), 40, 40);
+
+    dc.SetPen( wxPen(*wxBLACK,0,wxDOT_DASH) );
+    dc.DrawEllipse(50, 140, 100, 50);
+    dc.SetPen(*wxRED_PEN);
+
+    dc.DrawText( wxT("Test message: this is in 10 point text"), 10, 180);
+
+#if wxUSE_UNICODE
+    const char *test = "Hebrew    שלום -- Japanese (日本語)";
+    wxString tmp = wxConvUTF8.cMB2WC( test );
+    dc.DrawText( tmp, 10, 200 );
+#endif
+
+    wxPoint points[5];
+    points[0].x = 0;
+    points[0].y = 0;
+    points[1].x = 20;
+    points[1].y = 0;
+    points[2].x = 20;
+    points[2].y = 20;
+    points[3].x = 10;
+    points[3].y = 20;
+    points[4].x = 10;
+    points[4].y = -20;
+    dc.DrawPolygon( 5, points, 20, 250, wxODDEVEN_RULE );
+    dc.DrawPolygon( 5, points, 50, 250, wxWINDING_RULE );
+
+    dc.DrawEllipticArc( 80, 250, 60, 30, 0.0, 270.0 );
+
+    points[0].x = 150;
+    points[0].y = 250;
+    points[1].x = 180;
+    points[1].y = 250;
+    points[2].x = 180;
+    points[2].y = 220;
+    points[3].x = 200;
+    points[3].y = 220;
+    dc.DrawSpline( 4, points );
+
+    dc.DrawArc( 20,10, 10,10, 25,40 );
+
+    wxString str;
+    int i = 0;
+    str.Printf( wxT("---- Text at angle %d ----"), i );
+    dc.DrawRotatedText( str, 100, 300, i );
+
+    i = m_angle;
+    str.Printf( wxT("---- Text at angle %d ----"), i );
+    dc.DrawRotatedText( str, 100, 300, i );
+
+    wxIcon my_icon = wxICON(sample);
+
+    dc.DrawIcon( my_icon, 100, 100);
+
+    if (m_bitmap.IsOk())
+        dc.DrawBitmap( m_bitmap, 10, 10 );
+
+#if wxUSE_GRAPHICS_CONTEXT
+    wxGraphicsContext *gc = NULL;
+
+    wxPrinterDC *printer_dc = wxDynamicCast( &dc, wxPrinterDC );
+    if (printer_dc)
+        gc = wxGraphicsContext::Create( *printer_dc );
+
+    wxWindowDC *window_dc = wxDynamicCast( &dc, wxWindowDC );
+    if (window_dc)
+        gc = wxGraphicsContext::Create( *window_dc );
+
+#ifdef __WXMSW__
+    wxEnhMetaFileDC *emf_dc = wxDynamicCast( &dc, wxEnhMetaFileDC );
+    if (emf_dc)
+        gc = wxGraphicsContext::Create( *emf_dc );
+#endif
+
+    if (gc)
+    {
+        // make a path that contains a circle and some lines, centered at 100,100
+        gc->SetPen( *wxRED_PEN );
+
+        wxGraphicsPath path = gc->CreatePath();
+        path.AddCircle( 50.0, 50.0, 50.0 );
+        path.MoveToPoint(0.0, 50.0);
+        path.AddLineToPoint(100.0, 50.0);
+        path.MoveToPoint(50.0, 0.0);
+        path.AddLineToPoint(50.0, 100.0 );
+        path.CloseSubpath();
+        path.AddRectangle(25.0, 25.0, 50.0, 50.0);
+
+        gc->StrokePath(path);
+
+        // draw some text
+        wxString text("Text by wxGraphicsContext");
+        gc->SetFont( m_testFont, *wxBLACK );
+        gc->DrawText(text, 25.0, 60.0);
+
+        // draw rectangle around the text
+        double w, h, d, el;
+        gc->GetTextExtent(text, &w, &h, &d, &el);
+        gc->SetPen( *wxBLACK_PEN );
+        gc->DrawRectangle(25.0, 60.0, w, h);
+
+        delete gc;
+    }
+#endif
+}
+
+
+// ----------------------------------------------------------------------------
+// MyFrame
+// ----------------------------------------------------------------------------
+
+BEGIN_EVENT_TABLE(MyFrame, wxFrame)
+    EVT_MENU(wxID_EXIT, MyFrame::OnExit)
+    EVT_MENU(wxID_PRINT, MyFrame::OnPrint)
+    EVT_MENU(wxID_PREVIEW, MyFrame::OnPrintPreview)
+    EVT_MENU(WXPRINT_PAGE_SETUP, MyFrame::OnPageSetup)
+    EVT_MENU(wxID_ABOUT, MyFrame::OnPrintAbout)
+#if defined(__WXMSW__) &&wxTEST_POSTSCRIPT_IN_MSW
+    EVT_MENU(WXPRINT_PRINT_PS, MyFrame::OnPrintPS)
+    EVT_MENU(WXPRINT_PREVIEW_PS, MyFrame::OnPrintPreviewPS)
+    EVT_MENU(WXPRINT_PAGE_SETUP_PS, MyFrame::OnPageSetupPS)
+#endif
+#ifdef __WXMAC__
+    EVT_MENU(WXPRINT_PAGE_MARGINS, MyFrame::OnPageMargins)
+#endif
+    EVT_MENU(WXPRINT_ANGLEUP, MyFrame::OnAngleUp)
+    EVT_MENU(WXPRINT_ANGLEDOWN, MyFrame::OnAngleDown)
+
+    EVT_MENU_RANGE(WXPRINT_FRAME_MODAL_APP,
+                   WXPRINT_FRAME_MODAL_NON,
+                   MyFrame::OnPreviewFrameModalityKind)
+END_EVENT_TABLE()
+
+MyFrame::MyFrame(wxFrame *frame, const wxString&title, const wxPoint&pos, const wxSize&size)
+        : wxFrame(frame, wxID_ANY, title, pos, size)
+{
+    m_canvas = NULL;
+    m_previewModality = wxPreviewFrame_AppModal;
+
+#if wxUSE_STATUSBAR
+    // Give us a status line
+    CreateStatusBar(2);
+    SetStatusText(wxT("Printing demo"));
+#endif // wxUSE_STATUSBAR
+
     // Load icon and bitmap
     // Load icon and bitmap
-    frame->SetIcon( wxICON( mondrian) );
-    
+    SetIcon( wxICON( sample) );
+
     // Make a menubar
     wxMenu *file_menu = new wxMenu;
     // Make a menubar
     wxMenu *file_menu = new wxMenu;
-    
-    file_menu->Append(WXPRINT_PRINT, "&Print...",              "Print");
-    file_menu->Append(WXPRINT_PRINT_SETUP, "Print &Setup...",              "Setup printer properties");
-    file_menu->Append(WXPRINT_PAGE_SETUP, "Page Set&up...",              "Page setup");
-    file_menu->Append(WXPRINT_PREVIEW, "Print Pre&view",              "Preview");
-    
+
+    file_menu->Append(wxID_PRINT, wxT("&Print..."),                 wxT("Print"));
+    file_menu->Append(WXPRINT_PAGE_SETUP, wxT("Page Set&up..."),    wxT("Page setup"));
+#ifdef __WXMAC__
+    file_menu->Append(WXPRINT_PAGE_MARGINS, wxT("Page Margins..."), wxT("Page margins"));
+#endif
+    file_menu->Append(wxID_PREVIEW, wxT("Print Pre&view"),          wxT("Preview"));
+
+    wxMenu * const menuModalKind = new wxMenu;
+    menuModalKind->AppendRadioItem(WXPRINT_FRAME_MODAL_APP, "&App modal");
+    menuModalKind->AppendRadioItem(WXPRINT_FRAME_MODAL_WIN, "&Window modal");
+    menuModalKind->AppendRadioItem(WXPRINT_FRAME_MODAL_NON, "&Not modal");
+    file_menu->AppendSubMenu(menuModalKind, "Preview frame &modal kind");
+#if wxUSE_ACCEL
     // Accelerators
     wxAcceleratorEntry entries[1];
     // Accelerators
     wxAcceleratorEntry entries[1];
-    entries[0].Set(wxACCEL_CTRL, (int) 'V', WXPRINT_PREVIEW);
+    entries[0].Set(wxACCEL_CTRL, (int) 'V', wxID_PREVIEW);
     wxAcceleratorTable accel(1, entries);
     wxAcceleratorTable accel(1, entries);
-    frame->SetAcceleratorTable(accel);
-    
-#if defined(__WXMSW__) && wxTEST_POSTSCRIPT_IN_MSW
+    SetAcceleratorTable(accel);
+#endif
+
+#if defined(__WXMSW__) &&wxTEST_POSTSCRIPT_IN_MSW
     file_menu->AppendSeparator();
     file_menu->AppendSeparator();
-    file_menu->Append(WXPRINT_PRINT_PS, "Print PostScript...",              "Print (PostScript)");
-    file_menu->Append(WXPRINT_PRINT_SETUP_PS, "Print Setup PostScript...",              "Setup printer properties (PostScript)");
-    file_menu->Append(WXPRINT_PAGE_SETUP_PS, "Page Setup PostScript...",              "Page setup (PostScript)");
-    file_menu->Append(WXPRINT_PREVIEW_PS, "Print Preview PostScript",              "Preview (PostScript)");
+    file_menu->Append(WXPRINT_PRINT_PS, wxT("Print PostScript..."),           wxT("Print (PostScript)"));
+    file_menu->Append(WXPRINT_PAGE_SETUP_PS, wxT("Page Setup PostScript..."), wxT("Page setup (PostScript)"));
+    file_menu->Append(WXPRINT_PREVIEW_PS, wxT("Print Preview PostScript"),    wxT("Preview (PostScript)"));
 #endif
 #endif
+
     file_menu->AppendSeparator();
     file_menu->AppendSeparator();
-    file_menu->Append(WXPRINT_QUIT, "E&xit",                "Exit program");
-    
+    file_menu->Append(WXPRINT_ANGLEUP, wxT("Angle up\tAlt-U"),                wxT("Raise rotated text angle"));
+    file_menu->Append(WXPRINT_ANGLEDOWN, wxT("Angle down\tAlt-D"),            wxT("Lower rotated text angle"));
+    file_menu->AppendSeparator();
+    file_menu->Append(wxID_EXIT, wxT("E&xit"),                                wxT("Exit program"));
+
     wxMenu *help_menu = new wxMenu;
     wxMenu *help_menu = new wxMenu;
-    help_menu->Append(WXPRINT_ABOUT, "&About",              "About this demo");
-    
+    help_menu->Append(wxID_ABOUT, wxT("&About"),                              wxT("About this demo"));
+
     wxMenuBar *menu_bar = new wxMenuBar;
     wxMenuBar *menu_bar = new wxMenuBar;
-    
-    menu_bar->Append(file_menu, "&File");
-    menu_bar->Append(help_menu, "&Help");
-    
+
+    menu_bar->Append(file_menu, wxT("&File"));
+    menu_bar->Append(help_menu, wxT("&Help"));
+
     // Associate the menu bar with the frame
     // Associate the menu bar with the frame
-    frame->SetMenuBar(menu_bar);
-    
-    MyCanvas *canvas = new MyCanvas(frame, wxPoint(0, 0), wxSize(100, 100), wxRETAINED|wxHSCROLL|wxVSCROLL);
-    
-    // Give it scrollbars: the virtual canvas is 20 * 50 = 1000 pixels in each direction
-    canvas->SetScrollbars(20, 20, 50, 50);
-    
-    frame->canvas = canvas;
-    
-    frame->Centre(wxBOTH);
-    frame->Show(TRUE);
-    
-    frame->SetStatusText("Printing demo");
-    
-    SetTopWindow(frame);
-    
-    return TRUE;
-}
+    SetMenuBar(menu_bar);
 
 
-int MyApp::OnExit()
-{
-    delete wxGetApp().m_testFont;
-    delete g_printData;
-    delete g_pageSetupData;
-    return 1;
-}
 
 
-BEGIN_EVENT_TABLE(MyFrame, wxFrame)
-EVT_MENU(WXPRINT_QUIT, MyFrame::OnExit)
-EVT_MENU(WXPRINT_PRINT, MyFrame::OnPrint)
-EVT_MENU(WXPRINT_PREVIEW, MyFrame::OnPrintPreview)
-EVT_MENU(WXPRINT_PRINT_SETUP, MyFrame::OnPrintSetup)
-EVT_MENU(WXPRINT_PAGE_SETUP, MyFrame::OnPageSetup)
-EVT_MENU(WXPRINT_ABOUT, MyFrame::OnPrintAbout)
-#if defined(__WXMSW__) && wxTEST_POSTSCRIPT_IN_MSW
-EVT_MENU(WXPRINT_PRINT_PS, MyFrame::OnPrintPS)
-EVT_MENU(WXPRINT_PREVIEW_PS, MyFrame::OnPrintPreviewPS)
-EVT_MENU(WXPRINT_PRINT_SETUP_PS, MyFrame::OnPrintSetupPS)
-EVT_MENU(WXPRINT_PAGE_SETUP_PS, MyFrame::OnPageSetupPS)
-#endif
-END_EVENT_TABLE()
+    // create the canvas
+    // -----------------
 
 
-// Define my frame constructor
-MyFrame::MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos, const wxSize& size):
-wxFrame(frame, -1, title, pos, size)
-{
-    canvas = (MyCanvas *) NULL;
+    m_canvas = new MyCanvas(this, wxPoint(0, 0), wxSize(100, 100),
+                            wxRETAINED|wxHSCROLL|wxVSCROLL);
+
+    // Give it scrollbars: the virtual canvas is 20 * 50 = 1000 pixels in each direction
+    m_canvas->SetScrollbars(20, 20, 50, 50);
 }
 
 void MyFrame::OnExit(wxCommandEvent& WXUNUSED(event))
 {
 }
 
 void MyFrame::OnExit(wxCommandEvent& WXUNUSED(event))
 {
-    Close(TRUE);
+    Close(true /*force closing*/);
 }
 
 void MyFrame::OnPrint(wxCommandEvent& WXUNUSED(event))
 {
     wxPrintDialogData printDialogData(* g_printData);
 
 }
 
 void MyFrame::OnPrint(wxCommandEvent& WXUNUSED(event))
 {
     wxPrintDialogData printDialogData(* g_printData);
 
-    wxPrinter printer(& printDialogData);
-    MyPrintout printout("My printout");
-    if (!printer.Print(this, &printout, TRUE))
-        wxMessageBox("There was a problem printing.\nPerhaps your current printer is not set correctly?", "Printing", wxOK);
+    wxPrinter printer(&printDialogData);
+    MyPrintout printout(this, wxT("My printout"));
+    if (!printer.Print(this, &printout, true /*prompt*/))
+    {
+        if (wxPrinter::GetLastError() == wxPRINTER_ERROR)
+        {
+            wxLogError(wxT("There was a problem printing. Perhaps your current printer is not set correctly?"));
+        }
+        else
+        {
+            wxLogMessage(wxT("You canceled printing"));
+        }
+    }
     else
     {
         (*g_printData) = printer.GetPrintDialogData().GetPrintData();
     else
     {
         (*g_printData) = printer.GetPrintDialogData().GetPrintData();
@@ -201,48 +398,39 @@ void MyFrame::OnPrintPreview(wxCommandEvent& WXUNUSED(event))
 {
     // Pass two printout objects: for preview, and possible printing.
     wxPrintDialogData printDialogData(* g_printData);
 {
     // Pass two printout objects: for preview, and possible printing.
     wxPrintDialogData printDialogData(* g_printData);
-    wxPrintPreview *preview = new wxPrintPreview(new MyPrintout, new MyPrintout, & printDialogData);
-    if (!preview->Ok())
+    wxPrintPreview *preview =
+        new wxPrintPreview(new MyPrintout(this), new MyPrintout(this), &printDialogData);
+    if (!preview->IsOk())
     {
         delete preview;
     {
         delete preview;
-        wxMessageBox("There was a problem previewing.\nPerhaps your current printer is not set correctly?", "Previewing", wxOK);
+        wxLogError(wxT("There was a problem previewing.\nPerhaps your current printer is not set correctly?"));
         return;
     }
         return;
     }
-    
-    wxPreviewFrame *frame = new wxPreviewFrame(preview, this, "Demo Print Preview", wxPoint(100, 100), wxSize(600, 650));
-    frame->Centre(wxBOTH);
-    frame->Initialize();
-    frame->Show(TRUE);
-}
-
-void MyFrame::OnPrintSetup(wxCommandEvent& WXUNUSED(event))
-{
-    wxPrintDialogData printDialogData(* g_printData);
-    wxPrintDialog printerDialog(this, & printDialogData);
-    
-    printerDialog.GetPrintDialogData().SetSetupDialog(TRUE);
-    printerDialog.ShowModal();
 
 
-    (*g_printData) = printerDialog.GetPrintDialogData().GetPrintData();
+    wxPreviewFrame *frame =
+        new wxPreviewFrame(preview, this, wxT("Demo Print Preview"), wxPoint(100, 100), wxSize(600, 650));
+    frame->Centre(wxBOTH);
+    frame->InitializeWithModality(m_previewModality);
+    frame->Show();
 }
 
 void MyFrame::OnPageSetup(wxCommandEvent& WXUNUSED(event))
 {
 }
 
 void MyFrame::OnPageSetup(wxCommandEvent& WXUNUSED(event))
 {
-    (*g_pageSetupData) = * g_printData;
+    (*g_pageSetupData) = *g_printData;
 
     wxPageSetupDialog pageSetupDialog(this, g_pageSetupData);
     pageSetupDialog.ShowModal();
 
     wxPageSetupDialog pageSetupDialog(this, g_pageSetupData);
     pageSetupDialog.ShowModal();
-    
-    (*g_printData) = pageSetupDialog.GetPageSetupData().GetPrintData();
-    (*g_pageSetupData) = pageSetupDialog.GetPageSetupData();
+
+    (*g_printData) = pageSetupDialog.GetPageSetupDialogData().GetPrintData();
+    (*g_pageSetupData) = pageSetupDialog.GetPageSetupDialogData();
 }
 
 #if defined(__WXMSW__) && wxTEST_POSTSCRIPT_IN_MSW
 void MyFrame::OnPrintPS(wxCommandEvent& WXUNUSED(event))
 {
     wxPostScriptPrinter printer(g_printData);
 }
 
 #if defined(__WXMSW__) && wxTEST_POSTSCRIPT_IN_MSW
 void MyFrame::OnPrintPS(wxCommandEvent& WXUNUSED(event))
 {
     wxPostScriptPrinter printer(g_printData);
-    MyPrintout printout("My printout");
-    printer.Print(this, &printout, TRUE);
+    MyPrintout printout(wxT("My printout"));
+    printer.Print(this, &printout, true/*prompt*/);
 
     (*g_printData) = printer.GetPrintData();
 }
 
     (*g_printData) = printer.GetPrintData();
 }
@@ -251,22 +439,12 @@ void MyFrame::OnPrintPreviewPS(wxCommandEvent& WXUNUSED(event))
 {
     // Pass two printout objects: for preview, and possible printing.
     wxPrintDialogData printDialogData(* g_printData);
 {
     // Pass two printout objects: for preview, and possible printing.
     wxPrintDialogData printDialogData(* g_printData);
-    wxPrintPreview *preview = new wxPrintPreview(new MyPrintout, new MyPrintout, & printDialogData);
-    wxPreviewFrame *frame = new wxPreviewFrame(preview, this, "Demo Print Preview", wxPoint(100, 100), wxSize(600, 650));
+    wxPrintPreview *preview = new wxPrintPreview(new MyPrintout, new MyPrintout, &printDialogData);
+    wxPreviewFrame *frame =
+        new wxPreviewFrame(preview, this, wxT("Demo Print Preview"), wxPoint(100, 100), wxSize(600, 650));
     frame->Centre(wxBOTH);
     frame->Initialize();
     frame->Centre(wxBOTH);
     frame->Initialize();
-    frame->Show(TRUE);
-}
-
-void MyFrame::OnPrintSetupPS(wxCommandEvent& WXUNUSED(event))
-{
-    wxPrintDialogData printDialogData(* g_printData);
-    wxGenericPrintDialog printerDialog(this, & printDialogData);
-
-    printerDialog.GetPrintDialogData().SetSetupDialog(TRUE);
-    printerDialog.ShowModal();
-
-    (*g_printData) = printerDialog.GetPrintDialogData().GetPrintData();
+    frame->Show();
 }
 
 void MyFrame::OnPageSetupPS(wxCommandEvent& WXUNUSED(event))
 }
 
 void MyFrame::OnPageSetupPS(wxCommandEvent& WXUNUSED(event))
@@ -276,72 +454,74 @@ void MyFrame::OnPageSetupPS(wxCommandEvent& WXUNUSED(event))
     wxGenericPageSetupDialog pageSetupDialog(this, g_pageSetupData);
     pageSetupDialog.ShowModal();
 
     wxGenericPageSetupDialog pageSetupDialog(this, g_pageSetupData);
     pageSetupDialog.ShowModal();
 
-    (*g_printData) = pageSetupDialog.GetPageSetupData().GetPrintData();
-    (*g_pageSetupData) = pageSetupDialog.GetPageSetupData();
+    (*g_printData) = pageSetupDialog.GetPageSetupDialogData().GetPrintData();
+    (*g_pageSetupData) = pageSetupDialog.GetPageSetupDialogData();
 }
 #endif
 
 }
 #endif
 
+#ifdef __WXMAC__
+void MyFrame::OnPageMargins(wxCommandEvent& WXUNUSED(event))
+{
+    (*g_pageSetupData) = *g_printData;
+
+    wxMacPageMarginsDialog pageMarginsDialog(this, g_pageSetupData);
+    pageMarginsDialog.ShowModal();
+
+    (*g_printData) = pageMarginsDialog.GetPageSetupDialogData().GetPrintData();
+    (*g_pageSetupData) = pageMarginsDialog.GetPageSetupDialogData();
+}
+#endif
 
 void MyFrame::OnPrintAbout(wxCommandEvent& WXUNUSED(event))
 {
 
 void MyFrame::OnPrintAbout(wxCommandEvent& WXUNUSED(event))
 {
-    (void)wxMessageBox("wxWindows printing demo\nAuthor: Julian Smart julian.smart@ukonline.co.uk",
-        "About wxWindows printing demo", wxOK|wxCENTRE);
+    (void)wxMessageBox(wxT("wxWidgets printing demo\nAuthor: Julian Smart"),
+                       wxT("About wxWidgets printing demo"), wxOK|wxCENTRE);
 }
 
 }
 
-void MyFrame::Draw(wxDC& dc)
+void MyFrame::OnAngleUp(wxCommandEvent& WXUNUSED(event))
 {
 {
-    dc.SetFont(* wxGetApp().m_testFont);
-    
-    dc.SetBackgroundMode(wxTRANSPARENT);
-    
-    dc.SetBrush(* wxCYAN_BRUSH);
-    dc.SetPen(* wxRED_PEN);
-    
-    dc.DrawRectangle(0, 30, 200, 100);
-    dc.DrawText("Rectangle 200 by 100", 40, 40);
-    
-    dc.DrawEllipse(50, 140, 100, 50);
-    
-    dc.DrawText("Test message: this is in 10 point text", 10, 180);
-    
-    dc.SetPen(* wxBLACK_PEN);
-    dc.DrawLine(0, 0, 200, 200);
-    dc.DrawLine(200, 0, 0, 200);
-    
-    wxIcon my_icon = wxICON(mondrian) ;
-    
-    dc.DrawIcon( my_icon, 100, 100);
+    wxGetApp().IncrementAngle();
+    m_canvas->Refresh();
 }
 
 }
 
-void MyFrame::OnSize(wxSizeEvent& event )
+void MyFrame::OnAngleDown(wxCommandEvent& WXUNUSED(event))
 {
 {
-    wxFrame::OnSize(event);
+    wxGetApp().DecrementAngle();
+    m_canvas->Refresh();
 }
 
 }
 
-BEGIN_EVENT_TABLE(MyCanvas, wxScrolledWindow)
-EVT_MOUSE_EVENTS(MyCanvas::OnEvent)
-END_EVENT_TABLE()
-
-// Define a constructor for my canvas
-MyCanvas::MyCanvas(wxFrame *frame, const wxPoint& pos, const wxSize& size, long style):
-    wxScrolledWindow(frame, -1, pos, size, style)
+void MyFrame::OnPreviewFrameModalityKind(wxCommandEvent& event)
 {
 {
-    SetBackgroundColour(* wxWHITE);
+    m_previewModality = static_cast<wxPreviewFrameModalityKind>(
+                            wxPreviewFrame_AppModal +
+                                (event.GetId() - WXPRINT_FRAME_MODAL_APP));
 }
 
 }
 
-MyCanvas::~MyCanvas(void)
+// ----------------------------------------------------------------------------
+// MyCanvas
+// ----------------------------------------------------------------------------
+
+BEGIN_EVENT_TABLE(MyCanvas, wxScrolledWindow)
+  //  EVT_PAINT(MyCanvas::OnPaint)
+END_EVENT_TABLE()
+
+MyCanvas::MyCanvas(wxFrame *frame, const wxPoint&pos, const wxSize&size, long style)
+    : wxScrolledWindow(frame, wxID_ANY, pos, size, style)
 {
 {
+    SetBackgroundColour(*wxWHITE);
 }
 
 }
 
-// Define the repainting behaviour
+//void MyCanvas::OnPaint(wxPaintEvent& WXUNUSED(evt))
 void MyCanvas::OnDraw(wxDC& dc)
 {
 void MyCanvas::OnDraw(wxDC& dc)
 {
-    frame->Draw(dc);
+    //wxPaintDC dc(this);
+    wxGetApp().Draw(dc);
 }
 
 }
 
-void MyCanvas::OnEvent(wxMouseEvent& WXUNUSED(event))
-{
-}
+
+// ----------------------------------------------------------------------------
+// MyPrintout
+// ----------------------------------------------------------------------------
 
 bool MyPrintout::OnPrintPage(int page)
 {
 
 bool MyPrintout::OnPrintPage(int page)
 {
@@ -349,29 +529,28 @@ bool MyPrintout::OnPrintPage(int page)
     if (dc)
     {
         if (page == 1)
     if (dc)
     {
         if (page == 1)
-            DrawPageOne(dc);
+            DrawPageOne();
         else if (page == 2)
         else if (page == 2)
-            DrawPageTwo(dc);
-        
-        dc->SetDeviceOrigin(0, 0);
-        dc->SetUserScale(1.0, 1.0);
-        
-        char buf[200];
-        sprintf(buf, "PAGE %d", page);
-        dc->DrawText(buf, 10, 10);
-        
-        return TRUE;
+            DrawPageTwo();
+
+        // Draw page numbers at top left corner of printable area, sized so that
+        // screen size of text matches paper size.
+        MapScreenSizeToPage();
+
+        dc->DrawText(wxString::Format(wxT("PAGE %d"), page), 0, 0);
+
+        return true;
     }
     else
     }
     else
-        return FALSE;
+        return false;
 }
 
 bool MyPrintout::OnBeginDocument(int startPage, int endPage)
 {
     if (!wxPrintout::OnBeginDocument(startPage, endPage))
 }
 
 bool MyPrintout::OnBeginDocument(int startPage, int endPage)
 {
     if (!wxPrintout::OnBeginDocument(startPage, endPage))
-        return FALSE;
-    
-    return TRUE;
+        return false;
+
+    return true;
 }
 
 void MyPrintout::GetPageInfo(int *minPage, int *maxPage, int *selPageFrom, int *selPageTo)
 }
 
 void MyPrintout::GetPageInfo(int *minPage, int *maxPage, int *selPageFrom, int *selPageTo)
@@ -387,154 +566,216 @@ bool MyPrintout::HasPage(int pageNum)
     return (pageNum == 1 || pageNum == 2);
 }
 
     return (pageNum == 1 || pageNum == 2);
 }
 
-void MyPrintout::DrawPageOne(wxDC *dc)
+void MyPrintout::DrawPageOne()
 {
 {
-/* You might use THIS code if you were scaling
-* graphics of known size to fit on the page.
-    */
-    int w, h;
-    
-    // We know the graphic is 200x200. If we didn't know this,
-    // we'd need to calculate it.
-    float maxX = 200;
-    float maxY = 200;
-    
-    // Let's have at least 50 device units margin
-    float marginX = 50;
-    float marginY = 50;
-    
-    // Add the margin to the graphic size
-    maxX += (2*marginX);
-    maxY += (2*marginY);
-    
-    // Get the size of the DC in pixels
-    dc->GetSize(&w, &h);
-    
-    // Calculate a suitable scaling factor
-    float scaleX=(float)(w/maxX);
-    float scaleY=(float)(h/maxY);
-    
-    // Use x or y scaling factor, whichever fits on the DC
-    float actualScale = wxMin(scaleX,scaleY);
-    
-    // Calculate the position on the DC for centring the graphic
-    float posX = (float)((w - (200*actualScale))/2.0);
-    float posY = (float)((h - (200*actualScale))/2.0);
-    
-    // Set the scale and origin
-    dc->SetUserScale(actualScale, actualScale);
-    dc->SetDeviceOrigin( (long)posX, (long)posY );
-    
-    frame->Draw(*dc);
+    // You might use THIS code if you were scaling graphics of known size to fit
+    // on the page. The commented-out code illustrates different ways of scaling
+    // the graphics.
+
+    // We know the graphic is 230x350. If we didn't know this, we'd need to
+    // calculate it.
+    wxCoord maxX = 230;
+    wxCoord maxY = 350;
+
+    // This sets the user scale and origin of the DC so that the image fits
+    // within the paper rectangle (but the edges could be cut off by printers
+    // that can't print to the edges of the paper -- which is most of them. Use
+    // this if your image already has its own margins.
+//    FitThisSizeToPaper(wxSize(maxX, maxY));
+//    wxRect fitRect = GetLogicalPaperRect();
+
+    // This sets the user scale and origin of the DC so that the image fits
+    // within the page rectangle, which is the printable area on Mac and MSW
+    // and is the entire page on other platforms.
+//    FitThisSizeToPage(wxSize(maxX, maxY));
+//    wxRect fitRect = GetLogicalPageRect();
+
+    // This sets the user scale and origin of the DC so that the image fits
+    // within the page margins as specified by g_PageSetupData, which you can
+    // change (on some platforms, at least) in the Page Setup dialog. Note that
+    // on Mac, the native Page Setup dialog doesn't let you change the margins
+    // of a wxPageSetupDialogData object, so you'll have to write your own dialog or
+    // use the Mac-only wxMacPageMarginsDialog, as we do in this program.
+    FitThisSizeToPageMargins(wxSize(maxX, maxY), *g_pageSetupData);
+    wxRect fitRect = GetLogicalPageMarginsRect(*g_pageSetupData);
+
+    // This sets the user scale and origin of the DC so that the image appears
+    // on the paper at the same size that it appears on screen (i.e., 10-point
+    // type on screen is 10-point on the printed page) and is positioned in the
+    // top left corner of the page rectangle (just as the screen image appears
+    // in the top left corner of the window).
+//    MapScreenSizeToPage();
+//    wxRect fitRect = GetLogicalPageRect();
+
+    // You could also map the screen image to the entire paper at the same size
+    // as it appears on screen.
+//    MapScreenSizeToPaper();
+//    wxRect fitRect = GetLogicalPaperRect();
+
+    // You might also wish to do you own scaling in order to draw objects at
+    // full native device resolution. In this case, you should do the following.
+    // Note that you can use the GetLogicalXXXRect() commands to obtain the
+    // appropriate rect to scale to.
+//    MapScreenSizeToDevice();
+//    wxRect fitRect = GetLogicalPageRect();
+
+    // Each of the preceding Fit or Map routines positions the origin so that
+    // the drawn image is positioned at the top left corner of the reference
+    // rectangle. You can easily center or right- or bottom-justify the image as
+    // follows.
+
+    // This offsets the image so that it is centered within the reference
+    // rectangle defined above.
+    wxCoord xoff = (fitRect.width - maxX) / 2;
+    wxCoord yoff = (fitRect.height - maxY) / 2;
+    OffsetLogicalOrigin(xoff, yoff);
+
+    // This offsets the image so that it is positioned at the bottom right of
+    // the reference rectangle defined above.
+//    wxCoord xoff = (fitRect.width - maxX);
+//    wxCoord yoff = (fitRect.height - maxY);
+//    OffsetLogicalOrigin(xoff, yoff);
+
+    wxGetApp().Draw(*GetDC());
 }
 
 }
 
-void MyPrintout::DrawPageTwo(wxDC *dc)
+void MyPrintout::DrawPageTwo()
 {
 {
-/* You might use THIS code to set the printer DC to ROUGHLY reflect
-* the screen text size. This page also draws lines of actual length 5cm
-* on the page.
-    */
+    // You might use THIS code to set the printer DC to ROUGHLY reflect
+    // the screen text size. This page also draws lines of actual length
+    // 5cm on the page.
+
+    // Compare this to DrawPageOne(), which uses the really convenient routines
+    // from wxPrintout to fit the screen image onto the printed page. This page
+    // illustrates how to do all the scaling calculations yourself, if you're so
+    // inclined.
+
+    wxDC *dc = GetDC();
+
     // Get the logical pixels per inch of screen and printer
     int ppiScreenX, ppiScreenY;
     GetPPIScreen(&ppiScreenX, &ppiScreenY);
     int ppiPrinterX, ppiPrinterY;
     GetPPIPrinter(&ppiPrinterX, &ppiPrinterY);
     // Get the logical pixels per inch of screen and printer
     int ppiScreenX, ppiScreenY;
     GetPPIScreen(&ppiScreenX, &ppiScreenY);
     int ppiPrinterX, ppiPrinterY;
     GetPPIPrinter(&ppiPrinterX, &ppiPrinterY);
-    
-    // This scales the DC so that the printout roughly represents the
-    // the screen scaling. The text point size _should_ be the right size
-    // but in fact is too small for some reason. This is a detail that will
-    // need to be addressed at some point but can be fudged for the
-    // moment.
+
+    // This scales the DC so that the printout roughly represents the screen
+    // scaling. The text point size _should_ be the right size but in fact is
+    // too small for some reason. This is a detail that will need to be
+    // addressed at some point but can be fudged for the moment.
     float scale = (float)((float)ppiPrinterX/(float)ppiScreenX);
     float scale = (float)((float)ppiPrinterX/(float)ppiScreenX);
-    
-    // Now we have to check in case our real page size is reduced
-    // (e.g. because we're drawing to a print preview memory DC)
+
+    // Now we have to check in case our real page size is reduced (e.g. because
+    // we're drawing to a print preview memory DC)
     int pageWidth, pageHeight;
     int w, h;
     dc->GetSize(&w, &h);
     GetPageSizePixels(&pageWidth, &pageHeight);
     int pageWidth, pageHeight;
     int w, h;
     dc->GetSize(&w, &h);
     GetPageSizePixels(&pageWidth, &pageHeight);
-    
-    // If printer pageWidth == current DC width, then this doesn't
-    // change. But w might be the preview bitmap width, so scale down.
+
+    // If printer pageWidth == current DC width, then this doesn't change. But w
+    // might be the preview bitmap width, so scale down.
     float overallScale = scale * (float)(w/(float)pageWidth);
     dc->SetUserScale(overallScale, overallScale);
     float overallScale = scale * (float)(w/(float)pageWidth);
     dc->SetUserScale(overallScale, overallScale);
-    
-    // Calculate conversion factor for converting millimetres into
-    // logical units.
-    // There are approx. 25.1 mm to the inch. There are ppi
-    // device units to the inch. Therefore 1 mm corresponds to
-    // ppi/25.1 device units. We also divide by the
-    // screen-to-printer scaling factor, because we need to
+
+    // Calculate conversion factor for converting millimetres into logical
+    // units. There are approx. 25.4 mm to the inch. There are ppi device units
+    // to the inch. Therefore 1 mm corresponds to ppi/25.4 device units. We also
+    // divide by the screen-to-printer scaling factor, because we need to
     // unscale to pass logical units to DrawLine.
     // unscale to pass logical units to DrawLine.
-    
+
     // Draw 50 mm by 50 mm L shape
     // Draw 50 mm by 50 mm L shape
-    float logUnitsFactor = (float)(ppiPrinterX/(scale*25.1));
+    float logUnitsFactor = (float)(ppiPrinterX/(scale*25.4));
     float logUnits = (float)(50*logUnitsFactor);
     dc->SetPen(* wxBLACK_PEN);
     dc->DrawLine(50, 250, (long)(50.0 + logUnits), 250);
     dc->DrawLine(50, 250, 50, (long)(250.0 + logUnits));
     float logUnits = (float)(50*logUnitsFactor);
     dc->SetPen(* wxBLACK_PEN);
     dc->DrawLine(50, 250, (long)(50.0 + logUnits), 250);
     dc->DrawLine(50, 250, 50, (long)(250.0 + logUnits));
-    
-    dc->SetFont(* wxGetApp().m_testFont);
+
     dc->SetBackgroundMode(wxTRANSPARENT);
     dc->SetBackgroundMode(wxTRANSPARENT);
-    
-    dc->DrawText("Some test text", 200, 200 );
-    
+    dc->SetBrush(*wxTRANSPARENT_BRUSH);
+
+    { // GetTextExtent demo:
+        wxString words[7] = { wxT("This "), wxT("is "), wxT("GetTextExtent "),
+                             wxT("testing "), wxT("string. "), wxT("Enjoy "), wxT("it!") };
+        wxCoord w, h;
+        long x = 200, y= 250;
+        wxFont fnt(15, wxSWISS, wxNORMAL, wxNORMAL);
+
+        dc->SetFont(fnt);
+
+        for (int i = 0; i < 7; i++)
+        {
+            wxString word = words[i];
+            word.Remove( word.Len()-1, 1 );
+            dc->GetTextExtent(word, &w, &h);
+            dc->DrawRectangle(x, y, w, h);
+            dc->GetTextExtent(words[i], &w, &h);
+            dc->DrawText(words[i], x, y);
+            x += w;
+        }
+
+    }
+
+    dc->SetFont(wxGetApp().GetTestFont());
+
+    dc->DrawText(wxT("Some test text"), 200, 300 );
+
     // TESTING
     // TESTING
-    
+
     int leftMargin = 20;
     int rightMargin = 20;
     int topMargin = 20;
     int bottomMargin = 20;
     int leftMargin = 20;
     int rightMargin = 20;
     int topMargin = 20;
     int bottomMargin = 20;
-    
+
     int pageWidthMM, pageHeightMM;
     GetPageSizeMM(&pageWidthMM, &pageHeightMM);
     int pageWidthMM, pageHeightMM;
     GetPageSizeMM(&pageWidthMM, &pageHeightMM);
-    
+
     float leftMarginLogical = (float)(logUnitsFactor*leftMargin);
     float topMarginLogical = (float)(logUnitsFactor*topMargin);
     float bottomMarginLogical = (float)(logUnitsFactor*(pageHeightMM - bottomMargin));
     float rightMarginLogical = (float)(logUnitsFactor*(pageWidthMM - rightMargin));
     float leftMarginLogical = (float)(logUnitsFactor*leftMargin);
     float topMarginLogical = (float)(logUnitsFactor*topMargin);
     float bottomMarginLogical = (float)(logUnitsFactor*(pageHeightMM - bottomMargin));
     float rightMarginLogical = (float)(logUnitsFactor*(pageWidthMM - rightMargin));
-    
+
     dc->SetPen(* wxRED_PEN);
     dc->SetPen(* wxRED_PEN);
-    dc->DrawLine( (long)leftMarginLogical, (long)topMarginLogical, 
+    dc->DrawLine( (long)leftMarginLogical, (long)topMarginLogical,
         (long)rightMarginLogical, (long)topMarginLogical);
         (long)rightMarginLogical, (long)topMarginLogical);
-    dc->DrawLine( (long)leftMarginLogical, (long)bottomMarginLogical, 
+    dc->DrawLine( (long)leftMarginLogical, (long)bottomMarginLogical,
         (long)rightMarginLogical,  (long)bottomMarginLogical);
         (long)rightMarginLogical,  (long)bottomMarginLogical);
-    
-    WritePageHeader(this, dc, "A header", logUnitsFactor);
+
+    WritePageHeader(this, dc, wxT("A header"), logUnitsFactor);
 }
 
 // Writes a header on a page. Margin units are in millimetres.
 }
 
 // Writes a header on a page. Margin units are in millimetres.
-bool WritePageHeader(wxPrintout *printout, wxDC *dc, char *text, float mmToLogical)
+bool MyPrintout::WritePageHeader(wxPrintout *printout, wxDC *dc, const wxString&text, float mmToLogical)
 {
 {
-/*
-static wxFont *headerFont = (wxFont *) NULL;
-if (!headerFont)
-{
-headerFont = wxTheFontList->FindOrCreateFont(16, wxSWISS, wxNORMAL, wxBOLD);
-}
-dc->SetFont(headerFont);
-    */
-    
+#if 0
+    static wxFont *headerFont = (wxFont *) NULL;
+    if (!headerFont)
+    {
+        headerFont = wxTheFontList->FindOrCreateFont(16, wxSWISS, wxNORMAL, wxBOLD);
+    }
+    dc->SetFont(headerFont);
+#endif
+
     int pageWidthMM, pageHeightMM;
     int pageWidthMM, pageHeightMM;
-    
+
     printout->GetPageSizeMM(&pageWidthMM, &pageHeightMM);
     printout->GetPageSizeMM(&pageWidthMM, &pageHeightMM);
-    
+    wxUnusedVar(pageHeightMM);
+
     int leftMargin = 10;
     int topMargin = 10;
     int rightMargin = 10;
     int leftMargin = 10;
     int topMargin = 10;
     int rightMargin = 10;
-    
+
     float leftMarginLogical = (float)(mmToLogical*leftMargin);
     float topMarginLogical = (float)(mmToLogical*topMargin);
     float rightMarginLogical = (float)(mmToLogical*(pageWidthMM - rightMargin));
     float leftMarginLogical = (float)(mmToLogical*leftMargin);
     float topMarginLogical = (float)(mmToLogical*topMargin);
     float rightMarginLogical = (float)(mmToLogical*(pageWidthMM - rightMargin));
-    
-    long xExtent, yExtent;
+
+    wxCoord xExtent, yExtent;
     dc->GetTextExtent(text, &xExtent, &yExtent);
     dc->GetTextExtent(text, &xExtent, &yExtent);
+
     float xPos = (float)(((((pageWidthMM - leftMargin - rightMargin)/2.0)+leftMargin)*mmToLogical) - (xExtent/2.0));
     dc->DrawText(text, (long)xPos, (long)topMarginLogical);
     float xPos = (float)(((((pageWidthMM - leftMargin - rightMargin)/2.0)+leftMargin)*mmToLogical) - (xExtent/2.0));
     dc->DrawText(text, (long)xPos, (long)topMarginLogical);
-    
+
     dc->SetPen(* wxBLACK_PEN);
     dc->SetPen(* wxBLACK_PEN);
-    dc->DrawLine( (long)leftMarginLogical, (long)(topMarginLogical+yExtent), 
-        (long)rightMarginLogical, (long)topMarginLogical+yExtent );
-    
-    return TRUE;
+    dc->DrawLine( (long)leftMarginLogical, (long)(topMarginLogical+yExtent),
+                  (long)rightMarginLogical, (long)topMarginLogical+yExtent );
+
+    return true;
 }
 }