From a3751cfcbee70c99c938e0cbdede1d0d09629f7e Mon Sep 17 00:00:00 2001
From: Arvid Norberg <arvid@libtorrent.org>
Date: Thu, 25 Sep 2008 20:12:53 +0000
Subject: [PATCH] fixed bug in peer_connection::incoming_piece() in case blocks
 were skipped. Fixed warnings.

---
 src/peer_connection.cpp | 8 +++++++-
 src/session_impl.cpp    | 2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/peer_connection.cpp b/src/peer_connection.cpp
index c98d89c01..56eaee93d 100644
--- a/src/peer_connection.cpp
+++ b/src/peer_connection.cpp
@@ -1693,6 +1693,9 @@ namespace libtorrent
 			send_block_requests();
 			return;
 		}
+#ifndef NDEBUG
+		pending_block pending_b = *b;
+#endif
 
 		int block_index = b - m_download_queue.begin() - 1;
 		for (int i = 0; i < block_index; ++i)
@@ -1721,6 +1724,9 @@ namespace libtorrent
 				--block_index;
 			}
 		}
+		TORRENT_ASSERT(int(m_download_queue.size()) > block_index + 1);
+		b = m_download_queue.begin() + (block_index + 1);
+		TORRENT_ASSERT(b->block == pending_b.block);
 		
 		// if the block we got is already finished, then ignore it
 		if (picker.is_downloaded(block_finished))
@@ -3856,7 +3862,7 @@ namespace libtorrent
 		}
 
 		if (t->ready_for_connections() && m_initialized)
-			TORRENT_ASSERT(t->torrent_file().num_pieces() == m_have_piece.size());
+			TORRENT_ASSERT(t->torrent_file().num_pieces() == int(m_have_piece.size()));
 
 		if (m_ses.settings().close_redundant_connections)
 		{
diff --git a/src/session_impl.cpp b/src/session_impl.cpp
index e201f0c20..0ca598872 100644
--- a/src/session_impl.cpp
+++ b/src/session_impl.cpp
@@ -2508,7 +2508,7 @@ namespace aux {
 			++total_downloaders;
 			unique.insert(i->second->queue_position());
 		}
-		TORRENT_ASSERT(unique.size() == int(total_downloaders));
+		TORRENT_ASSERT(int(unique.size()) == total_downloaders);
 
 		TORRENT_ASSERT(m_max_connections > 0);
 		TORRENT_ASSERT(m_max_uploads > 0);