From 4d0caf6254fdb18f4626a3c7937e64422b40d40c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= <peterix@gmail.com>
Date: Tue, 5 Jan 2016 07:32:52 +0100
Subject: GH-1389 wrap QDesktopServices and QProcess::startDetached

Essentially do not pass some environment variables to subprocesses:
* LD_PRELOAD
* LD_LIBRARY_PATH
* LD_DEBUG
* QT_PLUGIN_PATH
* QT_FONTPATH
---
 application/MultiMC.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'application/MultiMC.cpp')

diff --git a/application/MultiMC.cpp b/application/MultiMC.cpp
index a2a14506..fb82510d 100644
--- a/application/MultiMC.cpp
+++ b/application/MultiMC.cpp
@@ -17,7 +17,6 @@
 #include <QLibraryInfo>
 #include <QMessageBox>
 #include <QStringList>
-#include <QDesktopServices>
 #include <QDebug>
 
 #include "InstanceList.h"
@@ -54,6 +53,7 @@
 
 #include <Commandline.h>
 #include <FileSystem.h>
+#include <DesktopServices.h>
 
 using namespace Commandline;
 
@@ -932,12 +932,12 @@ bool MultiMC::openJsonEditor(const QString &filename)
 	const QString file = QDir::current().absoluteFilePath(filename);
 	if (m_settings->get("JsonEditor").toString().isEmpty())
 	{
-		return QDesktopServices::openUrl(QUrl::fromLocalFile(file));
+		return DesktopServices::openUrl(QUrl::fromLocalFile(file));
 	}
 	else
 	{
-		return QProcess::startDetached(m_settings->get("JsonEditor").toString(), QStringList()
-																					 << file);
+		//return DesktopServices::openFile(m_settings->get("JsonEditor").toString(), file);
+		return DesktopServices::run(m_settings->get("JsonEditor").toString(), {});
 	}
 }
 
-- 
cgit v1.2.3