Fix the incomplete cleanup in Profile::UnloadMachineData that was resulting in

duplicate sessions being added when making preference changes to a brand new
profile with existing data.

Preference changes trigger a reload, see PreferencesDialog::Save and
needs_reload, leading to MainWindow::reloadProfile -> MainWindow::CloseProfile
-> Profile::UnloadMachineData.

This is the root cause of the duplicate sessions, but the fact that it only
happened with newly created profiles, rather than on subsequent launches,
demonstrates an inconsistency in state of Profile. It should be identical when
initially created and when loaded via subsequent application launch.

oscar/SleepLib/profiles.cpp

@@ -576,11 +576,6 @@ void Profile::DataFormatError(Machine *m)
 }
 void Profile::UnloadMachineData()
 {
-    if (!m_machopened) {
-        qCritical() << "Profile::UnloadMachineData() called with m_machopened==false";
-        return;
-    }
-
     for (auto & mach : m_machlist) {
         mach->saveSessionInfo();
         mach->sessionlist.clear();