if ( !wxListBox::Create(parent, wxID_ANY,
wxDefaultPosition, wxDefaultSize,
0, NULL,
- wxBORDER_SIMPLE | wxLB_INT_HEIGHT |
- m_combo->GetWindowStyle() & wxCB_SORT ? wxLB_SORT : 0) )
+ wxBORDER_SIMPLE |
+ ( m_combo->GetWindowStyle() & wxCB_SORT ? wxLB_SORT : 0 ) ) )
return false;
// we don't react to the mouse events outside the window at all
{
if ( !value.empty() )
{
- if (FindString(value) != wxNOT_FOUND)
- wxListBox::SetStringSelection(value);
- }
+ if (FindString(value) != wxNOT_FOUND)
+ wxListBox::SetStringSelection(value);
+ }
else
wxListBox::SetSelection(-1);
}
void wxComboBox::SetSelection(int n)
{
- wxCHECK_RET( IsValid(n), _T("invalid index in wxComboBox::Select") );
+ wxCHECK_RET( (n == wxNOT_FOUND || IsValid(n)), _T("invalid index in wxComboBox::Select") );
GetLBox()->SetSelection(n);
- if ( GetTextCtrl() ) GetTextCtrl()->SetValue(GetLBox()->GetString(n));
+
+ wxString str;
+ if ( n != wxNOT_FOUND )
+ str = GetLBox()->GetString(n);
+
+ SetText(str);
}
int wxComboBox::GetSelection() const