From 79a87bd4e7bae80fb5357e451df1609c94dc8c15 Mon Sep 17 00:00:00 2001
From: Rosen Penev <rosenp@gmail.com>
Date: Thu, 6 Feb 2025 14:41:01 -0800
Subject: [PATCH] download: default initialize

Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 src/download/available_list.h    | 4 +---
 src/download/delegator.h         | 4 +---
 src/download/download_wrapper.cc | 6 +-----
 src/download/download_wrapper.h  | 6 +++---
 src/manager.cc                   | 4 ++--
 src/manager.h                    | 2 +-
 6 files changed, 9 insertions(+), 17 deletions(-)

diff --git a/src/download/available_list.h b/src/download/available_list.h
index c064ad976..753382d72 100644
--- a/src/download/available_list.h
+++ b/src/download/available_list.h
@@ -71,8 +71,6 @@ class AvailableList : private std::vector<rak::socket_address> {
   using base_type::rbegin;
   using base_type::rend;
 
-  AvailableList() : m_maxSize(1000) {}
-
   value_type          pop_random();
 
   // Fuzzy size limit.
@@ -94,7 +92,7 @@ class AvailableList : private std::vector<rak::socket_address> {
   AddressList*        buffer()                            { return &m_buffer; }
 
 private:
-  size_type           m_maxSize;
+  size_type           m_maxSize{1000};
 
   AddressList         m_buffer;
 };
diff --git a/src/download/delegator.h b/src/download/delegator.h
index 7134bb7cb..849752c4f 100644
--- a/src/download/delegator.h
+++ b/src/download/delegator.h
@@ -59,8 +59,6 @@ class Delegator {
 
   static const unsigned int block_size = 1 << 14;
 
-  Delegator() : m_aggressive(false) { }
-
   TransferList*       transfer_list()                     { return &m_transfers; }
   const TransferList* transfer_list() const               { return &m_transfers; }
 
@@ -79,7 +77,7 @@ class Delegator {
 
   TransferList       m_transfers;
 
-  bool               m_aggressive;
+  bool               m_aggressive{false};
 
   // Propably should add a m_slotChunkStart thing, which will take
   // care of enabling etc, and will be possible to listen to.
diff --git a/src/download/download_wrapper.cc b/src/download/download_wrapper.cc
index 9459b1ff0..64beedd25 100644
--- a/src/download/download_wrapper.cc
+++ b/src/download/download_wrapper.cc
@@ -35,11 +35,7 @@
 namespace torrent {
 
 DownloadWrapper::DownloadWrapper() :
-  m_main(new DownloadMain),
-
-  m_bencode(NULL),
-  m_hashChecker(NULL),
-  m_connectionType(0) {
+  m_main(new DownloadMain) {
 
   m_main->delay_download_done().slot()       = std::bind(&download_data::call_download_done, data());
   m_main->delay_partially_done().slot()      = std::bind(&download_data::call_partially_done, data());
diff --git a/src/download/download_wrapper.h b/src/download/download_wrapper.h
index 0dfdd3ba3..fb51ba4d7 100644
--- a/src/download/download_wrapper.h
+++ b/src/download/download_wrapper.h
@@ -74,13 +74,13 @@ class DownloadWrapper {
   void                finished_download();
 
   DownloadMain*       m_main;
-  Object*             m_bencode;
-  HashTorrent*        m_hashChecker;
+  Object*             m_bencode{};
+  HashTorrent*        m_hashChecker{};
   HashQueue*          m_hashQueue;
 
   std::string         m_hash;
 
-  int                 m_connectionType;
+  int                 m_connectionType{0};
 };
 
 }
diff --git a/src/manager.cc b/src/manager.cc
index e8ea04a7d..076172600 100644
--- a/src/manager.cc
+++ b/src/manager.cc
@@ -42,9 +42,9 @@ Manager::Manager() :
   m_client_list(new ClientList),
 
   m_uploadThrottle(Throttle::create_throttle()),
-  m_downloadThrottle(Throttle::create_throttle()),
+  m_downloadThrottle(Throttle::create_throttle())
 
-  m_ticks(0) {
+  {
 
   m_hash_queue = std::make_unique<HashQueue>(&m_main_thread_disk);
 
diff --git a/src/manager.h b/src/manager.h
index 35e3d2dfe..81c38b59c 100644
--- a/src/manager.h
+++ b/src/manager.h
@@ -81,7 +81,7 @@ class Manager {
   Throttle*           m_uploadThrottle;
   Throttle*           m_downloadThrottle;
 
-  unsigned int        m_ticks;
+  unsigned int        m_ticks{0};
   rak::priority_item  m_taskTick;
 };