Formatting

src/blackcore/data/vatsimsetup.cpp

@@ -8,7 +8,6 @@
  */
 
 #include "blackcore/data/vatsimsetup.h"
-
 #include <QStringList>
 
 using namespace BlackMisc;
@@ -66,15 +65,12 @@ namespace BlackCore
             if (index.isMyself()) { return CVariant::from(*this); }
             if (ITimestampBased::canHandleIndex(index)) { return ITimestampBased::propertyByIndex(index); }
 
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
-            case IndexFsdServers:
+            case IndexFsdServers: return CVariant::fromValue(this->m_fsdServers);
-                return CVariant::fromValue(this->m_fsdServers);
+            case IndexDataFiles:  return CVariant::fromValue(this->m_dataFileUrls);
-            case IndexDataFiles:
+            default: return CValueObject::propertyByIndex(index);
-                return CVariant::fromValue(this->m_dataFileUrls);
-            default:
-                return CValueObject::propertyByIndex(index);
             }
         }
 
@@ -87,18 +83,12 @@ namespace BlackCore
                 return;
             }
 
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
-            case IndexFsdServers:
+            case IndexFsdServers: this->m_fsdServers = variant.value<CServerList>(); break;
-                this->m_fsdServers = variant.value<CServerList>();
+            case IndexDataFiles:  this->m_dataFileUrls = variant.value<CUrlList>();  break;
-                break;
+            default: CValueObject::setPropertyByIndex(index, variant); break;
-            case IndexDataFiles:
-                this->m_dataFileUrls = variant.value<CUrlList>();
-                break;
-            default:
-                CValueObject::setPropertyByIndex(index, variant);
-                break;
             }
         }
     } // ns

src/blackcore/db/databaseauthentication.cpp

@@ -89,8 +89,8 @@ namespace BlackCore
 
             QString query = params.toString();
             const QNetworkRequest request(CNetworkUtils::getNetworkRequest(url, CNetworkUtils::PostUrlEncoded));
-            sApp->postToNetwork(request, query.toUtf8(), { this, &CDatabaseAuthenticationService::parseServerResponse});
+            sApp->postToNetwork(request, CApplication::NoLogRequestId, query.toUtf8(), { this, &CDatabaseAuthenticationService::parseServerResponse});
-            QString rm("Sent request to authentication server %1");
+            static const QString rm("Sent request to authentication server '%1'");
             msgs.push_back(CStatusMessage(cats, CStatusMessage::SeverityInfo, rm.arg(url.toQString())));
             return msgs;
         }

src/blackcore/db/databasereader.cpp

