fixing duplicate rti info

[wxWidgets.git] / src / os2 / notebook.cpp

diff --git a/src/os2/notebook.cpp b/src/os2/notebook.cpp
index eaf16c77fa082ee9937bd6753798aa900321fd67..f0a88c0c6ab57d8ea4ea7e8cf961f5af95a25e0a 100644 (file)
--- a/src/os2/notebook.cpp
+++ b/src/os2/notebook.cpp
@@ -53,18 +53,14 @@
 // event table
 // ----------------------------------------------------------------------------
 
 // event table
 // ----------------------------------------------------------------------------
 

-DEFINE_EVENT_TYPE(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED)
-DEFINE_EVENT_TYPE(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING)
-
-BEGIN_EVENT_TABLE(wxNotebook, wxControl)
+BEGIN_EVENT_TABLE(wxNotebook, wxBookCtrlBase)

     EVT_NOTEBOOK_PAGE_CHANGED(wxID_ANY, wxNotebook::OnSelChange)
     EVT_SIZE(wxNotebook::OnSize)
     EVT_SET_FOCUS(wxNotebook::OnSetFocus)
     EVT_NAVIGATION_KEY(wxNotebook::OnNavigationKey)
 END_EVENT_TABLE()
 
     EVT_NOTEBOOK_PAGE_CHANGED(wxID_ANY, wxNotebook::OnSelChange)
     EVT_SIZE(wxNotebook::OnSize)
     EVT_SET_FOCUS(wxNotebook::OnSetFocus)
     EVT_NAVIGATION_KEY(wxNotebook::OnNavigationKey)
 END_EVENT_TABLE()
 

-IMPLEMENT_DYNAMIC_CLASS(wxNotebook, wxControl)
-IMPLEMENT_DYNAMIC_CLASS(wxNotebookEvent, wxNotifyEvent)
+IMPLEMENT_DYNAMIC_CLASS(wxNotebook, wxBookCtrlBase)

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


@@ -80,7 +76,6 @@ IMPLEMENT_DYNAMIC_CLASS(wxNotebookEvent, wxNotifyEvent)
 void wxNotebook::Init()
 {
     m_imageList  = NULL;
 void wxNotebook::Init()
 {
     m_imageList  = NULL;

-    m_nSelection = -1;

     m_nTabSize   = 0;
 } // end of wxNotebook::Init
 
     m_nTabSize   = 0;
 } // end of wxNotebook::Init
 


@@ -214,14 +209,14 @@ int wxNotebook::SetSelection( size_t nPage )
 {
     wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("notebook page out of range") );
 
 {
     wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("notebook page out of range") );
 

-    if (nPage != (size_t)m_nSelection)
+    if (nPage != (size_t)m_selection)

     {
     {

-        wxNotebookEvent             vEvent( wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING
+        wxBookCtrlEvent             vEvent( wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING

                                            ,m_windowId
                                           );
 
         vEvent.SetSelection(nPage);
                                            ,m_windowId
                                           );
 
         vEvent.SetSelection(nPage);

-        vEvent.SetOldSelection(m_nSelection);
+        vEvent.SetOldSelection(m_selection);

         vEvent.SetEventObject(this);
         if (!HandleWindowEvent(vEvent) || vEvent.IsAllowed())
         {
         vEvent.SetEventObject(this);
         if (!HandleWindowEvent(vEvent) || vEvent.IsAllowed())
         {


@@ -239,7 +234,7 @@ int wxNotebook::SetSelection( size_t nPage )
                         );
         }
     }
                         );
         }
     }

-    m_nSelection = nPage;
+    m_selection = nPage;

     return nPage;
 } // end of wxNotebook::SetSelection
 
     return nPage;
 } // end of wxNotebook::SetSelection
 


@@ -247,7 +242,7 @@ int wxNotebook::ChangeSelection( size_t nPage )
 {
     wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("notebook page out of range") );
 
 {
     wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("notebook page out of range") );
 

-    if (nPage != (size_t)m_nSelection)
+    if (nPage != (size_t)m_selection)

     {
         ::WinSendMsg( GetHWND()
                 ,BKM_TURNTOPAGE
     {
         ::WinSendMsg( GetHWND()
                 ,BKM_TURNTOPAGE


@@ -255,7 +250,7 @@ int wxNotebook::ChangeSelection( size_t nPage )
                         ,(MPARAM)0
                     );
     }
                         ,(MPARAM)0
                     );
     }

-    m_nSelection = nPage;
+    m_selection = nPage;

     return nPage;
 }
 
     return nPage;
 }
 


@@ -408,7 +403,7 @@ wxNotebookPage* wxNotebook::DoRemovePage ( size_t nPage )
         //
         // No selection any more, the notebook becamse empty
         //
         //
         // No selection any more, the notebook becamse empty
         //

