From b32638496ab9838e93ef4d51dc6f6d6f7077695d Mon Sep 17 00:00:00 2001 From: Nate Graham Date: Tue, 12 Apr 2022 10:20:53 -0600 Subject: [PATCH] kcms/colors: remove dead code We go through a bunch of effort to set m_applyToAlien, but then never use it anywhere. It must not be very important here. --- kcms/colors/colors.cpp | 7 ------- kcms/colors/colors.h | 2 -- 2 files changed, 9 deletions(-) diff --git a/kcms/colors/colors.cpp b/kcms/colors/colors.cpp index da5796400..82d1b0c49 100644 --- a/kcms/colors/colors.cpp +++ b/kcms/colors/colors.cpp @@ -329,13 +329,6 @@ void KCMColors::load() loadSelectedColorScheme(); - { - KConfig cfg(QStringLiteral("kcmdisplayrc"), KConfig::NoGlobals); - KConfigGroup group(m_config, "General"); - group = KConfigGroup(&cfg, "X11"); - m_applyToAlien = group.readEntry("exportKDEColors", true); - } - // If need save is true at the end of load() function, it will stay disabled forever. // setSelectedScheme() call due to unexisting scheme name in kdeglobals will trigger a need to save. // this following call ensure the apply button will work properly. diff --git a/kcms/colors/colors.h b/kcms/colors/colors.h index d056600d7..b6f7a2520 100644 --- a/kcms/colors/colors.h +++ b/kcms/colors/colors.h @@ -101,8 +101,6 @@ private: bool m_selectedSchemeDirty = false; bool m_activeSchemeEdited = false; - bool m_applyToAlien = true; - QProcess *m_editDialogProcess = nullptr; KSharedConfigPtr m_config;