Addresses Clément's review comments

Infomaniak · Oct 14, 2024 · 2649589 · 2649589
1 parent 805d338
commit 2649589
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 13 deletions.
diff --git a/src/libcommon/info/errorinfo.cpp b/src/libcommon/info/errorinfo.cpp
@@ -20,10 +20,7 @@
 
 namespace KDC {
 
-ErrorInfo::ErrorInfo() :
-    _time(0), _level(ErrorLevel::Unknown), _syncDbId(0), _exitCode(ExitCode::Unknown), _exitCause(ExitCause::Unknown),
-    _nodeType(NodeType::Unknown), _conflictType(ConflictType::None), _inconsistencyType(InconsistencyType::None),
-    _cancelType(CancelType::None) {}
+ErrorInfo::ErrorInfo() {}
 
 ErrorInfo::ErrorInfo(qint64 time, ErrorLevel level, const QString &functionName, int syncDbId, const QString &workerName,
                      ExitCode exitCode, ExitCause exitCause, const QString &localNodeId, const QString &remoteNodeId,

diff --git a/src/libcommon/info/errorinfo.h b/src/libcommon/info/errorinfo.h
@@ -81,21 +81,21 @@ class ErrorInfo {
 
     private:
         int _dbId{0};
-        qint64 _time;
-        ErrorLevel _level;
+        qint64 _time{0};
+        ErrorLevel _level{ErrorLevel::Unknown};
         QString _functionName;
-        int _syncDbId;
+        int _syncDbId{0};
         QString _workerName;
-        ExitCode _exitCode;
-        ExitCause _exitCause;
+        ExitCode _exitCode{ExitCode::Unknown};
+        ExitCause _exitCause{ExitCause::Unknown};
         QString _localNodeId;
         QString _remoteNodeId;
-        NodeType _nodeType;
+        NodeType _nodeType{NodeType::Unknown};
         QString _path;
         QString _destinationPath;
-        ConflictType _conflictType;
-        InconsistencyType _inconsistencyType;
-        CancelType _cancelType;
+        ConflictType _conflictType{ConflictType::None};
+        InconsistencyType _inconsistencyType{InconsistencyType::None};
+        CancelType _cancelType{CancelType::None};
         bool _autoResolved{false};
 };