added consts and minor refactor (#1137)

added consts and minor refactor

src/disk_io_thread.cpp

@@ -1989,7 +1989,7 @@ namespace libtorrent
 		, std::function<void(disk_io_job const*)> handler)
 	{
 		disk_io_job* j = allocate_job(disk_io_job::load_torrent);
-		j->requester = reinterpret_cast<char*>(params);
+		j->requester = reinterpret_cast<void*>(params);
 		j->callback = std::move(handler);
 
 		add_job(j);

src/file.cpp

@@ -362,7 +362,7 @@ namespace libtorrent
 #endif
 
 	void stat_file(std::string const& inf, file_status* s
-		, error_code& ec, int flags)
+		, error_code& ec, int const flags)
 	{
 		ec.clear();
 #ifdef TORRENT_WINDOWS

src/session_impl.cpp

@@ -4535,7 +4535,7 @@ namespace aux {
 		}
 	}
 
-	void session_impl::post_torrent_updates(std::uint32_t flags)
+	void session_impl::post_torrent_updates(std::uint32_t const flags)
 	{
 		INVARIANT_CHECK;
 
@@ -4557,10 +4557,8 @@ namespace aux {
 		// pushed back. Perhaps the status_update_alert could even have a fixed
 		// array of n entries rather than a vector, to further improve memory
 		// locality.
-		for (std::vector<torrent*>::iterator i = state_updates.begin()
-			, end(state_updates.end()); i != end; ++i)
+		for (auto& t : state_updates)
 		{
-			torrent* t = *i;
 			TORRENT_ASSERT(t->m_links[aux::session_impl::torrent_state_updates].in_list());
 			status.push_back(torrent_status());
 			// querying accurate download counters may require
@@ -4649,25 +4647,25 @@ namespace aux {
 		}
 
 		error_code ec;
-		torrent_handle handle = add_torrent(*params, ec);
+		add_torrent(*params, ec);
 		delete params;
 	}
 
 	void session_impl::on_async_load_torrent(disk_io_job const* j)
 	{
-		add_torrent_params* params = static_cast<add_torrent_params*>(j->requester);
-		error_code ec;
-		torrent_handle handle;
+		add_torrent_params* params = reinterpret_cast<add_torrent_params*>(j->requester);
 		if (j->error.ec)
 		{
-			ec = j->error.ec;
-			m_alerts.emplace_alert<add_torrent_alert>(handle, *params, ec);
+			m_alerts.emplace_alert<add_torrent_alert>(torrent_handle()
+				, *params, j->error.ec);
 		}
 		else
 		{
 			params->url.clear();
 			params->ti = std::shared_ptr<torrent_info>(j->buffer.torrent_file);
-			handle = add_torrent(*params, ec);
+
+			error_code ec;
+			add_torrent(*params, ec);
 		}
 
 		delete params;

src/stat_cache.cpp

@@ -52,7 +52,7 @@ namespace libtorrent
 		m_stat_cache[i].file_size = size;
 	}
 
-	void stat_cache::set_error(int i, error_code const& ec)
+	void stat_cache::set_error(int const i, error_code const& ec)
 	{
 		TORRENT_ASSERT(i >= 0);
 		if (i >= int(m_stat_cache.size()))
@@ -62,14 +62,14 @@ namespace libtorrent
 		m_stat_cache[i].file_size = file_error - error_index;
 	}
 
-	void stat_cache::set_dirty(int i)
+	void stat_cache::set_dirty(int const i)
 	{
 		TORRENT_ASSERT(i >= 0);
 		if (i >= int(m_stat_cache.size())) return;
 		m_stat_cache[i].file_size = not_in_cache;
 	}
 
-	std::int64_t stat_cache::get_filesize(int i, file_storage const& fs
+	std::int64_t stat_cache::get_filesize(int const i, file_storage const& fs
 		, std::string const& save_path, error_code& ec)
 	{
 		TORRENT_ASSERT(i < int(fs.num_files()));
@@ -84,7 +84,7 @@ namespace libtorrent
 		{
 			// query the filesystem
 			file_status s;
-			std::string file_path = fs.file_path(i, save_path);
+			std::string const file_path = fs.file_path(i, save_path);
 			stat_file(file_path, &s, ec);
 			if (ec)
 			{

src/storage.cpp

@@ -610,7 +610,7 @@ namespace libtorrent
 
 		for (int i = 0; i < files().num_files(); ++i)
 		{
-			std::int64_t sz = m_stat_cache.get_filesize(
+			std::int64_t const sz = m_stat_cache.get_filesize(
 				i, files(), m_save_path, ec.ec);
 
 			if (sz < 0)
@@ -645,7 +645,7 @@ namespace libtorrent
 		return false;
 	}
 
-	void default_storage::rename_file(int index, std::string const& new_filename
+	void default_storage::rename_file(int const index, std::string const& new_filename
 		, storage_error& ec)
 	{
 		if (index < 0 || index >= files().num_files()) return;
@@ -1498,11 +1498,10 @@ namespace libtorrent
 
 	int piece_manager::check_no_fastresume(storage_error& ec)
 	{
-		bool has_files = false;
 		if (!m_storage->settings().get_bool(settings_pack::no_recheck_incomplete_resume))
 		{
 			storage_error se;
-			has_files = m_storage->has_any_file(se);
+			bool const has_files = m_storage->has_any_file(se);
 
 			if (se)
 			{

src/torrent.cpp

@@ -1248,10 +1248,9 @@ namespace libtorrent
 
 		update_gauge();
 
-		for (peer_iterator i = m_connections.begin()
-			, end(m_connections.end()); i != end; ++i)
+		for (auto const& p : m_connections)
 		{
-			peer_has((*i)->get_bitfield(), *i);
+			peer_has(p->get_bitfield(), p);
 		}
 	}
 
@@ -2298,12 +2297,10 @@ namespace libtorrent
 
 				int const num_blocks_per_piece = torrent_file().piece_length() / block_size();
 
-				for (std::map<int, bitfield>::const_iterator i
-					= m_add_torrent_params->unfinished_pieces.begin()
-					, end(m_add_torrent_params->unfinished_pieces.end()); i != end; ++i)
+				for (auto const& p : m_add_torrent_params->unfinished_pieces)
 				{
-					int const piece = i->first;
-					bitfield const& blocks = i->second;
+					int const piece = p.first;
+					bitfield const& blocks = p.second;
 
 					if (piece < 0 || piece > torrent_file().num_pieces()) continue;
 
@@ -7881,9 +7878,8 @@ namespace libtorrent
 		}
 		else
 		{
-			for (std::vector<announce_entry>::iterator i = m_trackers.begin()
-				, end(m_trackers.end()); i != end; ++i)
-				i->complete_sent = true;
+			for (auto& t : m_trackers)
+				t.complete_sent = true;
 
 			if (m_state != torrent_status::finished
 				&& m_state != torrent_status::seeding)
@@ -10856,7 +10852,7 @@ namespace libtorrent
 		}
 	}
 
-	void torrent::set_state(torrent_status::state_t s)
+	void torrent::set_state(torrent_status::state_t const s)
 	{
 		TORRENT_ASSERT(is_single_thread());
 		TORRENT_ASSERT(s != 0); // this state isn't used anymore