@@ -601,10 +601,8 @@ namespace BlackCore
             Q_ASSERT_X(CEntityFlags::isSingleEntity(entity), Q_FUNC_INFO, "needs single entity");
             switch (mode)
             {
-            case CDbFlags::Shared:
+            case CDbFlags::Shared: return CDbInfo::entityToSharedName(entity);
-                return CDbInfo::entityToSharedName(entity);
+            case CDbFlags::SharedInfoOnly: return CDbInfo::sharedInfoFileName();
-            case CDbFlags::SharedInfoOnly:
-                return CDbInfo::sharedInfoFileName();
             default:
             case CDbFlags::DbReading:
                 return CDbInfo::entityToServiceName(entity);
@@ -613,7 +611,7 @@ namespace BlackCore
 
         QString CDatabaseReader::dateTimeToDbLatestTs(const QDateTime &ts)
         {
-            if (!ts.isValid()) return "";
+            if (!ts.isValid()) { return ""; }
             return ts.toUTC().toString(Qt::ISODate);
         }
 

src/blackcore/threadedreader.h

@@ -73,10 +73,11 @@ namespace BlackCore
         void pauseReader();
 
         //! Used in unit test
+        //! \remark needs to be done before started in different thread
         void markAsUsedInUnitTest() { m_unitTest = true; }
 
     protected:
-        mutable QReadWriteLock m_lock {QReadWriteLock::Recursive}; //!< lock which can be used from the derived classes
+        mutable QReadWriteLock m_lock { QReadWriteLock::Recursive }; //!< lock which can be used from the derived classes
 
         //! Constructor
         CThreadedReader(QObject *owner, const QString &name);
@@ -97,7 +98,7 @@ namespace BlackCore
         //! This method does the actual work in the derived class
         virtual void doWorkImpl() {}
 
-        //! Still enabled etc.
+        //! Still enabled etc.?
         bool doWorkCheck() const;
 
         //! Use this to log inconsistent data

src/blackgui/guiapplication.cpp

@@ -625,8 +625,10 @@ namespace BlackGui
     {
         if (CBuildConfig::isRunningOnWindowsNtPlatform())
         {
-            QMessageBox::information(nullptr, QGuiApplication::applicationDisplayName(),
+            QMessageBox::information(nullptr,
-                                     QGuiApplication::applicationDisplayName() + ' ' + QCoreApplication::applicationVersion());
+                                     QGuiApplication::applicationDisplayName(),
+                                     QGuiApplication::applicationDisplayName() + ' ' +
+                                     QCoreApplication::applicationVersion());
         }
         else
         {

src/blackmisc/db/datastoreobjectlist.cpp

@@ -227,6 +227,5 @@ namespace BlackMisc
         template class BLACKMISC_EXPORT_DEFINE_TEMPLATE IDatastoreObjectList<BlackMisc::CCountry, BlackMisc::CCountryList, QString>;
         template class BLACKMISC_EXPORT_DEFINE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CAirport, BlackMisc::Aviation::CAirportList, int>;
         //! \endcond
-
     } // ns
 } // ns

src/blackmisc/db/datastoreobjectlist.h

@@ -82,14 +82,13 @@ namespace BlackMisc
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CLivery, BlackMisc::Aviation::CLiveryList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CAircraftIcaoCode, BlackMisc::Aviation::CAircraftIcaoCodeList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CAirlineIcaoCode, BlackMisc::Aviation::CAirlineIcaoCodeList, int>;
+        extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CAirport, BlackMisc::Aviation::CAirportList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Db::CDbInfo, BlackMisc::Db::CDbInfoList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Db::CDistribution, BlackMisc::Db::CDistributionList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Simulation::CAircraftModel, BlackMisc::Simulation::CAircraftModelList, int>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Simulation::CDistributor, BlackMisc::Simulation::CDistributorList, QString>;
         extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::CCountry, BlackMisc::CCountryList, QString>;
-        extern template class BLACKMISC_EXPORT_DECLARE_TEMPLATE IDatastoreObjectList<BlackMisc::Aviation::CAirport, BlackMisc::Aviation::CAirportList, int>;
         //! \endcond
-
     } // ns
 } //ns
 

src/blackmisc/network/clientlist.cpp

@@ -7,23 +7,15 @@
  * contained in the LICENSE file.
  */
 
-#include "blackmisc/network/clientlist.h"
+#include "clientlist.h"
-#include "blackmisc/metaclassprivate.h"
-
-#include <QString>
-#include <tuple>
-
-using namespace BlackMisc::Aviation;
 
 namespace BlackMisc
 {
     namespace Network
     {
-
         CClientList::CClientList() { }
 
         CClientList::CClientList(const CSequence &other) : CSequence<CClient>(other)
         { }
-
     } // namespace
 } // namespace

src/blackmisc/network/clientlist.h

@@ -18,7 +18,6 @@
 #include "blackmisc/network/client.h"
 #include "blackmisc/sequence.h"
 #include "blackmisc/variant.h"
-
 #include <QMetaType>
 
 namespace BlackMisc
@@ -39,7 +38,6 @@ namespace BlackMisc
 
             //! Construct from a base class object.
             CClientList(const CSequence &other);
-
         };
     } //namespace
 } // namespace

src/blackmisc/network/rolelist.h

@@ -58,7 +58,6 @@ namespace BlackMisc
             //! From our database JSON format
             static CRoleList fromDatabaseJson(const QJsonArray &array);
         };
-
     } //namespace
 } // namespace
 

src/blackmisc/network/url.cpp

@@ -234,48 +234,32 @@ namespace BlackMisc
             return q;
         }
 
