Merge PR #5596: FIX(client): Audio wizard being shown repeatedly

The switch to a new settings format messed up the setting for whether or
not the audio wizard has been shown already or not. For new
installations, the wizard would be shown, but the fact that it has been
shown is not reflected in the settings causing the wizard to be re-shown
upon a restart.

This commit fixes this by making sure that the respective setting is set
after the wizard has been shown for the first time.

1 files changed, 6 insertions, 3 deletions

diff --git a/src/mumble/main.cpp b/src/mumble/main.cpp
index a67dd62ef..772f25f7a 100644
--- a/src/mumble/main.cpp
+++ b/src/mumble/main.cpp
@@ -56,6 +56,8 @@
 #include <QtGui/QDesktopServices>
 #include <QtWidgets/QMessageBox>
 
+#include <memory>
+
 #ifdef USE_DBUS
 #	include <QtDBus/QDBusInterface>
 #endif
@@ -721,9 +723,10 @@ int main(int argc, char **argv) {
 	a.setQuitOnLastWindowClosed(false);
 
 	if (!Global::get().s.audioWizardShown) {
-		AudioWizard *aw = new AudioWizard(Global::get().mw);
-		aw->exec();
-		delete aw;
+		auto wizard = std::make_unique< AudioWizard >(Global::get().mw);
+		wizard->exec();
+
+		Global::get().s.audioWizardShown = true;
 	}
 
 	if (!CertWizard::validateCert(Global::get().s.kpCertificate)) {