[libcalamaresui] Apply coding style to remainder

src/libcalamaresui/Branding.cpp

@@ -52,6 +52,8 @@ Branding::instance()
 }
 
 
+// *INDENT-OFF*
+// clang-format off
 const QStringList Branding::s_stringEntryStrings =
 {
     "productName",
@@ -82,15 +84,21 @@ const QStringList Branding::s_styleEntryStrings =
     "sidebarTextSelect",
     "sidebarTextHighlight"
 };
+// clang-format on
+// *INDENT-ON*
 
 const NamedEnumTable<Branding::WindowDimensionUnit>&
 Branding::WindowDimension::suffixes()
 {
     using Unit = Branding::WindowDimensionUnit;
+    // *INDENT-OFF*
+    // clang-format off
     static const NamedEnumTable<Unit> names{
         {"px", Unit::Pixies},
         {"em", Unit::Fonties}
     };
+    // clang-format on
+    // *INDENT-ON*
 
     return names;
 }

src/libcalamaresui/Branding.h

@@ -25,9 +25,9 @@
 
 #include "utils/NamedSuffix.h"
 
+#include <QMap>
 #include <QObject>
 #include <QStringList>
-#include <QMap>
 
 namespace YAML
 {
@@ -79,13 +79,23 @@ public:
     };
 
     /** @brief Setting for how much the main window may expand. */
-    enum class WindowExpansion { Normal, Fullscreen, Fixed } ;
+    enum class WindowExpansion
+    {
+        Normal,
+        Fullscreen,
+        Fixed
+    };
     /** @brief Setting for the main window size.
      *
      * The units are pixels (Pixies) or something-based-on-fontsize (Fonties), which
      * we suffix as "em", e.g. "600px" or "32em".
      */
-    enum class WindowDimensionUnit { None, Pixies, Fonties };
+    enum class WindowDimensionUnit
+    {
+        None,
+        Pixies,
+        Fonties
+    };
     class WindowDimension : public NamedSuffix< WindowDimensionUnit, WindowDimensionUnit::None >
     {
     public:
@@ -93,13 +103,15 @@ public:
         bool isValid() const;
 
         using NamedSuffix::NamedSuffix;
-        WindowDimension( const QString& s ) : NamedSuffix( suffixes(), s ) {}
+        WindowDimension( const QString& s )
+            : NamedSuffix( suffixes(), s )
+        {
+        }
     };
 
     static Branding* instance();
 
-    explicit Branding( const QString& brandingFilePath,
+    explicit Branding( const QString& brandingFilePath, QObject* parent = nullptr );
-                       QObject* parent = nullptr );
 
     /** @brief Complete path of the branding descriptor file. */
     QString descriptorPath() const { return m_descriptorPath; }
@@ -184,11 +196,14 @@ private:
     WindowExpansion m_windowExpansion;
 
     WindowDimension m_windowHeight, m_windowWidth;
-
 };
 
-template<typename U> inline QString operator*(U e) { return Branding::instance()->string( e ); }
+template < typename U >
-
+inline QString operator*( U e )
+{
+    return Branding::instance()->string( e );
 }
 
+}  // namespace Calamares
+
 #endif  // BRANDING_H

src/libcalamaresui/ExecutionViewStep.cpp

@@ -23,10 +23,10 @@
 #include "Branding.h"
 #include "Job.h"
 #include "JobQueue.h"
-#include "modulesystem/Module.h"
-#include "modulesystem/ModuleManager.h"
 #include "Settings.h"
 #include "ViewManager.h"
+#include "modulesystem/Module.h"
+#include "modulesystem/ModuleManager.h"
 
 #include "utils/CalamaresUtilsGui.h"
 #include "utils/Dirs.h"
@@ -117,9 +117,7 @@ ExecutionViewStep::ExecutionViewStep( QObject* parent )
 QString
 ExecutionViewStep::prettyName() const
 {
-    return Calamares::Settings::instance()->isSetupMode()
+    return Calamares::Settings::instance()->isSetupMode() ? tr( "Set up" ) : tr( "Install" );
-        ? tr( "Set up" )
-        : tr( "Install" );
 }
 
 
@@ -176,8 +174,7 @@ ExecutionViewStep::loadQmlV2()
     {
         m_qmlComponent = new QQmlComponent( m_qmlShow->engine(),
                                             QUrl::fromLocalFile( Calamares::Branding::instance()->slideshowPath() ),
-                                            QQmlComponent::CompilationMode::Asynchronous
+                                            QQmlComponent::CompilationMode::Asynchronous );
-                                          );
         connect( m_qmlComponent, &QQmlComponent::statusChanged, this, &ExecutionViewStep::loadQmlV2Complete );
     }
 }
@@ -194,14 +191,17 @@ ExecutionViewStep::loadQmlV2Complete()
         QObject* o = m_qmlComponent->create();
         m_qmlObject = qobject_cast< QQuickItem* >( o );
         if ( !m_qmlObject )
+        {
             delete o;
+        }
         else
         {
             // setContent() is public API, but not documented publicly.
             // It is marked \internal in the Qt sources, but does exactly
             // what is needed: sets up visual parent by replacing the root
             // item, and handling resizes.
-            m_qmlShow->setContent( QUrl::fromLocalFile( Calamares::Branding::instance()->slideshowPath() ), m_qmlComponent, m_qmlObject );
+            m_qmlShow->setContent(
+                QUrl::fromLocalFile( Calamares::Branding::instance()->slideshowPath() ), m_qmlComponent, m_qmlObject );
             if ( ViewManager::instance()->currentStep() == this )
             {
                 // We're alreay visible! Must have been slow QML loading, and we
@@ -236,8 +236,10 @@ ExecutionViewStep::onActivate()
             if ( module->isEmergency() )
             {
                 for ( auto& j : jl )
+                {
                     j->setEmergency( true );
                 }
+            }
             queue->enqueue( jl );
         }
     }
@@ -279,4 +281,4 @@ ExecutionViewStep::onLeave()
     }
 }
 
-} // namespace
+}  // namespace Calamares

src/libcalamaresui/ExecutionViewStep.h

@@ -77,6 +77,6 @@ private:
     void updateFromJobQueue( qreal percent, const QString& message );
 };
 
-}
+}  // namespace Calamares
 
 #endif /* EXECUTIONVIEWSTEP_H */