Merge pull request #12545 from Chocobo1/padding

Reduce padding in class

src/base/bittorrent/torrenthandleimpl.cpp

@@ -132,6 +132,8 @@ CreateTorrentParams::CreateTorrentParams(const AddTorrentParams &params)
     , category(params.category)
     , tags(params.tags)
     , savePath(params.savePath)
+    , uploadLimit(params.uploadLimit)
+    , downloadLimit(params.downloadLimit)
     , disableTempPath(params.disableTempPath)
     , sequential(params.sequential)
     , firstLastPiecePriority(params.firstLastPiecePriority)
@@ -144,8 +146,6 @@ CreateTorrentParams::CreateTorrentParams(const AddTorrentParams &params)
     , paused(params.addPaused == TriStateBool::Undefined
                 ? Session::instance()->isAddTorrentPaused()
                 : params.addPaused == TriStateBool::True)
-    , uploadLimit(params.uploadLimit)
-    , downloadLimit(params.downloadLimit)
     , filePriorities(params.filePriorities)
     , ratioLimit(params.ignoreShareLimits ? TorrentHandleImpl::NO_RATIO_LIMIT : TorrentHandleImpl::USE_GLOBAL_RATIO)
     , seedingTimeLimit(params.ignoreShareLimits ? TorrentHandleImpl::NO_SEEDING_TIME_LIMIT : TorrentHandleImpl::USE_GLOBAL_SEEDING_TIME)
@@ -166,21 +166,16 @@ TorrentHandleImpl::TorrentHandleImpl(Session *session, const lt::torrent_handle
     : QObject(session)
     , m_session(session)
     , m_nativeHandle(nativeHandle)
-    , m_state(TorrentState::Unknown)
-    , m_renameCount(0)
-    , m_useAutoTMM(params.savePath.isEmpty())
     , m_name(params.name)
     , m_savePath(Utils::Fs::toNativePath(params.savePath))
     , m_category(params.category)
     , m_tags(params.tags)
-    , m_hasSeedStatus(params.hasSeedStatus)
     , m_ratioLimit(params.ratioLimit)
     , m_seedingTimeLimit(params.seedingTimeLimit)
+    , m_hasSeedStatus(params.hasSeedStatus)
     , m_tempPathDisabled(params.disableTempPath)
-    , m_fastresumeDataRejected(false)
-    , m_hasMissingFiles(false)
     , m_hasRootFolder(params.hasRootFolder)
-    , m_needsToSetFirstLastPiecePriority(false)
+    , m_useAutoTMM(params.savePath.isEmpty())
 {
     if (m_useAutoTMM)
         m_savePath = Utils::Fs::toNativePath(m_session->categorySavePath(m_category));

src/base/bittorrent/torrenthandleimpl.h

@@ -55,12 +55,16 @@ namespace BitTorrent
 
     struct CreateTorrentParams
     {
-        bool restored = false; // is existing torrent job?
+        CreateTorrentParams() = default;
+        explicit CreateTorrentParams(const AddTorrentParams &params);
+
         // for both new and restored torrents
         QString name;
         QString category;
         QSet<QString> tags;
         QString savePath;
+        int uploadLimit = -1;
+        int downloadLimit = -1;
         bool disableTempPath = false;
         bool sequential = false;
         bool firstLastPiecePriority = false;
@@ -69,17 +73,15 @@ namespace BitTorrent
         bool hasRootFolder = true;
         bool forced = false;
         bool paused = false;
-        int uploadLimit = -1;
-        int downloadLimit = -1;
+        bool restored = false;  // is existing torrent job?
+
         // for new torrents
         QVector<DownloadPriority> filePriorities;
         QDateTime addedTime;
+
         // for restored torrents
         qreal ratioLimit = TorrentHandle::USE_GLOBAL_RATIO;
         int seedingTimeLimit = TorrentHandle::USE_GLOBAL_SEEDING_TIME;
-
-        CreateTorrentParams() = default;
-        explicit CreateTorrentParams(const AddTorrentParams &params);
     };
 
     enum class MoveStorageMode
@@ -298,39 +300,38 @@ namespace BitTorrent
         Session *const m_session;
         lt::torrent_handle m_nativeHandle;
         lt::torrent_status m_nativeStatus;
-        TorrentState m_state;
+        TorrentState m_state = TorrentState::Unknown;
         TorrentInfo m_torrentInfo;
         SpeedMonitor m_speedMonitor;
 
         InfoHash m_hash;
 
-        bool m_storageIsMoving = false;
         // m_moveFinishedTriggers is activated only when the following conditions are met:
         // all file rename jobs complete, all file move jobs complete
         QQueue<EventTrigger> m_moveFinishedTriggers;
-        int m_renameCount;
+        int m_renameCount = 0;
+        bool m_storageIsMoving = false;
 
         // Until libtorrent provide an "old_name" field in `file_renamed_alert`
         // we will rely on this workaround to remove empty leftover folders
         QHash<LTFileIndex, QVector<QString>> m_oldPath;
 
-        bool m_useAutoTMM;
+        QHash<QString, TrackerInfo> m_trackerInfos;
 
         // Persistent data
         QString m_name;
         QString m_savePath;
         QString m_category;
         QSet<QString> m_tags;
-        bool m_hasSeedStatus;
         qreal m_ratioLimit;
         int m_seedingTimeLimit;
+        bool m_hasSeedStatus;
         bool m_tempPathDisabled;
-        bool m_fastresumeDataRejected;
-        bool m_hasMissingFiles;
+        bool m_fastresumeDataRejected = false;
+        bool m_hasMissingFiles = false;
         bool m_hasRootFolder;
-        bool m_needsToSetFirstLastPiecePriority;
-
-        QHash<QString, TrackerInfo> m_trackerInfos;
+        bool m_needsToSetFirstLastPiecePriority = false;
+        bool m_useAutoTMM;
 
         bool m_unchecked = false;
     };