diff options
| author | Lioncash <mathew1800@gmail.com> | 2015-08-16 01:08:51 -0400 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2015-08-17 21:56:48 -0400 |
| commit | ca0b9481ea2be14032d4599bae3bc243f87b2179 (patch) | |
| tree | 68167e94190ba19d6d893b39d977b29756fbf63c /Source/Core/DolphinWX/NetPlay/NetWindow.cpp | |
| parent | af35f38491e39e5b030a55784f3a247d3453d7ba (diff) | |
ChangeGameDialog: Internally set the chosen game name
Passing in a reference to an empty string is unnecessary and
overcomplicates design.
Diffstat (limited to 'Source/Core/DolphinWX/NetPlay/NetWindow.cpp')
| -rw-r--r-- | Source/Core/DolphinWX/NetPlay/NetWindow.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/Source/Core/DolphinWX/NetPlay/NetWindow.cpp b/Source/Core/DolphinWX/NetPlay/NetWindow.cpp index c604e38956..346b8b0932 100644 --- a/Source/Core/DolphinWX/NetPlay/NetWindow.cpp +++ b/Source/Core/DolphinWX/NetPlay/NetWindow.cpp @@ -445,17 +445,16 @@ void NetPlayDialog::OnThread(wxThreadEvent& event) void NetPlayDialog::OnChangeGame(wxCommandEvent&) { - wxString game_name; - - ChangeGameDialog cgd(this, m_game_list, game_name); + ChangeGameDialog cgd(this, m_game_list); cgd.ShowModal(); - if (game_name.length()) - { - m_selected_game = WxStrToStr(game_name); - netplay_server->ChangeGame(m_selected_game); - m_game_btn->SetLabel(game_name.Prepend(_(" Game : "))); - } + wxString game_name = cgd.GetChosenGameName(); + if (game_name.empty()) + return; + + m_selected_game = WxStrToStr(game_name); + netplay_server->ChangeGame(m_selected_game); + m_game_btn->SetLabel(game_name.Prepend(_(" Game : "))); } void NetPlayDialog::OnConfigPads(wxCommandEvent&) |