Drop Qt5Compat (#4176)

launcher/LoggedProcess.cpp

@@ -36,10 +36,10 @@
 
 #include "LoggedProcess.h"
 #include <QDebug>
-#include <QTextDecoder>
+#include <QStringDecoder>
 #include "MessageLevel.h"
 
-LoggedProcess::LoggedProcess(const QTextCodec* output_codec, QObject* parent)
+LoggedProcess::LoggedProcess(const QStringConverter::Encoding output_codec, QObject* parent)
     : QProcess(parent), m_err_decoder(output_codec), m_out_decoder(output_codec)
 {
     // QProcess has a strange interface... let's map a lot of those into a few.
@@ -57,9 +57,9 @@ LoggedProcess::~LoggedProcess()
     }
 }
 
-QStringList LoggedProcess::reprocess(const QByteArray& data, QTextDecoder& decoder)
+QStringList LoggedProcess::reprocess(const QByteArray& data, QStringDecoder& decoder)
 {
-    auto str = decoder.toUnicode(data);
+    QString str = decoder(data);
 
     if (!m_leftover_line.isEmpty()) {
         str.prepend(m_leftover_line);

launcher/LoggedProcess.h

@@ -36,7 +36,7 @@
 #pragma once
 
 #include <QProcess>
-#include <QTextDecoder>
+#include <QStringDecoder>
 #include "MessageLevel.h"
 
 /*
@@ -49,7 +49,7 @@ class LoggedProcess : public QProcess {
     enum State { NotRunning, Starting, FailedToStart, Running, Finished, Crashed, Aborted };
 
    public:
-    explicit LoggedProcess(const QTextCodec* output_codec = QTextCodec::codecForLocale(), QObject* parent = 0);
+    explicit LoggedProcess(QStringConverter::Encoding outputEncoding = QStringConverter::System, QObject* parent = nullptr);
     virtual ~LoggedProcess();
 
     State state() const;
@@ -77,11 +77,11 @@ class LoggedProcess : public QProcess {
    private:
     void changeState(LoggedProcess::State state);
 
-    QStringList reprocess(const QByteArray& data, QTextDecoder& decoder);
+    QStringList reprocess(const QByteArray& data, QStringDecoder& decoder);
 
    private:
-    QTextDecoder m_err_decoder;
-    QTextDecoder m_out_decoder;
+    QStringDecoder m_err_decoder;
+    QStringDecoder m_out_decoder;
     QString m_leftover_line;
     bool m_killed = false;
     State m_state = NotRunning;

launcher/minecraft/launch/LauncherPartLaunch.cpp

@@ -50,7 +50,7 @@
 
 LauncherPartLaunch::LauncherPartLaunch(LaunchTask* parent)
     : LaunchStep(parent)
-    , m_process(parent->instance()->getJavaVersion().defaultsToUtf8() ? QTextCodec::codecForName("UTF-8") : QTextCodec::codecForLocale())
+    , m_process(parent->instance()->getJavaVersion().defaultsToUtf8() ? QStringConverter::Utf8 : QStringConverter::System)
 {
     if (parent->instance()->settings()->get("CloseAfterLaunch").toBool()) {
         static const QRegularExpression s_settingUser(".*Setting user.+", QRegularExpression::CaseInsensitiveOption);

launcher/ui/dialogs/AboutDialog.cpp

@@ -48,9 +48,12 @@ namespace {
 QString getCreditsHtml()
 {
     QFile dataFile(":/documents/credits.html");
-    dataFile.open(QIODevice::ReadOnly);
-
+    if (!dataFile.open(QIODevice::ReadOnly)) {
+        qWarning() << "Failed to open file '" << dataFile.fileName() << "' for reading!";
+        return {};
+    }
     QString fileContent = QString::fromUtf8(dataFile.readAll());
+    dataFile.close();
 
     return fileContent.arg(QObject::tr("%1 Developers").arg(BuildConfig.LAUNCHER_DISPLAYNAME), QObject::tr("MultiMC Developers"),
                            QObject::tr("With special thanks to"));