Bladeren bron

initialize all variables

done with clang-tidy

run-clang-tidy -p build -config="{Checks: '*', CheckOptions: [{key: UseAssignment, value: true}]}" -checks='-*,cppcoreguidelines-init-variables' -fix

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Matthieu Gallien 3 jaren geleden
bovenliggende
commit
b4a8981c03

+ 2 - 2
src/3rdparty/kirigami/wheelhandler.cpp

@@ -351,8 +351,8 @@ bool WheelHandler::scrollFlickable(QPointF pixelDelta, QPointF angleDelta, Qt::K
 
     const qreal xTicks = angleDelta.x() / 120;
     const qreal yTicks = angleDelta.y() / 120;
-    qreal xChange;
-    qreal yChange;
+    qreal xChange = NAN;
+    qreal yChange = NAN;
     bool scrolled = false;
 
     // Scroll X

+ 1 - 1
src/gui/cloudproviders/cloudprovidermanager.cpp

@@ -26,7 +26,7 @@ CloudProvidersProviderExporter *_providerExporter;
 void on_name_acquired (GDBusConnection *connection, const gchar *name, gpointer user_data)
 {
     Q_UNUSED(name);
-    CloudProviderManager *self;
+    CloudProviderManager *self = nullptr;
     self = static_cast<CloudProviderManager*>(user_data);
     _providerExporter = cloud_providers_provider_exporter_new(connection, LIBCLOUDPROVIDERS_DBUS_BUS_NAME, LIBCLOUDPROVIDERS_DBUS_OBJECT_PATH);
     cloud_providers_provider_exporter_set_name (_providerExporter, APPLICATION_NAME);

+ 6 - 6
src/gui/cloudproviders/cloudproviderwrapper.cpp

@@ -34,8 +34,8 @@ GSimpleActionGroup *actionGroup = nullptr;
 CloudProviderWrapper::CloudProviderWrapper(QObject *parent, Folder *folder, int folderId, CloudProvidersProviderExporter* cloudprovider) : QObject(parent)
   , _folder(folder)
 {
-    GMenuModel *model;
-    GActionGroup *action_group;
+    GMenuModel *model = nullptr;
+    GActionGroup *action_group = nullptr;
     QString accountName = QString("Folder/%1").arg(folderId);
 
     _cloudProvider = CLOUD_PROVIDERS_PROVIDER_EXPORTER(cloudprovider);
@@ -147,7 +147,7 @@ void CloudProviderWrapper::slotUpdateProgress(const QString &folder, const Progr
 
     if (!progress._lastCompletedItem.isEmpty()
             && shouldShowInRecentsMenu(progress._lastCompletedItem)) {
-        GMenuItem* item;
+        GMenuItem* item = nullptr;
         g_menu_remove_all (G_MENU(_recentMenu));
         if(!_recentlyChanged.isEmpty()) {
             QList<QPair<QString, QString>>::iterator i;
@@ -208,8 +208,8 @@ void CloudProviderWrapper::slotSyncFinished(const SyncResult &result)
 
 GMenuModel* CloudProviderWrapper::getMenuModel() {
 
-    GMenu* section;
-    GMenuItem* item;
+    GMenu* section = nullptr;
+    GMenuItem* item = nullptr;
     QString item_label;
 
     _mainMenu = g_menu_new();
@@ -314,7 +314,7 @@ activate_action_pause (GSimpleAction *action,
 {
     Q_UNUSED(parameter);
     auto *self = static_cast<CloudProviderWrapper*>(user_data);
-    GVariant *old_state, *new_state;
+    GVariant *old_state = nullptr, *new_state = nullptr;
 
     old_state = g_action_get_state (G_ACTION (action));
     new_state = g_variant_new_boolean (!(bool)g_variant_get_boolean (old_state));

+ 1 - 1
test/teste2efiletransfer.cpp

@@ -41,7 +41,7 @@ private slots:
     {
         {
             EndToEndTestHelper _helper;
-            OCC::Folder *_testFolder;
+            OCC::Folder *_testFolder = nullptr;
 
             QSignalSpy accountReady(&_helper, &EndToEndTestHelper::accountReady);
             _helper.startAccountConfig();