use WX_ASSERT_STR/SIZET_EQUAL instead of CPPUNIT_ASSERT_EQUAL with casts

[wxWidgets.git] / src / common / nbkbase.cpp

diff --git a/src/common/nbkbase.cpp b/src/common/nbkbase.cpp
index c493e023803297ff4f13d6dbf21a39d1996e1c3e..ec391bb57febafa767cb241a8e00501c786189cd 100644 (file)
--- a/src/common/nbkbase.cpp
+++ b/src/common/nbkbase.cpp
@@ -17,10 +17,6 @@
 // headers
 // ----------------------------------------------------------------------------
 
-#ifdef __GNUG__
-    #pragma implementation "notebookbase.h"
-#endif
-
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
@@ -33,7 +29,6 @@
 #ifndef WX_PRECOMP
 #endif //WX_PRECOMP
 
-#include "wx/imaglist.h"
 #include "wx/notebook.h"
 
 // ============================================================================
@@ -41,112 +36,49 @@
 // ============================================================================
 
 // ----------------------------------------------------------------------------
-// constructors and destructors
+// geometry
 // ----------------------------------------------------------------------------
 
-void wxNotebookBase::Init()
+wxSize wxNotebookBase::CalcSizeFromPage(const wxSize& sizePage) const
 {
-    m_imageList = NULL;
-    m_ownsImageList = FALSE;
-}
+    // this is, of course, totally bogus -- but we must do something by
+    // default because not all ports implement this
+    wxSize sizeTotal = sizePage;
 
-wxNotebookBase::~wxNotebookBase()
-{
-    if ( m_ownsImageList )
+    if ( HasFlag(wxBK_LEFT) || HasFlag(wxBK_RIGHT) )
     {
-        // may be NULL, ok
-        delete m_imageList;
+        sizeTotal.x += 90;
+        sizeTotal.y += 10;
     }
-}
-
-// ----------------------------------------------------------------------------
-// image list
-// ----------------------------------------------------------------------------
-
-void wxNotebookBase::SetImageList(wxImageList* imageList)
-{
-    if ( m_ownsImageList )
+    else // tabs on top/bottom side
     {
-        // may be NULL, ok
-        delete m_imageList;
-
-        m_ownsImageList = FALSE;
-    }
-
-    m_imageList = imageList;
-}
-
-void wxNotebookBase::AssignImageList(wxImageList* imageList)
-{
-    SetImageList(imageList);
-    m_ownsImageList = TRUE;
-}
-
-// ----------------------------------------------------------------------------
-// geometry
-// ----------------------------------------------------------------------------
-
-wxSize wxNotebookBase::CalcSizeFromPage(const wxSize& sizePage)
-{
-    // this was just taken from wxNotebookSizer::CalcMin() and is, of
-    // course, totally bogus - just like the original code was
-    wxSize sizeTotal = sizePage;
-    if ( HasFlag(wxNB_LEFT) || HasFlag(wxNB_RIGHT) )
-        sizeTotal.x += 90;
-    else
+        sizeTotal.x += 10;
         sizeTotal.y += 40;
+    }
 
     return sizeTotal;
 }
 
 // ----------------------------------------------------------------------------
-// pages management
+// events
 // ----------------------------------------------------------------------------
 
-bool wxNotebookBase::DeletePage(int nPage)
-{
-    wxNotebookPage *page = DoRemovePage(nPage);
-    if ( !page )
-        return FALSE;
-
-    delete page;
-
-    return TRUE;
-}
-
-wxNotebookPage *wxNotebookBase::DoRemovePage(int nPage)
+bool wxNotebookBase::SendPageChangingEvent(int nPage)
 {
-    wxCHECK_MSG( nPage >= 0 && (size_t)nPage < m_pages.GetCount(), NULL,
-                 _T("invalid page index in wxNotebookBase::DoRemovePage()") );
-
-    wxNotebookPage *pageRemoved = m_pages[nPage];
-    m_pages.Remove(nPage);
-
-    return pageRemoved;
+    wxNotebookEvent event(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING, GetId());
+    event.SetSelection(nPage);
+    event.SetOldSelection(GetSelection());
+    event.SetEventObject(this);
+    return !GetEventHandler()->ProcessEvent(event) || event.IsAllowed();
 }
 
-int wxNotebookBase::GetNextPage(bool forward) const
+void wxNotebookBase::SendPageChangedEvent(int nPageOld, int nPageNew)
 {
-    int nPage;
-
-    int nMax = GetPageCount();
-    if ( nMax-- ) // decrement it to get the last valid index
-    {
-        int nSel = GetSelection();
-
-        // change selection wrapping if it becomes invalid
-        nPage = forward ? nSel == nMax ? 0
-                                       : nSel + 1
-                        : nSel == 0 ? nMax
-                                    : nSel - 1;
-    }
-    else // notebook is empty, no next page
-    {
-        nPage = -1;
-    }
-
-    return nPage;
+    wxNotebookEvent event(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED, GetId());
+    event.SetSelection(nPageNew == -1 ? GetSelection() : nPageNew);
+    event.SetOldSelection(nPageOld);
+    event.SetEventObject(this);
+    GetEventHandler()->ProcessEvent(event);
 }
 
 #endif // wxUSE_NOTEBOOK
-