From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Tue, 16 Apr 2002 19:34:42 +0000 (+0000)
Subject: removed redundant code
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/b5b49e42939fd7ef098241733648b534f71b526c

removed redundant code


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

diff --git a/src/common/dlgcmn.cpp b/src/common/dlgcmn.cpp
index 04ad2b6756..d874e1a2fe 100644
--- a/src/common/dlgcmn.cpp
+++ b/src/common/dlgcmn.cpp
@@ -142,21 +142,15 @@ wxSizer *wxDialogBase::CreateButtonSizer( long flags )
     wxButton *yes = (wxButton *) NULL;
     wxButton *no = (wxButton *) NULL;
 
-    // always show an OK button, unless only YES_NO is given
-    if ((flags & wxYES_NO) == 0) flags = flags | wxOK;
+    // always show an OK button, unless we have both YES and NO
+    if ( (flags & wxYES_NO) != wxYES_NO )
+        flags |= wxOK;
 
-    if (flags & wxYES_NO)
-    {
-        yes = new wxButton( this, wxID_YES, _("Yes"),wxDefaultPosition,wxDefaultSize,wxCLIP_SIBLINGS );
-        box->Add( yes, 0, wxLEFT|wxRIGHT, margin );
-        no = new wxButton( this, wxID_NO, _("No") ,wxDefaultPosition,wxDefaultSize,wxCLIP_SIBLINGS);
-        box->Add( no, 0, wxLEFT|wxRIGHT, margin );
-    } else
     if (flags & wxYES)
     {
         yes = new wxButton( this, wxID_YES, _("Yes"),wxDefaultPosition,wxDefaultSize,wxCLIP_SIBLINGS );
         box->Add( yes, 0, wxLEFT|wxRIGHT, margin );
-    } else
+    }
     if (flags & wxNO)
     {
         no = new wxButton( this, wxID_NO, _("No"),wxDefaultPosition,wxDefaultSize,wxCLIP_SIBLINGS );
@@ -190,6 +184,7 @@ wxSizer *wxDialogBase::CreateButtonSizer( long flags )
         box->Add( cancel, 0, wxLEFT|wxRIGHT, margin );
     }
 
+    // choose the default button
     if (flags & wxNO_DEFAULT)
     {
         if (no)