Enable the modernize-use-using check on clang-tidy
authorKevin Ottens <kevin.ottens@nextcloud.com>
Wed, 12 Aug 2020 13:23:11 +0000 (15:23 +0200)
committerKevin Ottens <kevin.ottens@nextcloud.com>
Wed, 12 Aug 2020 16:18:57 +0000 (18:18 +0200)
Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>
30 files changed:
.clang-tidy
shell_integration/dolphin/ownclouddolphinoverlayplugin.cpp
src/3rdparty/qtokenizer/qtokenizer.h
src/3rdparty/qtsingleapplication/qtlocalpeer.cpp
src/cmd/netrcparser.h
src/csync/csync.h
src/csync/csync_exclude.h
src/csync/csync_update.h
src/csync/csync_util.cpp
src/csync/std/c_private.h
src/csync/vio/csync_vio.h
src/csync/vio/csync_vio_local_unix.cpp
src/csync/vio/csync_vio_local_win.cpp
src/gui/accountstate.h
src/gui/cloudproviders/cloudproviderwrapper.h
src/gui/folder.h
src/gui/sharee.h
src/gui/sharemanager.h
src/gui/socketapi.h
src/gui/tray/ActivityData.h
src/libsync/account.h
src/libsync/accountfwd.h
src/libsync/configfile.h
src/libsync/syncfileitem.h
src/libsync/syncfilestatustracker.h
test/csync/csync_tests/check_csync_exclude.cpp
test/csync/vio_tests/check_vio_ext.cpp
test/testconcaturl.cpp
test/testnextcloudpropagator.cpp
test/testsyncjournaldb.cpp

index cda2510ad1946a949b36f3a55b136032ed05e595..34de4128fa414c40e8c646b8fdf7a7243a5d8c70 100644 (file)
@@ -20,6 +20,7 @@ Checks: '-*,
     modernize-user-override,
     modernize-use-transparent-functors,
     modernize-use-uncaught-exceptions,
+    modernize-use-using,
 '
 WarningsAsErrors: '*'
 HeaderFilterRegex: '.*'
index d89aa917404ff7fbcb9e0b7d36950b98d605e65a..f11e7ac0b39e60dd9d1443fd772add0706fdd538 100644 (file)
@@ -30,7 +30,7 @@ class OwncloudDolphinPlugin : public KOverlayIconPlugin
     Q_PLUGIN_METADATA(IID "com.owncloud.ovarlayiconplugin" FILE "ownclouddolphinoverlayplugin.json")
     Q_OBJECT
 
-    typedef QHash<QByteArray, QByteArray> StatusMap;
+    using StatusMap = QHash<QByteArray, QByteArray>;
     StatusMap m_status;
 
 public:
