From 598a92426f49ad8c1097f1f59455f53cf4bcea9e Mon Sep 17 00:00:00 2001 From: Lioncash Date: Sun, 2 Apr 2017 07:44:08 -0400 Subject: [PATCH] TASInputDlg: Get rid of unnecessary event casts wxCommandEvent already stores the necessary data; the control object itself doesn't need to be retrieved and casted. --- Source/Core/DolphinWX/TASInputDlg.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/Source/Core/DolphinWX/TASInputDlg.cpp b/Source/Core/DolphinWX/TASInputDlg.cpp index 7b021b6366..f6d313dd3a 100644 --- a/Source/Core/DolphinWX/TASInputDlg.cpp +++ b/Source/Core/DolphinWX/TASInputDlg.cpp @@ -941,16 +941,19 @@ void TASInputDlg::UpdateFromSliders(wxCommandEvent& event) if (control != nullptr && event.GetId() == control->slider_id) text = control->text; } - int value = ((wxSlider*)event.GetEventObject())->GetValue(); - if (text) - text->SetValue(std::to_string(value)); + + if (!text) + return; + + const int slider_value = event.GetInt(); + text->SetValue(std::to_string(slider_value)); } void TASInputDlg::UpdateFromText(wxCommandEvent& event) { unsigned long value; - if (!((wxTextCtrl*)event.GetEventObject())->GetValue().ToULong(&value)) + if (!event.GetString().ToULong(&value)) return; for (Control* const control : m_controls)