GH-1053 explode launch task into many small steps, each a Task

34 files changed, 1253 insertions, 588 deletions

diff --git a/logic/BaseInstance.cpp b/logic/BaseInstance.cpp
index 066d2642..9ed7ce50 100644
--- a/logic/BaseInstance.cpp
+++ b/logic/BaseInstance.cpp
@@ -52,6 +52,21 @@ BaseInstance::BaseInstance(SettingsObjectPtr globalSettings, SettingsObjectPtr s
 	m_settings->registerOverride(globalSettings->getSetting("LogPrePostOutput"));
 }
 
+QString BaseInstance::getPreLaunchCommand()
+{
+	return settings()->get("PreLaunchCommand").toString();
+}
+
+QString BaseInstance::getWrapperCommand()
+{
+	return settings()->get("WrapperCommand").toString();
+}
+
+QString BaseInstance::getPostExitCommand()
+{
+	return settings()->get("PostExitCommand").toString();
+}
+
 void BaseInstance::iconUpdated(QString key)
 {
 	if(iconKey() == key)
diff --git a/logic/BaseInstance.h b/logic/BaseInstance.h
index 7152ba2d..6201f1ac 100644
--- a/logic/BaseInstance.h
+++ b/logic/BaseInstance.h
@@ -18,6 +18,7 @@
 #include <QObject>
 #include <QDateTime>
 #include <QSet>
+#include <QProcess>
 
 #include "settings/SettingsObject.h"
 
@@ -89,6 +90,10 @@ public:
 	void setGroupInitial(QString val);
 	void setGroupPost(QString val);
 
+	QString getPreLaunchCommand();
+	QString getPostExitCommand();
+	QString getWrapperCommand();
+
 	virtual QStringList extraArguments() const;
 
 	virtual QString intendedVersionId() const = 0;
@@ -146,6 +151,12 @@ public:
 	 */
 	virtual std::shared_ptr<Task> createJarModdingTask() = 0;
 
+
+	/*!
+	 * Create envrironment variables for running the instance
+	 */
+	virtual QProcessEnvironment createEnvironment() = 0;
+
 	/*!
 	 * does any necessary cleanups after the instance finishes. also runs before\
 	 * TODO: turn into a task that can run asynchronously
@@ -157,6 +168,9 @@ public:
 	/// FIXME: this really should be elsewhere...
 	virtual QString instanceConfigFolder() const = 0;
 
+	/// get variables this instance exports
+	virtual QMap<QString, QString> getVariables() const = 0;
+
 	enum InstanceFlag
 	{
 		VersionBrokenFlag = 0x01,
diff --git a/logic/CMakeLists.txt b/logic/CMakeLists.txt
index 53715fa6..7afc0213 100644
--- a/logic/CMakeLists.txt
+++ b/logic/CMakeLists.txt
@@ -93,12 +93,28 @@ set(LOGIC_SOURCES
 	auth/flows/ValidateTask.cpp
 
 	# Game launch logic
-	launch/LoggedProcess.h
+	launch/steps/CheckJava.cpp
+	launch/steps/CheckJava.h
+	launch/steps/LaunchCommand.cpp
+	launch/steps/LaunchCommand.h
+	launch/steps/ModMinecraftJar.cpp
+	launch/steps/ModMinecraftJar.h
+	launch/steps/PostLaunchCommand.cpp
+	launch/steps/PostLaunchCommand.h
+	launch/steps/PreLaunchCommand.cpp
+	launch/steps/PreLaunchCommand.h
+	launch/steps/TextPrint.cpp
+	launch/steps/TextPrint.h
+	launch/steps/Update.cpp
+	launch/steps/Update.h
+	launch/LaunchStep.cpp
+	launch/LaunchStep.h
+	launch/LaunchTask.cpp
+	launch/LaunchTask.h
 	launch/LoggedProcess.cpp
+	launch/LoggedProcess.h
 	launch/MessageLevel.cpp
 	launch/MessageLevel.h
-	launch/LaunchTask.h
-	launch/LaunchTask.cpp
 
 	# Update system
 	updater/GoUpdate.h
diff --git a/logic/NullInstance.h b/logic/NullInstance.h
index bad4b4de..aba3e484 100644
--- a/logic/NullInstance.h
+++ b/logic/NullInstance.h
@@ -62,4 +62,12 @@ public:
 	{
 		return nullptr;
 	};
+	virtual QProcessEnvironment createEnvironment()
+	{
+		return QProcessEnvironment();
+	}
+	virtual QMap<QString, QString> getVariables() const
+	{
+		return QMap<QString, QString>();
+	}
 };
diff --git a/logic/auth/YggdrasilTask.cpp b/logic/auth/YggdrasilTask.cpp
index 3b6e0cfb..929ab0bf 100644
--- a/logic/auth/YggdrasilTask.cpp
+++ b/logic/auth/YggdrasilTask.cpp
@@ -73,12 +73,13 @@ void YggdrasilTask::heartbeat()
 	progress(count, timeout_max);
 }
 
-void YggdrasilTask::abort()
+bool YggdrasilTask::abort()
 {
 	progress(timeout_max, timeout_max);
 	// TODO: actually use this in a meaningful way
 	m_aborted = YggdrasilTask::BY_USER;
 	m_netReply->abort();
+	return true;
 }
 
 void YggdrasilTask::abortByTimeout()
diff --git a/logic/auth/YggdrasilTask.h b/logic/auth/YggdrasilTask.h
index 306d7358..e81eeb44 100644
--- a/logic/auth/YggdrasilTask.h
+++ b/logic/auth/YggdrasilTask.h
@@ -131,7 +131,7 @@ slots:
 	void changeState(State newState, QString reason=QString());
 public
 slots:
-	virtual void abort() override;
+	virtual bool abort() override;
 	void abortByTimeout();
 	State state();
 protected:
diff --git a/logic/forge/ForgeVersionList.cpp b/logic/forge/ForgeVersionList.cpp
index 4f9870f7..7c0e851a 100644
--- a/logic/forge/ForgeVersionList.cpp
+++ b/logic/forge/ForgeVersionList.cpp
@@ -144,9 +144,9 @@ void ForgeListLoadTask::executeTask()
 	listJob->start();
 }
 
-void ForgeListLoadTask::abort()
+bool ForgeListLoadTask::abort()
 {
-	listJob->abort();
+	return listJob->abort();
 }
 
 bool ForgeListLoadTask::parseForgeList(QList<BaseVersionPtr> &out)
diff --git a/logic/forge/ForgeVersionList.h b/logic/forge/ForgeVersionList.h
index 2ffc2d68..35af3233 100644
--- a/logic/forge/ForgeVersionList.h
+++ b/logic/forge/ForgeVersionList.h
@@ -66,7 +66,7 @@ public:
 	explicit ForgeListLoadTask(ForgeVersionList *vlist);
 
 	virtual void executeTask();
-	virtual void abort();
+	virtual bool abort();
 
 protected
 slots:
diff --git a/logic/java/JavaCheckerJob.h b/logic/java/JavaCheckerJob.h
index 84b6fc6f..4e2038ca 100644
--- a/logic/java/JavaCheckerJob.h
+++ b/logic/java/JavaCheckerJob.h
@@ -68,9 +68,6 @@ signals:
 	void started();
 	void finished(QList<JavaCheckResult>);
 
-public slots:
-	virtual void abort() {};
-
 private slots:
 	void partFinished(JavaCheckResult result);
 
diff --git a/logic/launch/LaunchStep.cpp b/logic/launch/LaunchStep.cpp
new file mode 100644
index 00000000..9799d767
--- /dev/null
+++ b/logic/launch/LaunchStep.cpp
@@ -0,0 +1,26 @@
+/* Copyright 2013-2015 MultiMC Contributors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include "LaunchStep.h"
+#include "LaunchTask.h"
+
+void LaunchStep::bind(LaunchTask *parent)
+{
+	m_parent = parent;
+	connect(this, &LaunchStep::readyForLaunch, parent, &LaunchTask::onReadyForLaunch);
+	connect(this, &LaunchStep::logLine, parent, &LaunchTask::onLogLine);
+	connect(this, &LaunchStep::logLines, parent, &LaunchTask::onLogLines);
+	connect(this, &LaunchStep::finished, parent, &LaunchTask::onStepFinished);
+}
diff --git a/logic/launch/LaunchStep.h b/logic/launch/LaunchStep.h
new file mode 100644
index 00000000..9b9631cb
--- /dev/null
+++ b/logic/launch/LaunchStep.h
@@ -0,0 +1,47 @@
+/* Copyright 2013-2015 MultiMC Contributors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#pragma once
+
+#include "tasks/Task.h"
+#include "MessageLevel.h"
+
+#include <QStringList>
+
+class LaunchTask;
+class LaunchStep: public Task
+{
+	Q_OBJECT
+public: /* methods */
+	explicit LaunchStep(LaunchTask *parent):Task(nullptr), m_parent(parent)
+	{
+		bind(parent);
+	};
+	virtual ~LaunchStep() {};
+
+protected: /* methods */
+	virtual void bind(LaunchTask *parent);
+
+signals:
+	void logLines(QStringList lines, MessageLevel::Enum level);
+	void logLine(QString line, MessageLevel::Enum level);
+	void readyForLaunch();
+
+public slots:
+	virtual void proceed() {};
+
+protected: /* data */
+	LaunchTask *m_parent;
+};
\ No newline at end of file
diff --git a/logic/launch/LaunchTask.cpp b/logic/launch/LaunchTask.cpp
index 83bbba97..248761f0 100644
--- a/logic/launch/LaunchTask.cpp
+++ b/logic/launch/LaunchTask.cpp
@@ -27,105 +27,14 @@
 #include <QRegularExpression>
 #include <QCoreApplication>
 #include <QStandardPaths>