index 95f1e3212b13c7ae6c9690aefc53bc9632c6737f..b8de15a7ddfcb195da53f5f9c139c3486561c90f 100644 (file)
@@ -50,7 +50,7 @@ QT_BEGIN_NAMESPACE
 
 template <class T, class const_iterator>
 struct QTokenizerPrivate {
-    typedef typename T::value_type char_type;
+    using char_type = typename T::value_type;
 
     struct State {
         bool inQuote = false;
@@ -113,7 +113,7 @@ struct QTokenizerPrivate {
 template <class T, class const_iterator>
 class QTokenizer {
 public:
-    typedef typename T::value_type char_type;
+    using char_type = typename T::value_type;
 
     /*!
        \class QTokenizer
@@ -253,9 +253,9 @@ public:
     }
 };
 
-typedef QTokenizer<QByteArray, QByteArray::const_iterator> QByteArrayTokenizer;
-typedef QTokenizer<std::string, std::string::const_iterator> StringTokenizer;
-typedef QTokenizer<std::wstring, std::wstring::const_iterator> WStringTokenizer;
+using QByteArrayTokenizer = QTokenizer<QByteArray, QByteArray::const_iterator>;
+using StringTokenizer = QTokenizer<std::string, std::string::const_iterator>;
+using WStringTokenizer = QTokenizer<std::wstring, std::wstring::const_iterator>;
 
 QT_END_NAMESPACE
 
index 4630abd3f3c0c51cfe3d43f4b39614c703f3134b..c563e2f2864d756e84cc59c63739c593b84540dd 100644 (file)
@@ -36,7 +36,7 @@
 #if defined(Q_OS_WIN)
 #include <QLibrary>
 #include <qt_windows.h>
-typedef BOOL(WINAPI*PProcessIdToSessionId)(DWORD,DWORD*);
+using PProcessIdToSessionId = BOOL (WINAPI*)(DWORD, DWORD*);
 static PProcessIdToSessionId pProcessIdToSessionId = 0;
 #endif
 
index 028ee13806f2d5e5bd0642052f12dede706119f0..8ad3465519bd61fd1e13b370cfae777d5e425611 100644 (file)
@@ -27,7 +27,7 @@ namespace OCC {
 class NetrcParser
 {
 public:
-    typedef QPair<QString, QString> LoginPair;
+    using LoginPair = QPair<QString, QString>;
 
     NetrcParser(const QString &file = QString());
     bool parse();
index eab9699b185f257ba542bf69323f4a85513eb784..c57cd6f3d688c786b08fbeff157ef4a041fada50 100644 (file)
@@ -95,7 +95,7 @@ enum csync_status_codes_e {
     CSYNC_STATUS_INDIVIDUAL_CANNOT_ENCODE
 };
 
-typedef enum csync_status_codes_e CSYNC_STATUS;
+using CSYNC_STATUS = enum csync_status_codes_e;
 
 #ifndef likely
 # define likely(x) (x)
@@ -146,7 +146,7 @@ enum ItemType {
 // currently specified at https://github.com/owncloud/core/issues/8322 are 9 to 10
 #define REMOTE_PERM_BUF_SIZE 15
 
-typedef struct csync_file_stat_s csync_file_stat_t;
+using csync_file_stat_t = struct csync_file_stat_s;
 
 struct OCSYNC_EXPORT csync_file_stat_s {
   time_t modtime = 0;
@@ -193,26 +193,19 @@ struct OCSYNC_EXPORT csync_file_stat_s {
 /**
  * csync handle
  */
-typedef struct csync_s CSYNC;
+using CSYNC = struct csync_s;
 
-typedef int (*csync_auth_callback) (const char *prompt, char *buf, size_t len,
-    int echo, int verify, void *userdata);
+using csync_auth_callback = int (*)(const char *prompt, char *buf, size_t len, int echo, int verify, void *userdata);
 
-typedef void (*csync_update_callback) (bool local,
-                                    const char *dirUrl,
-                                    void *userdata);
+using csync_update_callback = void (*)(bool local, const char *dirUrl, void *userdata);
 
-typedef void csync_vio_handle_t;
-typedef csync_vio_handle_t* (*csync_vio_opendir_hook) (const char *url,
-                                    void *userdata);
-typedef std::unique_ptr<csync_file_stat_t> (*csync_vio_readdir_hook) (csync_vio_handle_t *dhhandle,
-                                                              void *userdata);
-typedef void (*csync_vio_closedir_hook) (csync_vio_handle_t *dhhandle,
-                                                              void *userdata);
+using csync_vio_handle_t = void;
+using csync_vio_opendir_hook = csync_vio_handle_t *(*)(const char *url, void *userdata);
+using csync_vio_readdir_hook = std::unique_ptr<csync_file_stat_t> (*)(csync_vio_handle_t *dhandle, void *userdata);
+using csync_vio_closedir_hook = void (*)(csync_vio_handle_t *dhandle, void *userdata);
 
 /* Compute the checksum of the given \a checksumTypeId for \a path. */
-typedef QByteArray (*csync_checksum_hook)(
-    const QByteArray &path, const QByteArray &otherChecksumHeader, void *userdata);
+using csync_checksum_hook = QByteArray (*)(const QByteArray &path, const QByteArray &otherChecksumHeader, void *userdata);
 
 /**
  * @brief Update detection
index 0b11477917bd066b87e8257032e465d65713f73f..2ec59eb30eb56c0787a189730156dfe4a02d07ce 100644 (file)
@@ -45,7 +45,7 @@ enum csync_exclude_type_e {
   CSYNC_FILE_EXCLUDE_CONFLICT,
   CSYNC_FILE_EXCLUDE_CANNOT_ENCODE
 };
-typedef enum csync_exclude_type_e CSYNC_EXCLUDE_TYPE;
+using CSYNC_EXCLUDE_TYPE = enum csync_exclude_type_e;
 
 class ExcludedFilesTest;
 
index 1b4c50995cbbbdab8fc34824f30996911a6d7d46..66afd7c0784c2b8ecce7c9f66a85928e37bc7c76 100644 (file)
@@ -37,7 +37,7 @@
  * @{
  */
 
-typedef int (*csync_walker_fn) (CSYNC *ctx, std::unique_ptr<csync_file_stat_t> fs);
+using csync_walker_fn = int (*)(CSYNC *ctx, std::unique_ptr<csync_file_stat_t> fs);
 
 /**
  * @brief The walker function to use in the file tree walker.
index a65c2b4270c3e64510dd999aefcde592601c1210..60bc348ec583f23120efa8c2b48b6bca920a288a 100644 (file)
 Q_LOGGING_CATEGORY(lcCSyncUtils, "nextcloud.sync.csync.utils", QtInfoMsg)
 
 
-typedef struct {
+using _instr_code_struct = struct {
   const char *instr_str;
   enum csync_instructions_e instr_code;
-} _instr_code_struct;
+};
 
 static const _instr_code_struct _instr[] =
 {
index 76f25e453807cc48789d17db2317723678a90e1c..ba4f986e1e708e62dee4f9d595a6fc4c532833d0 100644 (file)
 
 
 #ifdef _WIN32
-typedef struct stat64 csync_stat_t;
+typedef struct stat64 csync_stat_t; // NOLINT this is sometimes compiled in C mode
 #define _FILE_OFFSET_BITS 64
 #else
-typedef struct stat csync_stat_t;
+typedef struct stat csync_stat_t; // NOLINT this is sometimes compiled in C mode
 #endif
 
 #ifndef O_NOATIME
@@ -112,7 +112,7 @@ typedef struct stat csync_stat_t;
 #endif
 
 #if defined _WIN32 && defined _UNICODE
-typedef  wchar_t         mbchar_t;
+typedef  wchar_t         mbchar_t; // NOLINT this is sometimes compiled in C mode
 #define _topen           _wopen
 #define _tdirent         _wdirent
 #define _topendir        _wopendir
@@ -133,7 +133,7 @@ typedef  wchar_t         mbchar_t;
 #define _tchdir          _wchdir
 #define _tgetcwd         _wgetcwd
 #else
-typedef char           mbchar_t;
+typedef char           mbchar_t; // NOLINT this is sometimes compiled in C mode
 #define _tdirent       dirent
 #define _topen         open
 #define _topendir      opendir
index 7d3a2e0b4eea37b3538eda7e9198d8285b02596c..54b71cb7a2d48c4c883f17d031fad71466ff44fa 100644 (file)
 #include "csync.h"
 #include "csync_private.h"
 
-typedef struct fhandle_s {
+using fhandle_t = struct fhandle_s {
+
   int fd;
-} fhandle_t;
+
+};
 
 csync_vio_handle_t *csync_vio_opendir(CSYNC *ctx, const char *name);
 int csync_vio_closedir(CSYNC *ctx, csync_vio_handle_t *dhandle);
index bac8801a77a2e8a6ba1fa78241958543d0bc3897..8f704dd24ee3ee1ae1ec3ec1d85762ea1662d05a 100644 (file)
@@ -43,10 +43,10 @@ Q_LOGGING_CATEGORY(lcCSyncVIOLocal, "nextcloud.sync.csync.vio_local", QtInfoMsg)
  * directory functions
  */
 
-typedef struct dhandle_s {
+using dhandle_t = struct dhandle_s {
   DIR *dh;
   char *path;
-} dhandle_t;
+};
 
 static int _csync_vio_local_stat_mb(const mbchar_t *wuri, csync_file_stat_t *buf);
 
index dceabcdaa4a053614e9a430642bf0178429f93b3..38446289dc1ddb72f322663b577a26bfcba60461 100644 (file)
@@ -43,12 +43,12 @@ Q_LOGGING_CATEGORY(lcCSyncVIOLocal, "nextcloud.sync.csync.vio_local", QtInfoMsg)
  * directory functions
  */
 
-typedef struct dhandle_s {
+using dhandle_t = struct dhandle_s {
   WIN32_FIND_DATA ffd;
   HANDLE hFind;
   int firstFind;
   mbchar_t *path; // Always ends with '\'
-} dhandle_t;
+};
 
 static int _csync_vio_local_stat_mb(const mbchar_t *uri, csync_file_stat_t *buf);
 
index f1209a47a7f798961c8dd791c6f8c11d15ae4d6f..7e390e16703ca12dfacb3ed1bce3a781ffbb0f5a 100644 (file)
@@ -32,8 +32,8 @@ class Account;
 class AccountApp;
 class RemoteWipe;
 
-typedef QExplicitlySharedDataPointer<AccountState> AccountStatePtr;
-typedef QList<AccountApp*> AccountAppList;
+using AccountStatePtr = QExplicitlySharedDataPointer<AccountState>;
+using AccountAppList = QList<AccountApp *>;
 
 /**
  * @brief Extra info about an ownCloud server account.
@@ -75,7 +75,7 @@ public:
     };
 
     /// The actual current connectivity status.
-    typedef ConnectionValidator::Status ConnectionStatus;
+    using ConnectionStatus = ConnectionValidator::Status;
 
     /// Use the account as parent
     explicit AccountState(AccountPtr account);
index e3a46cc435d9f43e674a4b40d9818edc5e338924..22caf362985c5e8a6d6389646a699c0169ccb1f1 100644 (file)
 
 /* Forward declaration required since gio header files interfere with QObject headers */
 struct _CloudProvidersProviderExporter;
-typedef _CloudProvidersProviderExporter CloudProvidersProviderExporter;
+using CloudProvidersProviderExporter = _CloudProvidersProviderExporter;
 struct _CloudProvidersAccountExporter;
-typedef _CloudProvidersAccountExporter CloudProvidersAccountExporter;
+using CloudProvidersAccountExporter = _CloudProvidersAccountExporter;
 struct _GMenuModel;
-typedef _GMenuModel GMenuModel;
+using GMenuModel = _GMenuModel;
 struct _GMenu;
-typedef _GMenu GMenu;
+using GMenu = _GMenu;
 struct _GActionGroup;
-typedef _GActionGroup GActionGroup;
-typedef char gchar;
-typedef void* gpointer;
+using GActionGroup = _GActionGroup;
+using gchar = char;
+using gpointer = void*;
 
 using namespace OCC;
 
index 518353f6a4774392fe58b926d3cafd79dac2b157..72c651b735d22136d6f6708f037802039458e2f8 100644 (file)
@@ -95,8 +95,8 @@ public:
 
     ~Folder();
 
-    typedef QMap<QString, Folder *> Map;
-    typedef QMapIterator<QString, Folder *> MapIterator;
+    using Map = QMap<QString, Folder *>;
+    using MapIterator = QMapIterator<QString, Folder *>;
 
     /**
      * The account the folder is configured on.
index 690ec0047f755a2ba956209bc809903712464c44..c6fbcfe29078363559c354773ea2e1dc6d828dd2 100644 (file)
@@ -66,7 +66,7 @@ class ShareeModel : public QAbstractListModel
 public:
     explicit ShareeModel(const AccountPtr &account, const QString &type, QObject *parent = nullptr);
 
-    typedef QVector<QSharedPointer<Sharee>> ShareeSet; // FIXME: make it a QSet<Sharee> when Sharee can be compared
+    using ShareeSet = QVector<QSharedPointer<Sharee>>; // FIXME: make it a QSet<Sharee> when Sharee can be compared
     void fetch(const QString &search, const ShareeSet &blacklist);
     int rowCount(const QModelIndex &parent = QModelIndex()) const override;
     QVariant data(const QModelIndex &index, int role) const override;
index a7a7a718c93204c2ce1b770a802366107bc262eb..daac8bfc8a1d141b6dfc2f662e080e11d4417f58 100644 (file)
@@ -48,7 +48,7 @@ public:
         TypeCircle = Sharee::Circle
     };
 
-    typedef SharePermissions Permissions;
+    using Permissions = SharePermissions;
 
     /*
      * Constructor for shares
index b31294c5b1450587cdf75cfb7f3b839f793dc2ea..4822629d5bdb516561cef2fc238f50d4267c5712 100644 (file)
@@ -25,7 +25,7 @@
 #include "socketapisocket_mac.h"
 #else
 #include <QLocalServer>
-typedef QLocalServer SocketApiServer;
+using SocketApiServer = QLocalServer;
 #endif
 
 class QUrl;
index f2fc0e34567668165a38006c509e207e00d4493d..be30f77a2ac01390316965838dc00067e41dbcc0 100644 (file)
@@ -45,7 +45,7 @@ public:
 class Activity
 {
 public:
-    typedef QPair<qlonglong, QString> Identifier;
+    using Identifier = QPair<qlonglong, QString>;
 
     enum Type {
         ActivityType,
@@ -92,8 +92,7 @@ bool operator<(const Activity &rhs, const Activity &lhs);
  *
  * A QList based list of Activities
  */
-
-typedef QList<Activity> ActivityList;
+using ActivityList = QList<Activity>;
 }
 
 Q_DECLARE_METATYPE(OCC::Activity::Type)
index 183e54218598e9e558873f11df06b8b45dc7f843..3df7a29a16f75986c5026a2e8439d9bffa90ebac 100644 (file)
@@ -51,7 +51,7 @@ namespace OCC {
 
 class AbstractCredentials;
 class Account;
-typedef QSharedPointer<Account> AccountPtr;
+using AccountPtr = QSharedPointer<Account>;
 class AccessManager;
 class SimpleNetworkJob;
 
index 9345bc1e377851efbadb68c1594a3318f72b8f52..ae87639c10fa1c0104cd314e8572b0b32b5f7af2 100644 (file)
@@ -20,9 +20,9 @@
 namespace OCC {
 
 class Account;
-typedef QSharedPointer<Account> AccountPtr;
+using AccountPtr = QSharedPointer<Account>;
 class AccountState;
-typedef QExplicitlySharedDataPointer<AccountState> AccountStatePtr;
+using AccountStatePtr = QExplicitlySharedDataPointer<AccountState>;
 
 } // namespace OCC
 
index 432720a3a88070822c3cb258400d1c30fe19f9c6..e0629c1df5910adea00e790663c4881812339bae 100644 (file)
@@ -201,7 +201,7 @@ private:
     QString keychainProxyPasswordKey() const;
 
 private:
-    typedef QSharedPointer<AbstractCredentials> SharedCreds;
+    using SharedCreds = QSharedPointer<AbstractCredentials>;
 
     static bool _askedUser;
     static QString _oCVersion;
index 2215dca42ff6611b6aec9ca3d1b291d8d1005a68..adcd4ae4d821df6bd80bd9b632a0e26d61dd5555 100644 (file)
@@ -27,7 +27,7 @@ namespace OCC {
 
 class SyncFileItem;
 class SyncJournalFileRecord;
-typedef QSharedPointer<SyncFileItem> SyncFileItemPtr;
+using SyncFileItemPtr = QSharedPointer<SyncFileItem>;
 
 /**
  * @brief The SyncFileItem class
@@ -254,7 +254,7 @@ inline bool operator<(const SyncFileItemPtr &item1, const SyncFileItemPtr &item2
     return *item1 < *item2;
 }
 
-typedef QVector<SyncFileItemPtr> SyncFileItemVector;
+using SyncFileItemVector = QVector<SyncFileItemPtr>;
 }
 
 Q_DECLARE_METATYPE(OCC::SyncFileItem)
index f4b31a1305998e59dd3412ff31db9471c6a70963..4a6b89f0d02d030d5980a8f226c8cabe63da8b05 100644 (file)
@@ -54,7 +54,7 @@ private:
     struct PathComparator {
         bool operator()( const QString& lhs, const QString& rhs ) const;
     };
-    typedef std::map<QString, SyncFileStatus::SyncFileStatusTag, PathComparator> ProblemsMap;
+    using ProblemsMap = std::map<QString, SyncFileStatus::SyncFileStatusTag, PathComparator>;
     SyncFileStatus::SyncFileStatusTag lookupProblem(const QString &pathToMatch, const ProblemsMap &problemMap);
 
     enum SharedFlag { UnknownShared,
index c052f242498119946f95c2a83edda51c6cbf23ac..304e1c5ce25bf56a9c2b020faea421b48a737aec 100644 (file)
@@ -698,7 +698,7 @@ static void check_csync_exclude_expand_escapes(void **state)
 
 int torture_run_tests(void)
 {
-    typedef ExcludedFilesTest T;
+    using T = ExcludedFilesTest;
 
     const struct CMUnitTest tests[] = {
         cmocka_unit_test_setup_teardown(T::check_csync_exclude_add, T::setup, T::teardown),
index afcb0c87ff642b807f97d79ced6386955f04f9d0..567dbd70bae1fa1e3469ecb7bca561216a544b26 100644 (file)
 
 static mbchar_t wd_buffer[WD_BUFFER_SIZE];
 
-typedef struct {
+using statevar = struct {
     CSYNC *csync;
     char  *result;
     char *ignored_dir;
-} statevar;
+};
 
 /* remove the complete test dir */
 static int wipe_testdir()
index 16f76ed34e20ec879fe9458eab3b05a9926790d4..a6caac8b1c3768916db70c0411e506781a72b99f 100644 (file)
@@ -14,7 +14,7 @@
 
 using namespace OCC;
 
-typedef QList< QPair<QString,QString> > QueryItems;
+using QueryItems = QList<QPair<QString, QString>>;
 
 Q_DECLARE_METATYPE(QueryItems)
 
index 443d3a430330d3ac706a3a72f2883c60a34c77d3..954c665f5f91ccbabf500ab77242638ffcf4c480 100644 (file)
@@ -65,7 +65,7 @@ private slots:
 
     void testParseEtag()
     {
-        typedef QPair<const char*, const char*> Test;
+        using Test = QPair<const char*, const char*>;
         QList<Test> tests;
         tests.append(Test("\"abcd\"", "abcd"));
         tests.append(Test("\"\"", ""));
index 75df2cf5174312e67d8ff1de1bb21288f202b249..61f00fffd4f863d9fa9ff975dfef3b3d4c2af894 100644 (file)
@@ -130,7 +130,7 @@ private slots:
 
     void testDownloadInfo()
     {
-        typedef SyncJournalDb::DownloadInfo Info;
+        using Info = SyncJournalDb::DownloadInfo;
         Info record = _db.getDownloadInfo("nonexistant");
         QVERIFY(!record._valid);