diff --git a/pcsx2/gui/RecentIsoList.cpp b/pcsx2/gui/RecentIsoList.cpp index 34a2c284c2..492a08fd1a 100644 --- a/pcsx2/gui/RecentIsoList.cpp +++ b/pcsx2/gui/RecentIsoList.cpp @@ -121,15 +121,14 @@ void RecentIsoManager::Repopulate() m_Separator = m_Menu->AppendSeparator(); - // The following line is important - // Update: It also makes debug WX assert on Windows. Can we review wether this is needed in Linux at all? - // Below is a version that doesn't trigger the assert (WX on Windows complains about an empty string), but - // ideally this line should get removed. - - //wxString temp = L"isupresswarnings"; - //m_Menu->Remove( m_Menu->Append( -1, temp ) ); + // From arcum's comment on r5141 + // What was happening is that when all the radio button menu items in a group are deleted, + // wxwidgets deletes the group, but when you start adding radio menu items again, + // it trys to add them to a group that doesn't exist. Since the group doesn't exist, + // it starts a new group, but it also spews a couple warnings about it in Linux. +#ifdef __LINUX__ m_Menu->Remove( m_Menu->Append( -1, wxEmptyString ) ); - +#endif //Note: the internal recent iso list (m_Items) has the most recent item last (also at the INI file) // but the menu is composed in reverse order such that the most recent item appears at the top. for( int i=cnt-1; i>=0; --i )