-
-#define IBUS "@im=ibus"
-
-void LaunchTask::initializeEnvironment()
-{
-	// prepare the process environment
-	QProcessEnvironment rawenv = QProcessEnvironment::systemEnvironment();
-
-	QStringList ignored =
-	{
-		"JAVA_ARGS",
-		"CLASSPATH",
-		"CONFIGPATH",
-		"JAVA_HOME",
-		"JRE_HOME",
-		"_JAVA_OPTIONS",
-		"JAVA_OPTIONS",
-		"JAVA_TOOL_OPTIONS"
-	};
-	for(auto key: rawenv.keys())
-	{
-		auto value = rawenv.value(key);
-		// filter out dangerous java crap
-		if(ignored.contains(key))
-		{
-			qDebug() << "Env: ignoring" << key << value;
-			continue;
-		}
-		// filter MultiMC-related things
-		if(key.startsWith("QT_"))
-		{
-			qDebug() << "Env: ignoring" << key << value;
-			continue;
-		}
-#ifdef Q_OS_LINUX
-		// Do not pass LD_* variables to java. They were intended for MultiMC
-		if(key.startsWith("LD_"))
-		{
-			qDebug() << "Env: ignoring" << key << value;
-			continue;
-		}
-		// Strip IBus
-		// IBus is a Linux IME framework. For some reason, it breaks MC?
-		if (key == "XMODIFIERS" && value.contains(IBUS))
-		{
-			QString save = value;
-			value.replace(IBUS, "");
-			qDebug() << "Env: stripped" << IBUS << "from" << save << ":" << value;
-		}
-		if(key == "GAME_PRELOAD")
-		{
-			m_env.insert("LD_PRELOAD", value);
-			continue;
-		}
-		if(key == "GAME_LIBRARY_PATH")
-		{
-			m_env.insert("LD_LIBRARY_PATH", value);
-			continue;
-		}
-#endif
-		qDebug() << "Env: " << key << value;
-		m_env.insert(key, value);
-	}
-#ifdef Q_OS_LINUX
-	// HACK: Workaround for QTBUG42500
-	if(!m_env.contains("LD_LIBRARY_PATH"))
-	{
-		m_env.insert("LD_LIBRARY_PATH", "");
-	}
-#endif
-
-	// export some infos
-	auto variables = getVariables();
-	for (auto it = variables.begin(); it != variables.end(); ++it)
-	{
-		m_env.insert(it.key(), it.value());
-	}
-}
+#include <assert.h>
 
 void LaunchTask::init()
 {
-	initializeEnvironment();
-
-	m_process.setProcessEnvironment(m_env);
-	connect(&m_process, &LoggedProcess::log, this, &LaunchTask::on_log);
-	connect(&m_process, &LoggedProcess::stateChanged, this, &LaunchTask::on_state);
-
-	m_prelaunchprocess.setProcessEnvironment(m_env);
-	connect(&m_prelaunchprocess, &LoggedProcess::log, this, &LaunchTask::on_log);
-	connect(&m_prelaunchprocess, &LoggedProcess::stateChanged, this, &LaunchTask::on_pre_state);
-
-	m_postlaunchprocess.setProcessEnvironment(m_env);
-	connect(&m_postlaunchprocess, &LoggedProcess::log, this, &LaunchTask::on_log);
-	connect(&m_postlaunchprocess, &LoggedProcess::stateChanged, this, &LaunchTask::on_post_state);
-
 	m_instance->setRunning(true);
 }
 
