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
This commit is contained in:
@@ -780,10 +780,12 @@ void wxWizard::OnBackOrNext(wxCommandEvent& event)
|
|||||||
(event.GetEventObject() == m_btnPrev),
|
(event.GetEventObject() == m_btnPrev),
|
||||||
wxT("unknown button") );
|
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
|
// ask the current page first: notice that we do it before calling
|
||||||
// GetNext/Prev() because the data transfered from the controls of the page
|
// GetNext/Prev() because the data transfered from the controls of the page
|
||||||
// may change the value returned by these methods
|
// 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
|
// the page data is incorrect, don't do anything
|
||||||
return;
|
return;
|
||||||
|
Reference in New Issue
Block a user