Avoid double free in wxComboBox::SetClientObject.

[wxWidgets.git] / include / wx / generic / prntdlgg.h

diff --git a/include/wx/generic/prntdlgg.h b/include/wx/generic/prntdlgg.h
index da5f97f15c090a5b609a63a14508a2ec050253db..d0114adf29bc80b47db4792a6e9f169d1e606141 100644 (file)
--- a/include/wx/generic/prntdlgg.h
+++ b/include/wx/generic/prntdlgg.h
@@ -13,7 +13,7 @@
 #ifndef __PRINTDLGH_G_
 #define __PRINTDLGH_G_
 
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
     #pragma interface "prntdlgg.h"
 #endif
 
@@ -22,7 +22,7 @@
 #if wxUSE_PRINTING_ARCHITECTURE
 
 #include "wx/dialog.h"
-#include "wx/dialog.h"
+#include "wx/cmndata.h"
 
 #if wxUSE_POSTSCRIPT
     #include "wx/dcps.h"
@@ -31,11 +31,11 @@
 class WXDLLEXPORT wxTextCtrl;
 class WXDLLEXPORT wxButton;
 class WXDLLEXPORT wxCheckBox;
-class WXDLLEXPORT wxChoice;
+class WXDLLEXPORT wxComboBox;
 class WXDLLEXPORT wxStaticText;
 class WXDLLEXPORT wxRadioBox;
 class WXDLLEXPORT wxPrintSetupData;
-
+class WXDLLEXPORT wxPageSetupData;
 // ----------------------------------------------------------------------------
 // constants
 // ----------------------------------------------------------------------------
@@ -78,8 +78,6 @@ enum
 #if wxUSE_POSTSCRIPT
 class WXDLLEXPORT wxGenericPrintDialog : public wxDialog
 {
-    DECLARE_DYNAMIC_CLASS(wxGenericPrintDialog)
-
 public:
     wxGenericPrintDialog(wxWindow *parent,
                          wxPrintDialogData* data = (wxPrintDialogData*)NULL);
@@ -122,12 +120,11 @@ protected:
 
 private:
     DECLARE_EVENT_TABLE()
+    DECLARE_DYNAMIC_CLASS(wxGenericPrintDialog)
 };
 
 class WXDLLEXPORT wxGenericPrintSetupDialog : public wxDialog
 {
-    DECLARE_CLASS(wxGenericPrintSetupDialog)
-
 public:
     // There are no configuration options for the dialog, so we
     // just pass the wxPrintData object (no wxPrintSetupDialogData class needed)
@@ -140,29 +137,31 @@ public:
     virtual bool TransferDataFromWindow();
     virtual bool TransferDataToWindow();
 
-    wxChoice *CreatePaperTypeChoice(int* x, int* y);
+    wxComboBox *CreatePaperTypeChoice(int* x, int* y);
 
 public:
     wxRadioBox*         m_orientationRadioBox;
     wxTextCtrl*         m_printerCommandText;
     wxTextCtrl*         m_printerOptionsText;
     wxCheckBox*         m_colourCheckBox;
-    wxChoice*           m_paperTypeChoice;
+    wxComboBox*           m_paperTypeChoice;
 
 #if wxUSE_POSTSCRIPT
     wxPrintData         m_printData;
     wxPrintData&        GetPrintData() { return m_printData; }
 #endif // wxUSE_POSTSCRIPT
+
+private:
+    DECLARE_CLASS(wxGenericPrintSetupDialog)
 };
 #endif
     // wxUSE_POSTSCRIPT
 
 class WXDLLEXPORT wxGenericPageSetupDialog : public wxDialog
 {
-    DECLARE_CLASS(wxGenericPageSetupDialog)
-
 public:
-    wxGenericPageSetupDialog(wxWindow *parent, wxPageSetupData* data = (wxPageSetupData*) NULL);
+    wxGenericPageSetupDialog(wxWindow *parent = NULL,
+                             wxPageSetupData* data = NULL);
     virtual ~wxGenericPageSetupDialog();
 
     virtual bool TransferDataFromWindow();
@@ -170,7 +169,7 @@ public:
 
     void OnPrinter(wxCommandEvent& event);
 
-    wxChoice *CreatePaperTypeChoice(int* x, int* y);
+    wxComboBox *CreatePaperTypeChoice(int* x, int* y);
     wxPageSetupData& GetPageSetupData() { return m_pageData; }
 
 public:
@@ -180,7 +179,7 @@ public:
     wxTextCtrl*     m_marginTopText;
     wxTextCtrl*     m_marginRightText;
     wxTextCtrl*     m_marginBottomText;
-    wxChoice*       m_paperTypeChoice;
+    wxComboBox*       m_paperTypeChoice;
 
     static bool     m_pageSetupDialogCancelled;
 
@@ -188,6 +187,7 @@ public:
 
 private:
     DECLARE_EVENT_TABLE()
+    DECLARE_DYNAMIC_CLASS_NO_COPY(wxGenericPageSetupDialog)
 };
 
 #endif