-std::shared_ptr<LaunchTask> LaunchTask::create(MinecraftInstancePtr inst)
+std::shared_ptr<LaunchTask> LaunchTask::create(InstancePtr inst)
 {
 	std::shared_ptr<LaunchTask> proc(new LaunchTask(inst));
 	proc->init();
@@ -136,25 +45,77 @@ LaunchTask::LaunchTask(InstancePtr instance): m_instance(instance)
 {
 }
 
-QString LaunchTask::censorPrivateInfo(QString in)
+void LaunchTask::appendStep(std::shared_ptr<LaunchStep> step)
 {
-	if (!m_session)
-		return in;
+	m_steps.append(step);
+}
 
-	if (m_session->session != "-")
-		in.replace(m_session->session, "<SESSION ID>");
-	in.replace(m_session->access_token, "<ACCESS TOKEN>");
-	in.replace(m_session->client_token, "<CLIENT TOKEN>");
-	in.replace(m_session->uuid, "<PROFILE ID>");
-	in.replace(m_session->player_name, "<PROFILE NAME>");
+void LaunchTask::prependStep(std::shared_ptr<LaunchStep> step)
+{
+	m_steps.prepend(step);
+}
 
-	auto i = m_session->u.properties.begin();
-	while (i != m_session->u.properties.end())
+void LaunchTask::executeTask()
+{
+	if(!m_steps.size())
 	{
-		in.replace(i.value(), "<" + i.key().toUpper() + ">");
-		++i;
+		state = LaunchTask::Finished;
+		emitSucceeded();
 	}
+	state = LaunchTask::Running;
+	onStepFinished();
+}
+
+void LaunchTask::onReadyForLaunch()
+{
+	state = LaunchTask::Waiting;
+	emit readyForLaunch();
+}
+
+void LaunchTask::onStepFinished()
+{
+	// initial -> just start the first step
+	if(currentStep == -1)
+	{
+		currentStep ++;
+		m_steps[currentStep]->start();
+		return;
+	}
+
+	auto step = m_steps[currentStep];
+	if(step->successful())
+	{
+		// end?
+		if(currentStep == m_steps.size() - 1)
+		{
+			emitSucceeded();
+		}
+		else
+		{
+			currentStep ++;
+			step = m_steps[currentStep];
+			step->start();
+		}
+	}
+	else
+	{
+		emitFailed(step->failReason());
+	}
+}
 
+void LaunchTask::setCensorFilter(QMap<QString, QString> filter)
+{
+	m_censorFilter = filter;
+}
+
+QString LaunchTask::censorPrivateInfo(QString in)
+{
+	auto iter = m_censorFilter.begin();
+	while (iter != m_censorFilter.end())
+	{
+		in.replace(iter.key(), iter.value());
+		iter++;
+	}
 	return in;
 }
 
@@ -199,163 +160,58 @@ MessageLevel::Enum LaunchTask::guessLevel(const QString &line, MessageLevel::Enu
 	return level;
 }
 
-QMap<QString, QString> LaunchTask::getVariables() const
+void LaunchTask::proceed()
 {
-	auto mcInstance = std::dynamic_pointer_cast<MinecraftInstance>(m_instance);
-	QMap<QString, QString> out;
-	out.insert("INST_NAME", mcInstance->name());
-	out.insert("INST_ID", mcInstance->id());
-	out.insert("INST_DIR", QDir(mcInstance->instanceRoot()).absolutePath());
-	out.insert("INST_MC_DIR", QDir(mcInstance->minecraftRoot()).absolutePath());
-	out.insert("INST_JAVA", mcInstance->settings()->get("JavaPath").toString());
-	out.insert("INST_JAVA_ARGS", javaArguments().join(' '));
-	return out;
-}
-
-QStringList LaunchTask::javaArguments() const
-{
-	QStringList args;
-
-	// custom args go first. we want to override them if we have our own here.
-	args.append(m_instance->extraArguments());
-
-	// OSX dock icon and name
-#ifdef Q_OS_MAC
-	args << "-Xdock:icon=icon.png";
-	args << QString("-Xdock:name=\"%1\"").arg(m_instance->windowTitle());
-#endif
-
-	// HACK: Stupid hack for Intel drivers. See: https://mojang.atlassian.net/browse/MCL-767
-#ifdef Q_OS_WIN32
-	args << QString("-XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_"
-					"minecraft.exe.heapdump");
-#endif
-
-	args << QString("-Xms%1m").arg(m_instance->settings()->get("MinMemAlloc").toInt());
-	args << QString("-Xmx%1m").arg(m_instance->settings()->get("MaxMemAlloc").toInt());
-
-	// No PermGen in newer java.
-	auto javaVersion = m_instance->settings()->get("JavaVersion");
-	if(Strings::naturalCompare(javaVersion.toString(), "1.8.0", Qt::CaseInsensitive) < 0)
-	{
-		auto permgen = m_instance->settings()->get("PermGen").toInt();
-		if (permgen != 64)
-		{
-			args << QString("-XX:PermSize=%1m").arg(permgen);
-		}
-	}
-
-	args << "-Duser.language=en";
-	if (!m_nativeFolder.isEmpty())
-		args << QString("-Djava.library.path=%1").arg(m_nativeFolder);
-	args << "-jar" << PathCombine(QCoreApplication::applicationDirPath(), "jars", "NewLaunch.jar");
-
-	return args;
-}
-
-void LaunchTask::checkJava()
-{
-	m_javaPath = m_instance->settings()->get("JavaPath").toString();
-	emit log("Java path is:\n" + m_javaPath + "\n\n");
-
-	auto realJavaPath = QStandardPaths::findExecutable(m_javaPath);
-	if (realJavaPath.isEmpty())
-	{
-		emit log(tr("The java binary \"%1\" couldn't be found. You may have to set up java "
-					"if Minecraft fails to launch.").arg(m_javaPath),
-				 MessageLevel::Warning);
-	}
-
-	QFileInfo javaInfo(realJavaPath);
-	qlonglong javaUnixTime = javaInfo.lastModified().toMSecsSinceEpoch();
-	auto storedUnixTime = m_instance->settings()->get("JavaTimestamp").toLongLong();
-	this->m_javaUnixTime = javaUnixTime;
-	// if they are not the same, check!
-	if(javaUnixTime != storedUnixTime)
+	if(state != LaunchTask::Waiting)
 	{
-		m_JavaChecker = std::make_shared<JavaChecker>();
-		bool successful = false;
-		QString errorLog;
-		QString version;
-		emit log(tr("Checking Java version..."), MessageLevel::MultiMC);
-		connect(m_JavaChecker.get(), &JavaChecker::checkFinished, this, &LaunchTask::checkJavaFinished);
-		m_JavaChecker->m_path = realJavaPath;
-		m_JavaChecker->performCheck();
+		return;
 	}
-	preLaunch();
+	m_steps[currentStep]->proceed();
 }
 
-void LaunchTask::checkJavaFinished(JavaCheckResult result)
+bool LaunchTask::abort()
 {
-	if(!result.valid)
+	switch(state)
 	{
-		// Error message displayed if java can't start
-		emit log(tr("Could not start java:"), MessageLevel::Error);
-		auto lines = result.errorLog.split('\n');
-		for(auto line: lines)
+		case LaunchTask::Aborted:
+		case LaunchTask::Failed:
+		case LaunchTask::Finished:
+			return true;
+		case LaunchTask::NotStarted:
 		{
-			emit log(line, MessageLevel::Error);
+			state = LaunchTask::Aborted;
+			emitFailed("Aborted");
+			return true;
 		}
-		emit log("\nCheck your MultiMC Java settings.", MessageLevel::MultiMC);
-		emitFailed(tr("Could not start java!"));
-	}
-	else
-	{
-		emit log(tr("Java version is %1!\n").arg(result.javaVersion), MessageLevel::MultiMC);
-		m_instance->settings()->set("JavaVersion", result.javaVersion);
-		m_instance->settings()->set("JavaTimestamp", m_javaUnixTime);
-		preLaunch();
+		case LaunchTask::Running:
+		case LaunchTask::Waiting:
+		{
+			auto step = m_steps[currentStep];
+			if(!step->canAbort())
+			{
+				return false;
+			}
+			if(step->abort())
+			{
+				state = LaunchTask::Aborted;
+				return true;
+			}
+		}
+		default:
+			break;
 	}
+	return false;
 }
 
-void LaunchTask::executeTask()
-{
-	printHeader();
-	emit log("Minecraft folder is:\n" + m_process.workingDirectory() + "\n\n");
-
-	checkJava();
-}
-
-void LaunchTask::launch()
-{
-	QString launchString("launch\n");
-	m_process.write(launchString.toUtf8());
-}
-
-void LaunchTask::abort()
-{
-	QString launchString("abort\n");
-	m_process.write(launchString.toUtf8());
-}
-
-
-void LaunchTask::setWorkdir(QString path)
-{
-	QDir mcDir(path);
-	m_process.setWorkingDirectory(mcDir.absolutePath());
-	m_prelaunchprocess.setWorkingDirectory(mcDir.absolutePath());
-	m_postlaunchprocess.setWorkingDirectory(mcDir.absolutePath());
-}
-
-void LaunchTask::printHeader()
-{
-	emit log(m_header);
-}
-
-void LaunchTask::on_log(QStringList lines, MessageLevel::Enum level)
-{
-	logOutput(lines, level);
-}
-
-void LaunchTask::logOutput(const QStringList &lines, MessageLevel::Enum defaultLevel)
+void LaunchTask::onLogLines(const QStringList &lines, MessageLevel::Enum defaultLevel)
 {
 	for (auto & line: lines)
 	{
-		logOutput(line, defaultLevel);
+		onLogLine(line, defaultLevel);
 	}
 }
 
-void LaunchTask::logOutput(QString line, MessageLevel::Enum level)
+void LaunchTask::onLogLine(QString line, MessageLevel::Enum level)
 {
 	// if the launcher part set a log level, use it
 	auto innerLevel = MessageLevel::fromLine(line);
@@ -376,230 +232,6 @@ void LaunchTask::logOutput(QString line, MessageLevel::Enum level)
 	emit log(line, level);
 }
 
-void LaunchTask::preLaunch()
-{
-	QString prelaunch_cmd = m_instance->settings()->get("PreLaunchCommand").toString();
-	if (!prelaunch_cmd.isEmpty())
-	{
-		prelaunch_cmd = substituteVariables(prelaunch_cmd);
-		// Launch
-		emit log(tr("Running Pre-Launch command: %1").arg(prelaunch_cmd));
-		m_prelaunchprocess.start(prelaunch_cmd);
-	}
-	else
-	{
-		on_pre_state(LoggedProcess::Skipped);
-	}
-}
-
-void LaunchTask::on_pre_state(LoggedProcess::State state)
-{
-	switch(state)
-	{
-		case LoggedProcess::Aborted:
-		case LoggedProcess::Crashed:
-		case LoggedProcess::FailedToStart:
-		{
-			QString error = tr("Pre-Launch command failed with code %1.\n\n").arg(m_prelaunchprocess.exitCode());
-			emit log(error, MessageLevel::Fatal);
-			emitFailed(error);
-			return;
-		}
-		case LoggedProcess::Finished:
-		{
-			emit log(tr("Pre-Launch command ran successfully.\n\n"));
-		}
-		case LoggedProcess::Skipped:
-		{
-			m_instance->reload();
-			updateInstance();
-		}
-		default:
-			break;
-	}
-}
-
-void LaunchTask::updateInstance()
-{
-	m_updateTask = m_instance->createUpdateTask();
-	if(m_updateTask)
-	{
-		connect(m_updateTask.get(), SIGNAL(finished()), this, SLOT(updateFinished()));
-		m_updateTask->start();
-		return;
-	}
-	makeReady();
-}
-
-void LaunchTask::updateFinished()
-{
-	if(m_updateTask->successful())
-	{
-		doJarModding();
-	}
-	else
-	{
-		QString reason = tr("Instance update failed because: %1.\n\n").arg(m_updateTask->failReason());
-		emit log(reason, MessageLevel::Fatal);
-		emitFailed(reason);
-	}
-}
-
-void LaunchTask::doJarModding()
-{
-	m_jarModTask = m_instance->createJarModdingTask();
-	if(!m_jarModTask)
-	{
-		jarModdingSucceeded();
-	}
-	connect(m_jarModTask.get(), SIGNAL(succeeded()), this, SLOT(jarModdingSucceeded()));
-	connect(m_jarModTask.get(), SIGNAL(failed(QString)), this, SLOT(jarModdingFailed(QString)));
-	m_jarModTask->start();
-}
-
-void LaunchTask::jarModdingSucceeded()
-{
-	makeReady();
-}
-
-void LaunchTask::jarModdingFailed(QString reason)
-{
-	emitFailed(reason);
-}
-
-void LaunchTask::makeReady()
-{
-	QStringList args = javaArguments();
-	QString allArgs = args.join(", ");
-	emit log("Java Arguments:\n[" + censorPrivateInfo(allArgs) + "]\n\n");
-
-	QString wrapperCommand = m_instance->settings()->get("WrapperCommand").toString();
-	if(!wrapperCommand.isEmpty())
-	{
-		auto realWrapperCommand = QStandardPaths::findExecutable(wrapperCommand);
-		if (realWrapperCommand.isEmpty())
-		{
-			QString reason = tr("The wrapper command \"%1\" couldn't be found.").arg(wrapperCommand);
-			emit log(reason, MessageLevel::Fatal);
-			emitFailed(reason);
-			return;
-		}
-		emit log("Wrapper command is:\n" + wrapperCommand + "\n\n");
-		args.prepend(m_javaPath);
-		m_process.start(wrapperCommand, args);
-	}
-	else
-	{
-		m_process.start(m_javaPath, args);
-	}
-
-	// instantiate the launcher part
-	if (!m_process.waitForStarted())
-	{
-		//: Error message displayed if instace can't start
-		QString reason = tr("Could not launch minecraft!");
-		emit log(reason, MessageLevel::Fatal);
-		emitFailed(reason);
-		return;
-	}
-
-	emit log(tr("Minecraft process ID: %1\n\n").arg(m_process.processId()), MessageLevel::MultiMC);
-
-	// send the launch script to the launcher part
-	m_process.write(launchScript.toUtf8());
-
-	emit readyForLaunch();
-}
-
-void LaunchTask::on_state(LoggedProcess::State state)
-{
-	QProcess::ExitStatus estat = QProcess::NormalExit;
-	switch(state)
-	{
-		case LoggedProcess::Aborted:
-		case LoggedProcess::Crashed:
-		case LoggedProcess::FailedToStart:
-		{
-			estat = QProcess::CrashExit;
-			emitFailed("Game crashed.");
-			return;
-		}
-		case LoggedProcess::Finished:
-		{
-			auto exitCode = m_process.exitCode();
-			m_postlaunchprocess.processEnvironment().insert("INST_EXITCODE", QString(exitCode));
-			// run post-exit
-			postLaunch();
-			break;
-		}
-		case LoggedProcess::Skipped:
-			qWarning() << "Illegal game state: Skipped"