aboutsummaryrefslogtreecommitdiff
path: root/launcher/minecraft/auth
diff options
context:
space:
mode:
authorPandaNinjas <admin@malwarefight.wip.la>2023-06-07 19:37:54 -0400
committerPandaNinjas <admin@malwarefight.wip.la>2023-06-07 19:37:54 -0400
commit318d11481d719cf537ecdc00f8d676494bab22b6 (patch)
tree081a070aeeb4ca0f05fceef19eb64d5075edaf04 /launcher/minecraft/auth
parent5d425ecc025aa1fc4a5292961a5ff1b18a108885 (diff)
downloadPrismLauncher-318d11481d719cf537ecdc00f8d676494bab22b6.tar.gz
PrismLauncher-318d11481d719cf537ecdc00f8d676494bab22b6.tar.bz2
PrismLauncher-318d11481d719cf537ecdc00f8d676494bab22b6.zip
Resolve other switch fallthrough issues
Diffstat (limited to 'launcher/minecraft/auth')
-rw-r--r--launcher/minecraft/auth/AccountList.cpp9
-rw-r--r--launcher/minecraft/auth/Yggdrasil.cpp1
2 files changed, 6 insertions, 4 deletions
diff --git a/launcher/minecraft/auth/AccountList.cpp b/launcher/minecraft/auth/AccountList.cpp
index e454bcc4..184f8e10 100644
--- a/launcher/minecraft/auth/AccountList.cpp
+++ b/launcher/minecraft/auth/AccountList.cpp
@@ -357,11 +357,12 @@ QVariant AccountList::data(const QModelIndex &index, int role) const
return QVariant::fromValue(account);
case Qt::CheckStateRole:
- switch (index.column())
- {
- case ProfileNameColumn:
- return account == m_defaultAccount ? Qt::Checked : Qt::Unchecked;
+ if (index.column() == ProfileNameColumn) {
+ return account == m_defaultAccount ? Qt::Checked : Qt::Unchecked;
+ } else {
+ return QVariant();
}
+
default:
return QVariant();
diff --git a/launcher/minecraft/auth/Yggdrasil.cpp b/launcher/minecraft/auth/Yggdrasil.cpp
index 29978411..d3e7ccdd 100644
--- a/launcher/minecraft/auth/Yggdrasil.cpp
+++ b/launcher/minecraft/auth/Yggdrasil.cpp
@@ -273,6 +273,7 @@ void Yggdrasil::processReply() {
AccountTaskState::STATE_FAILED_GONE,
tr("The Mojang account no longer exists. It may have been migrated to a Microsoft account.")
);
+ return;
}
default:
changeState(