Don't set value unnecessarily in wxSpinCtrl::Create() in wxMSW.
This is not only unnecessary but also results in an unexpected event (which could be avoided by using ChangeValue() but why bother calling it at all if we don't need to). Closes #13589. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@70800 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
@@ -394,14 +394,15 @@ bool wxSpinCtrl::Create(wxWindow *parent,
|
|||||||
// Set the range in the native control
|
// Set the range in the native control
|
||||||
SetRange(min, max);
|
SetRange(min, max);
|
||||||
|
|
||||||
if ( !value.empty() )
|
// If necessary, set the textual value. Don't do it if it's the same as the
|
||||||
|
// numeric value though.
|
||||||
|
if ( value != wxString::Format("%d", initial) )
|
||||||
{
|
{
|
||||||
SetValue(value);
|
SetValue(value);
|
||||||
m_oldValue = (int) wxAtol(value);
|
m_oldValue = (int) wxAtol(value);
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
SetValue(wxString::Format(wxT("%d"), initial));
|
|
||||||
m_oldValue = initial;
|
m_oldValue = initial;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user