From: David Elliott <dfe@tgwbd.org>
Date: Wed, 14 Apr 2004 16:03:53 +0000 (+0000)
Subject: Quick hack to rid the code of wxDIALOG_MODAL
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/2c326adae6bcbcc9510710116ae9eb4b5581192d?ds=inline

Quick hack to rid the code of wxDIALOG_MODAL


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@26773 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/include/wx/mac/carbon/dialog.h b/include/wx/mac/carbon/dialog.h
index e6834011c5..490cd0af5c 100644
--- a/include/wx/mac/carbon/dialog.h
+++ b/include/wx/mac/carbon/dialog.h
@@ -30,7 +30,7 @@ class WXDLLEXPORT wxDialog : public wxDialogBase
     DECLARE_DYNAMIC_CLASS(wxDialog)
 
 public:
-    wxDialog();
+    wxDialog() { Init(); }
 
     // Constructor with a modal flag, but no window id - the old convention
     wxDialog(wxWindow *parent,
@@ -39,9 +39,10 @@ public:
              long style = wxDEFAULT_DIALOG_STYLE,
              const wxString& name = wxDialogNameStr)
     {
-        long modalStyle = modal ? wxDIALOG_MODAL : wxDIALOG_MODELESS ;
+        Init();
+        m_isModalStyle = modal;
         Create(parent, -1, title, wxPoint(x, y), wxSize(width, height),
-               style | modalStyle, name);
+               style, name);
     }
 
     // Constructor with no modal flag - the new convention.
@@ -52,6 +53,7 @@ public:
              long style = wxDEFAULT_DIALOG_STYLE,
              const wxString& name = wxDialogNameStr)
     {
+        Init();
         Create(parent, id, title, pos, size, style, name);
     }
 
@@ -98,6 +100,8 @@ public:
     void DoShowModal();
 
 private:
+    void Init();
+    bool m_isModalStyle;
     DECLARE_EVENT_TABLE()
 };
 
diff --git a/src/mac/carbon/dialog.cpp b/src/mac/carbon/dialog.cpp
index 9881fac0bd..af14cbc258 100644
--- a/src/mac/carbon/dialog.cpp
+++ b/src/mac/carbon/dialog.cpp
@@ -44,9 +44,9 @@ END_EVENT_TABLE()
 
 #endif
 
-wxDialog::wxDialog()
+void wxDialog::Init()
 {
-    m_isShown = FALSE;
+    m_isModalStyle = false;
 }
 
 bool wxDialog::Create(wxWindow *parent, wxWindowID id,
@@ -71,7 +71,7 @@ void wxDialog::SetModal(bool flag)
 {
     if ( flag )
     {
-        m_windowStyle |= wxDIALOG_MODAL;
+        m_isModalStyle = true;
 
         wxModelessWindows.DeleteObject(this);
 #if TARGET_CARBON
@@ -80,7 +80,7 @@ void wxDialog::SetModal(bool flag)
     }
     else
     {
-        m_windowStyle &= ~wxDIALOG_MODAL;
+        m_isModalStyle = false;
 
         wxModelessWindows.Append(this);
     }
@@ -113,7 +113,7 @@ void wxDialog::OnCharHook(wxKeyEvent& event)
 
 bool wxDialog::IsModal() const
 {
-    return (GetWindowStyleFlag() & wxDIALOG_MODAL) != 0;
+    return m_isModalStyle;
 }