Fix for probably rare but potential refcount leak.

[wxWidgets.git] / src / generic / fdrepdlg.cpp

diff --git a/src/generic/fdrepdlg.cpp b/src/generic/fdrepdlg.cpp
index d613e1e4c9bb4d41fcbf28fa14140d6d5b00fa9e..29e31d7b60d6c51dd5b083e0b60be626f49cbb28 100644 (file)
--- a/src/generic/fdrepdlg.cpp
+++ b/src/generic/fdrepdlg.cpp
@@ -5,15 +5,15 @@
 // Modified by:
 // Created:     05/25/01
 // RCS-ID:
 // Modified by:
 // Created:     05/25/01
 // RCS-ID:

-// Copyright:   (c) wxWindows team
-// Licence:     wxWindows license
+// Copyright:   (c) wxWidgets team
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================
 // declarations
 // ============================================================================
 
 /////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================
 // declarations
 // ============================================================================
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

     #pragma implementation "genericfdrepdlg.h"
 #endif
 
     #pragma implementation "genericfdrepdlg.h"
 #endif
 


@@ -49,12 +49,6 @@
 // constants
 // ----------------------------------------------------------------------------
 
 // constants
 // ----------------------------------------------------------------------------
 

-enum
-{
-    wxID_REPLACE = 5890,
-    wxID_REPLACE_ALL
-};
-

 // ============================================================================
 // implementation
 // ============================================================================
 // ============================================================================
 // implementation
 // ============================================================================


