Add missing WXK constants for the control keys

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

diff --git a/src/os2/notebook.cpp b/src/os2/notebook.cpp
index 5876c5b5a76868094bd2b9686e6396e1d2f40c27..f1c52e8ed43da48455f6a198320c3e83f5212a3a 100644 (file)
--- a/src/os2/notebook.cpp
+++ b/src/os2/notebook.cpp
@@ -60,8 +60,6 @@ BEGIN_EVENT_TABLE(wxNotebook, wxBookCtrlBase)
     EVT_NAVIGATION_KEY(wxNotebook::OnNavigationKey)
 END_EVENT_TABLE()
 
     EVT_NAVIGATION_KEY(wxNotebook::OnNavigationKey)
 END_EVENT_TABLE()
 

-IMPLEMENT_DYNAMIC_CLASS(wxNotebook, wxBookCtrlBase)
-

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


@@ -75,8 +73,6 @@ IMPLEMENT_DYNAMIC_CLASS(wxNotebook, wxBookCtrlBase)
 //
 void wxNotebook::Init()
 {
 //
 void wxNotebook::Init()
 {

-    m_imageList  = NULL;
-    m_nSelection = -1;

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


@@ -210,14 +206,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)

     {
         wxBookCtrlEvent             vEvent( wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING
                                            ,m_windowId
                                           );
 
         vEvent.SetSelection(nPage);
     {
         wxBookCtrlEvent             vEvent( wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING
                                            ,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())
         {


@@ -235,7 +231,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
 


@@ -243,7 +239,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


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

-    m_nSelection = nPage;
+    m_selection = nPage;

     return nPage;
 }
 
     return nPage;
 }
 


@@ -328,7 +324,7 @@ bool wxNotebook::SetPageImage (
 , int                               nImage
 )
 {
 , int                               nImage
 )
 {

-    wxBitmap                        vBitmap = (wxBitmap)m_imageList->GetBitmap(nImage);
+    wxBitmap                        vBitmap = (wxBitmap)GetImageList()->GetBitmap(nImage);

 
     return (bool)::WinSendMsg( GetHWND()
                               ,BKM_SETTABBITMAP
 
     return (bool)::WinSendMsg( GetHWND()
                               ,BKM_SETTABBITMAP


@@ -404,7 +400,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
     {


@@ -413,19 +409,19 @@ 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
         {


@@ -437,16 +433,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 != wxNOT_FOUND)
         {
             //
         }
 
         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);
         }
     }


@@ -469,7 +465,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


@@ -569,12 +565,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)


@@ -649,19 +645,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 = wxNOT_FOUND;
-
-    if (bSelect)
-        nSelNew = nPage;
-    else if ( m_nSelection == -1 )
-        nSelNew = 0;
-
-    if (nSelNew != wxNOT_FOUND)
-        SetSelection(nSelNew);
+    DoSetSelectionAfterInsertion(nPage, bSelect);

 
     InvalidateBestSize();
 
 
     InvalidateBestSize();
 


@@ -725,7 +709,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;

     }
 
     //
     }
 
     //


@@ -746,8 +730,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
 


@@ -786,7 +770,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


@@ -794,7 +778,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))
                 {