From 8f590848bde053e0e2eaca492222b9537e5a4e56 Mon Sep 17 00:00:00 2001
From: Adriaan de Groot <groot@kde.org>
Date: Wed, 11 Nov 2020 12:45:19 +0100
Subject: [PATCH] [plasmalnf] Simplify code

- Look-and-Feel job has no status to speak of
- Coding style
---
 src/modules/plasmalnf/PlasmaLnfJob.cpp | 17 ++++-------------
 src/modules/plasmalnf/PlasmaLnfJob.h   |  1 -
 2 files changed, 4 insertions(+), 14 deletions(-)

diff --git a/src/modules/plasmalnf/PlasmaLnfJob.cpp b/src/modules/plasmalnf/PlasmaLnfJob.cpp
index 0721bcd88..7ffe3aff7 100644
--- a/src/modules/plasmalnf/PlasmaLnfJob.cpp
+++ b/src/modules/plasmalnf/PlasmaLnfJob.cpp
@@ -20,30 +20,19 @@ PlasmaLnfJob::PlasmaLnfJob( const QString& lnfPath, const QString& id )
 {
 }
 
-
 PlasmaLnfJob::~PlasmaLnfJob() {}
 
-
 QString
 PlasmaLnfJob::prettyName() const
 {
     return tr( "Plasma Look-and-Feel Job" );
 }
 
-QString
-PlasmaLnfJob::prettyStatusMessage() const
-{
-    return prettyName();
-}
-
-
 Calamares::JobResult
 PlasmaLnfJob::exec()
 {
-    cDebug() << "Plasma Look-and-Feel Job";
-
-    auto system = CalamaresUtils::System::instance();
-    Calamares::GlobalStorage* gs = Calamares::JobQueue::instance()->globalStorage();
+    auto* system = CalamaresUtils::System::instance();
+    auto* gs = Calamares::JobQueue::instance()->globalStorage();
 
     QStringList command( { "sudo",
                            "-E",
@@ -59,8 +48,10 @@ PlasmaLnfJob::exec()
 
     int r = system->targetEnvCall( command );
     if ( r )
+    {
         return Calamares::JobResult::error( tr( "Could not select KDE Plasma Look-and-Feel package" ),
                                             tr( "Could not select KDE Plasma Look-and-Feel package" ) );
+    }
 
     return Calamares::JobResult::ok();
 }
diff --git a/src/modules/plasmalnf/PlasmaLnfJob.h b/src/modules/plasmalnf/PlasmaLnfJob.h
index 83360434e..4eaf81014 100644
--- a/src/modules/plasmalnf/PlasmaLnfJob.h
+++ b/src/modules/plasmalnf/PlasmaLnfJob.h
@@ -24,7 +24,6 @@ public:
     ~PlasmaLnfJob() override;
 
     QString prettyName() const override;
-    QString prettyStatusMessage() const override;
 
     Calamares::JobResult exec() override;