@@ -96,16 +90,16 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
                                         const wxString& title,
                                         int style)
 {
                                         const wxString& title,
                                         int style)
 {

-    if ( !wxDialog::Create(parent, -1, title,
+    if ( !wxDialog::Create(parent, wxID_ANY, title,

                            wxDefaultPosition, wxDefaultSize,
                            wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER | style) )
     {
                            wxDefaultPosition, wxDefaultSize,
                            wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER | style) )
     {

-        return FALSE;
+        return false;

     }
 
     SetData(data);
 
     }
 
     SetData(data);
 

-    wxCHECK_MSG( m_FindReplaceData, FALSE,
+    wxCHECK_MSG( m_FindReplaceData, false,

                  _T("can't create dialog without data") );
 
     wxBoxSizer *leftsizer = new wxBoxSizer( wxVERTICAL );
                  _T("can't create dialog without data") );
 
     wxBoxSizer *leftsizer = new wxBoxSizer( wxVERTICAL );


@@ -114,27 +108,27 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
     wxFlexGridSizer *sizer2Col = new wxFlexGridSizer(3);
     sizer2Col->AddGrowableCol(2);
 
     wxFlexGridSizer *sizer2Col = new wxFlexGridSizer(3);
     sizer2Col->AddGrowableCol(2);
 

-    sizer2Col->Add(new wxStaticText(this, -1, _("Search for:"),
-                                    wxDefaultPosition, wxSize(80, -1)),
+    sizer2Col->Add(new wxStaticText(this, wxID_ANY, _("Search for:"),
+                                    wxDefaultPosition, wxSize(80, wxDefaultCoord)),

                                     0,
                                     wxALIGN_CENTRE_VERTICAL | wxALIGN_RIGHT);
 
     sizer2Col->Add(10, 0);
 
                                     0,
                                     wxALIGN_CENTRE_VERTICAL | wxALIGN_RIGHT);
 
     sizer2Col->Add(10, 0);
 

-    m_textFind = new wxTextCtrl(this, -1, m_FindReplaceData->GetFindString());
+    m_textFind = new wxTextCtrl(this, wxID_ANY, m_FindReplaceData->GetFindString());

     sizer2Col->Add(m_textFind, 1, wxALIGN_CENTRE_VERTICAL | wxEXPAND);
 
     if ( style & wxFR_REPLACEDIALOG )
     {
     sizer2Col->Add(m_textFind, 1, wxALIGN_CENTRE_VERTICAL | wxEXPAND);
 
     if ( style & wxFR_REPLACEDIALOG )
     {

-        sizer2Col->Add(new wxStaticText(this, -1, _("Replace with:"),
-                                        wxDefaultPosition, wxSize(80, -1)),
+        sizer2Col->Add(new wxStaticText(this, wxID_ANY, _("Replace with:"),
+                                        wxDefaultPosition, wxSize(80, wxDefaultCoord)),

                                         0,
                                         wxALIGN_CENTRE_VERTICAL |
                                         wxALIGN_RIGHT | wxTOP, 5);
 
         sizer2Col->Add(10, 0);
 
                                         0,
                                         wxALIGN_CENTRE_VERTICAL |
                                         wxALIGN_RIGHT | wxTOP, 5);
 
         sizer2Col->Add(10, 0);
 

-        m_textRepl = new wxTextCtrl(this, -1,
+        m_textRepl = new wxTextCtrl(this, wxID_ANY,

                                     m_FindReplaceData->GetReplaceString());
         sizer2Col->Add(m_textRepl, 1,
                        wxALIGN_CENTRE_VERTICAL | wxEXPAND | wxTOP, 5);
                                     m_FindReplaceData->GetReplaceString());
         sizer2Col->Add(m_textRepl, 1,
                        wxALIGN_CENTRE_VERTICAL | wxEXPAND | wxTOP, 5);


@@ -146,16 +140,16 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
 
     wxBoxSizer *chksizer = new wxBoxSizer( wxVERTICAL);
 
 
     wxBoxSizer *chksizer = new wxBoxSizer( wxVERTICAL);
 

-    m_chkWord = new wxCheckBox(this, -1, _("Whole word"));
+    m_chkWord = new wxCheckBox(this, wxID_ANY, _("Whole word"));

     chksizer->Add(m_chkWord, 0, wxALL, 3);
 
     chksizer->Add(m_chkWord, 0, wxALL, 3);
 

-    m_chkCase = new wxCheckBox(this, -1, _("Match case"));
+    m_chkCase = new wxCheckBox(this, wxID_ANY, _("Match case"));

     chksizer->Add(m_chkCase, 0, wxALL, 3);
 
     optsizer->Add(chksizer, 0, wxALL, 10);
 
     static const wxString searchDirections[] = {_("Up"), _("Down")};
     chksizer->Add(m_chkCase, 0, wxALL, 3);
 
     optsizer->Add(chksizer, 0, wxALL, 10);
 
     static const wxString searchDirections[] = {_("Up"), _("Down")};

-    m_radioDir = new wxRadioBox(this, -1, _("Search direction"),
+    m_radioDir = new wxRadioBox(this, wxID_ANY, _("Search direction"),

                                 wxDefaultPosition, wxDefaultSize,
                                 WXSIZEOF(searchDirections), searchDirections);
 
                                 wxDefaultPosition, wxDefaultSize,
                                 WXSIZEOF(searchDirections), searchDirections);
 


@@ -165,9 +159,9 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
 
     wxBoxSizer *bttnsizer = new wxBoxSizer(wxVERTICAL);
 
 
     wxBoxSizer *bttnsizer = new wxBoxSizer(wxVERTICAL);
 

-    bttnsizer->Add(new wxButton(this, wxID_FIND, _("&Find")), 0, wxALL, 3);
+    bttnsizer->Add(new wxButton(this, wxID_FIND), 0, wxALL, 3);

 
 

-    bttnsizer->Add(new wxButton(this, wxID_CANCEL, _("&Cancel")), 0, wxALL, 3);
+    bttnsizer->Add(new wxButton(this, wxID_CANCEL), 0, wxALL, 3);

 
     if ( style & wxFR_REPLACEDIALOG )
     {
 
     if ( style & wxFR_REPLACEDIALOG )
     {


@@ -186,23 +180,23 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
     int flags = m_FindReplaceData->GetFlags();
 
     if ( flags & wxFR_MATCHCASE )
     int flags = m_FindReplaceData->GetFlags();
 
     if ( flags & wxFR_MATCHCASE )

-        m_chkCase->SetValue(TRUE);
+        m_chkCase->SetValue(true);

 
     if ( flags & wxFR_WHOLEWORD )
 
     if ( flags & wxFR_WHOLEWORD )

-        m_chkWord->SetValue(TRUE);
+        m_chkWord->SetValue(true);

 
     m_radioDir->SetSelection( flags & wxFR_DOWN );
 
     if ( style & wxFR_NOMATCHCASE )
 
     m_radioDir->SetSelection( flags & wxFR_DOWN );
 
     if ( style & wxFR_NOMATCHCASE )

-        m_chkCase->Enable(FALSE);
+        m_chkCase->Enable(false);

 
     if ( style & wxFR_NOWHOLEWORD )
 
     if ( style & wxFR_NOWHOLEWORD )

-        m_chkWord->Enable(FALSE);
+        m_chkWord->Enable(false);

 
     if ( style & wxFR_NOUPDOWN)
 
     if ( style & wxFR_NOUPDOWN)

-        m_radioDir->Enable(FALSE);
+        m_radioDir->Enable(false);

 
 

-    SetAutoLayout( TRUE );
+    SetAutoLayout( true );

     SetSizer( topsizer );
 
     topsizer->SetSizeHints( this );
     SetSizer( topsizer );
 
     topsizer->SetSizeHints( this );


@@ -212,7 +206,7 @@ bool wxGenericFindReplaceDialog::Create(wxWindow *parent,
 
     m_textFind->SetFocus();
 
 
     m_textFind->SetFocus();
 

-    return TRUE;
+    return true;

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -270,7 +264,7 @@ void wxGenericFindReplaceDialog::OnCancel(wxCommandEvent& WXUNUSED(event))
 {
     SendEvent(wxEVT_COMMAND_FIND_CLOSE);
 
 {
     SendEvent(wxEVT_COMMAND_FIND_CLOSE);
 

-    Show(FALSE);
+    Show(false);

 }
 
 void wxGenericFindReplaceDialog::OnUpdateFindUI(wxUpdateUIEvent &event)
 }
 
 void wxGenericFindReplaceDialog::OnUpdateFindUI(wxUpdateUIEvent &event)