From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Tue, 7 Mar 2006 23:24:55 +0000 (+0000)
Subject: make it clear that m_page is not NULL in OnBackOrNext() (coverity checked CID 20)
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/47e646643969edf9851834bbde1e8470df2358c6

make it clear that m_page is not NULL in OnBackOrNext() (coverity checked CID 20)


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

diff --git a/src/generic/wizard.cpp b/src/generic/wizard.cpp
index 7054c96707..4690af5eae 100644
--- a/src/generic/wizard.cpp
+++ b/src/generic/wizard.cpp
@@ -780,10 +780,12 @@ void wxWizard::OnBackOrNext(wxCommandEvent& event)
                   (event.GetEventObject() == m_btnPrev),
                   wxT("unknown button") );
 
+    wxCHECK_RET( m_page, _T("should have a valid current page") );
+
     // ask the current page first: notice that we do it before calling
     // GetNext/Prev() because the data transfered from the controls of the page
     // may change the value returned by these methods
-    if ( m_page && (!m_page->Validate() || !m_page->TransferDataFromWindow()) )
+    if ( !m_page->Validate() || !m_page->TransferDataFromWindow() )
     {
         // the page data is incorrect, don't do anything
         return;