-        CVariant CUrl::propertyByIndex(const BlackMisc::CPropertyIndex &index) const
+        CVariant CUrl::propertyByIndex(const CPropertyIndex &index) const
         {
             if (index.isMyself()) { return CVariant::from(*this); }
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
-            case IndexHost:
+            case IndexHost:   return CVariant::fromValue(this->m_host);
-                return CVariant::fromValue(this->m_host);
+            case IndexPort:   return CVariant::fromValue(this->m_port);
-            case IndexPort:
+            case IndexScheme: return CVariant::fromValue(this->m_scheme);
-                return CVariant::fromValue(this->m_port);
+            case IndexPath:   return CVariant::fromValue(this->m_path);
-            case IndexScheme:
+            default:          return CValueObject::propertyByIndex(index);
-                return CVariant::fromValue(this->m_scheme);
-            case IndexPath:
-                return CVariant::fromValue(this->m_path);
-            default:
-                return CValueObject::propertyByIndex(index);
             }
         }
 
         void CUrl::setPropertyByIndex(const CPropertyIndex &index, const CVariant &variant)
         {
             if (index.isMyself()) { (*this) = variant.to<CUrl>(); return; }
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
-            case IndexHost:
+            case IndexHost: this->setHost(variant.value<QString>()); break;
-                this->setHost(variant.value<QString>());
+            case IndexPort: this->setPort(variant.value<qint32>());  break;
-                break;
+            case IndexPath: this->setPath(variant.value<QString>()); break;
-            case IndexPort:
+            case IndexScheme: this->setScheme(variant.value<QString>()); break;
-                this->setPort(variant.value<qint32>());
+            default: CValueObject::setPropertyByIndex(index, variant); break;
-                break;
-            case IndexPath:
-                this->setPath(variant.value<QString>());
-                break;
-            case IndexScheme:
-                this->setScheme(variant.value<QString>());
-                break;
-            default:
-                CValueObject::setPropertyByIndex(index, variant);
-                break;
             }
         }
-
     } // namespace
 } // namespace

src/blackmisc/network/url.h

@@ -9,8 +9,8 @@
 
 //! \file
 
-#ifndef BLACKMISC_NETWORK_NETWORKLOCATION_H
+#ifndef BLACKMISC_NETWORK_URL_H
-#define BLACKMISC_NETWORK_NETWORKLOCATION_H
+#define BLACKMISC_NETWORK_URL_H
 
 #include "blackmisc/blackmiscexport.h"
 #include "blackmisc/metaclass.h"

src/blackmisc/network/urllist.h

@@ -9,8 +9,8 @@
 
 //! \file
 
-#ifndef BLACKMISC_NETWORK_NETWORKLOCATIONLIST_H
+#ifndef BLACKMISC_NETWORK_URLLIST_H
-#define BLACKMISC_NETWORK_NETWORKLOCATIONLIST_H
+#define BLACKMISC_NETWORK_URLLIST_H
 
 #include "blackmisc/blackmiscexport.h"
 #include "blackmisc/collection.h"
@@ -26,7 +26,7 @@ namespace BlackMisc
 {
     namespace Network
     {
-        //! Value object encapsulating a list of servers.
+        //! Value object encapsulating a list of URLs.
         class BLACKMISC_EXPORT CUrlList :
             public CSequence<CUrl>,
             public BlackMisc::Mixin::MetaType<CUrlList>

src/blackmisc/timestampbased.cpp

@@ -174,7 +174,7 @@ namespace BlackMisc
     bool ITimestampBased::canHandleIndex(const CPropertyIndex &index)
     {
         if (index.isEmpty()) { return false; }
-        int i = index.frontCasted<int>();
+        const int i = index.frontCasted<int>();
         return (i >= static_cast<int>(IndexUtcTimestamp)) && (i <= static_cast<int>(IndexMSecsSinceEpoch));
     }
 
@@ -182,7 +182,7 @@ namespace BlackMisc
     {
         if (!index.isEmpty())
         {
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
             case IndexUtcTimestamp:
@@ -212,7 +212,7 @@ namespace BlackMisc
     {
         if (!index.isEmpty())
         {
-            ColumnIndex i = index.frontCasted<ColumnIndex>();
+            const ColumnIndex i = index.frontCasted<ColumnIndex>();
             switch (i)
             {
             case IndexUtcTimestamp: