From 169b07691391e3cfee09a41ac6a239278d08f283 Mon Sep 17 00:00:00 2001 From: BreadFish64 Date: Thu, 18 Jan 2018 21:02:55 -0600 Subject: [PATCH] use enum and add default case --- src/citra_qt/compatdb.cpp | 6 ++++-- src/citra_qt/compatdb.h | 2 ++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/citra_qt/compatdb.cpp b/src/citra_qt/compatdb.cpp index 8bee64b05..a3e6ba3e2 100644 --- a/src/citra_qt/compatdb.cpp +++ b/src/citra_qt/compatdb.cpp @@ -33,12 +33,12 @@ void CompatDB::Submit() { compatibility->addButton(ui->radioButton_IntroMenu, 4); compatibility->addButton(ui->radioButton_WontBoot, 5); switch (currentId()) { - case 1: + case CompatDBPage::SelectionPage: if (compatibility->checkedId() == -1) { button(NextButton)->setEnabled(false); } break; - case 2: + case CompatDBPage::FinalPage: LOG_DEBUG(Frontend, "Compatibility Rating: %d", compatibility->checkedId()); Core::Telemetry().AddField(Telemetry::FieldType::UserFeedback, "Compatibility", compatibility->checkedId()); @@ -46,6 +46,8 @@ void CompatDB::Submit() { // workaround button(QWizard::CancelButton)->setVisible(false); break; + default: + LOG_ERROR(Frontend, "Unexpected page: %d", currentId()); } } diff --git a/src/citra_qt/compatdb.h b/src/citra_qt/compatdb.h index 0a0f27cca..dfc259b4f 100644 --- a/src/citra_qt/compatdb.h +++ b/src/citra_qt/compatdb.h @@ -25,3 +25,5 @@ private slots: void Submit(); void EnableNext(); }; + +enum CompatDBPage { IntroPage = 0, SelectionPage = 1, FinalPage = 2 };