-        m_nSelection = -1;
+        m_selection = wxNOT_FOUND;

     }
     else // notebook still not empty
     {
     }
     else // notebook still not empty
     {


@@ -417,23 +412,23 @@ wxNotebookPage* wxNotebook::DoRemovePage ( size_t nPage )
         //
         int                         nSelNew;
 
         //
         int                         nSelNew;
 

-        if (m_nSelection == (int)GetPageCount())
+        if (m_selection == (int)GetPageCount())

         {
             //
             // Last page deleted, make the new last page the new selection
             //
         {
             //
             // Last page deleted, make the new last page the new selection
             //

-            nSelNew = m_nSelection - 1;
+            nSelNew = m_selection - 1;

         }
         }

-        else if (nPage <= (size_t)m_nSelection)
+        else if (nPage <= (size_t)m_selection)

         {
             //
             // We must show another page, even if it has the same index
             //
         {
             //
             // We must show another page, even if it has the same index
             //

-            nSelNew = m_nSelection;
+            nSelNew = m_selection;

         }
         else // nothing changes for the currently selected page
         {
         }
         else // nothing changes for the currently selected page
         {

-            nSelNew = -1;
+            nSelNew = wxNOT_FOUND;

 
             //
             // We still must refresh the current page: this needs to be done
 
             //
             // We still must refresh the current page: this needs to be done


@@ -441,16 +436,16 @@ wxNotebookPage* wxNotebook::DoRemovePage ( size_t nPage )
             // control (i.e. when there are too many pages) -- otherwise after
             // deleting a page nothing at all is shown
             //
             // control (i.e. when there are too many pages) -- otherwise after
             // deleting a page nothing at all is shown
             //

-            m_pages[m_nSelection]->Refresh();
+            m_pages[m_selection]->Refresh();

         }
 
         }
 

-        if (nSelNew != -1)
+        if (nSelNew != wxNOT_FOUND)

         {
             //
         {
             //

-            // m_nSelection must be always valid so reset it before calling
+            // m_selection must be always valid so reset it before calling

             // SetSelection()
             //
             // SetSelection()
             //

-            m_nSelection = -1;
+            m_selection = wxNOT_FOUND;

             SetSelection(nSelNew);
         }
     }
             SetSelection(nSelNew);
         }
     }


@@ -473,7 +468,7 @@ bool wxNotebook::DeleteAllPages()
                  ,(MPARAM)0
                  ,(MPARAM)BKA_ALL
                 );
                  ,(MPARAM)0
                  ,(MPARAM)BKA_ALL
                 );

-    m_nSelection = -1;
+    m_selection = wxNOT_FOUND;

 
     return true;
 } // end of wxNotebook::DeleteAllPages
 
     return true;
 } // end of wxNotebook::DeleteAllPages


@@ -573,12 +568,12 @@ bool wxNotebook::InsertPage ( size_t          nPage,
     // If the inserted page is before the selected one, we must update the
     // index of the selected page
     //
     // If the inserted page is before the selected one, we must update the
     // index of the selected page
     //

-    if (nPage <= (size_t)m_nSelection)
+    if (nPage <= (size_t)m_selection)

     {
         //
         // One extra page added
         //
     {
         //
         // One extra page added
         //

-        m_nSelection++;
+        m_selection++;

     }
 
     if (pPage)
     }
 
     if (pPage)


@@ -653,19 +648,7 @@ bool wxNotebook::InsertPage ( size_t          nPage,
         pPage->Show(false);
     }
 
         pPage->Show(false);
     }
 

-    //
-    // Some page should be selected: either this one or the first one if there is
-    // still no selection
-    //
-    int nSelNew = -1;
-
-    if (bSelect)
-        nSelNew = nPage;
-    else if ( m_nSelection == -1 )
-        nSelNew = 0;
-
-    if (nSelNew != -1)
-        SetSelection(nSelNew);
+    DoSetSelectionAfterInsertion(nPage, bSelect);

 
     InvalidateBestSize();
 
 
     InvalidateBestSize();
 


@@ -683,7 +666,7 @@ void wxNotebook::OnSize(
 } // end of wxNotebook::OnSize
 
 void wxNotebook::OnSelChange (
 } // end of wxNotebook::OnSize
 
 void wxNotebook::OnSelChange (

-  wxNotebookEvent&                  rEvent
+  wxBookCtrlEvent&                  rEvent

 )
 {
     //
 )
 {
     //


@@ -729,7 +712,7 @@ void wxNotebook::OnSelChange (
         wxNotebookPage*         pPage = m_pages[nSel];
 
         pPage->Show(true);
         wxNotebookPage*         pPage = m_pages[nSel];
 
         pPage->Show(true);

-        m_nSelection = nSel;
+        m_selection = nSel;

     }
 
     //
     }
 
     //


@@ -750,8 +733,8 @@ void wxNotebook::OnSetFocus (
     //
     // set focus to the currently selected page if any
     //
     //
     // set focus to the currently selected page if any
     //

-    if (m_nSelection != -1)
-        m_pages[m_nSelection]->SetFocus();
+    if (m_selection != wxNOT_FOUND)
+        m_pages[m_selection]->SetFocus();

     rEvent.Skip();
 } // end of wxNotebook::OnSetFocus
 
     rEvent.Skip();
 } // end of wxNotebook::OnSetFocus
 


@@ -790,7 +773,7 @@ void wxNotebook::OnNavigationKey (
             //
             // No, it doesn't come from child, case (b): forward to a page
             //
             //
             // No, it doesn't come from child, case (b): forward to a page
             //

-            if (m_nSelection != -1)
+            if (m_selection != wxNOT_FOUND)

             {
                 //
                 // So that the page knows that the event comes from it's parent
             {
                 //
                 // So that the page knows that the event comes from it's parent


@@ -798,7 +781,7 @@ void wxNotebook::OnNavigationKey (
                 //
                 rEvent.SetEventObject(this);
 
                 //
                 rEvent.SetEventObject(this);
 

-                wxWindow*           pPage = m_pages[m_nSelection];
+                wxWindow*           pPage = m_pages[m_selection];

 
                 if (!pPage->HandleWindowEvent(rEvent))
                 {
 
                 if (!pPage->HandleWindowEvent(rEvent))
                 {