added wxCHECK to ensure that we don't dereference a NULL pointer

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

diff --git a/src/motif/spinbutt.cpp b/src/motif/spinbutt.cpp
index dd10927d967dfa0b4bd1a1586a070dca729c4a3e..a84d1f38dd822bec74eaf937e49659b1cae619d3 100644 (file)
--- a/src/motif/spinbutt.cpp
+++ b/src/motif/spinbutt.cpp
@@ -189,6 +189,8 @@ bool wxArrowButton::Create( wxSpinButton* parent,
                             ArrowDirection d,
                             const wxPoint& pos, const wxSize& size )
 {
                             ArrowDirection d,
                             const wxPoint& pos, const wxSize& size )
 {

+    wxCHECK_MSG( parent, false, _T("must have a valid parent") );
+

     int arrow_dir = XmARROW_UP;
 
     switch( d )
     int arrow_dir = XmARROW_UP;
 
     switch( d )


@@ -207,7 +209,7 @@ bool wxArrowButton::Create( wxSpinButton* parent,
         break;
     }
 
         break;
     }
 

-    if( parent ) parent->AddChild( this );
+    parent->AddChild( this );

 
     Widget parentWidget = (Widget) parent->GetClientWidget();
     m_mainWidget = (WXWidget) XtVaCreateManagedWidget( "XmArrowButton",
 
     Widget parentWidget = (Widget) parent->GetClientWidget();
     m_mainWidget = (WXWidget) XtVaCreateManagedWidget( "XmArrowButton",