cpp/openttd-patchpack/source Changeset - r17444:de3de14c6d35

Changeset - r17444:de3de14c6d35

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

rubidium - 13 years ago 2011-03-06 10:11:59
rubidium@openttd.org

(svn r22208) -Fix [FS#4543]: When downloading a file via HTTP failed mid-way and OpenTTD fell back to the old system the partial downloaded amount would be counted twice

3 files changed with 10 insertions and 6 deletions:

src/network/network_content.cpp

src/network/network_content.h

src/network/network_content_gui.cpp

0 comments (0 inline, 0 general)

src/network/network_content.cpp

➞

Show inline comments

@@ @@ -453,7 +453,7 @@ DEF_CONTENT_RECEIVE_COMMAND(Client, PACK @@
 			return false;
+		}
-		this->OnDownloadProgress(this->curInfo, (uint)toRead);
+		this->OnDownloadProgress(this->curInfo, (int)toRead);
 		if (toRead == 0) this->AfterDownload();
+	}
@@ @@ -526,6 +526,10 @@ void ClientNetworkContentSocketHandler:: @@
 	this->http_response_index = -2;
 	if (this->curFile != NULL) {
 		/* Revert the download progress when we are going for the old system. */
 		long size = ftell(this->curFile);
 		if (size > 0) this->OnDownloadProgress(this->curInfo, (int)-size);
 		fclose(this->curFile);
 		this->curFile = NULL;
+	}
@@ @@ -559,7 +563,7 @@ void ClientNetworkContentSocketHandler:: @@
 			this->OnFailure();
 		} else {
 			/* Just received the data. */
-			this->OnDownloadProgress(this->curInfo, (uint)length);
+			this->OnDownloadProgress(this->curInfo, (int)length);
+		}
 		/* Nothing more to do now. */
 		return;
@@ @@ -1023,7 +1027,7 @@ void ClientNetworkContentSocketHandler:: @@
+	}
+}
-void ClientNetworkContentSocketHandler::OnDownloadProgress(const ContentInfo *ci, uint bytes)
+void ClientNetworkContentSocketHandler::OnDownloadProgress(const ContentInfo *ci, int bytes)
+{
 	for (ContentCallback **iter = this->callbacks.Begin(); iter != this->callbacks.End(); /* nothing */) {
 		ContentCallback *cb = *iter;

src/network/network_content.h

➞

Show inline comments

@@ @@ -49,7 +49,7 @@ struct ContentCallback { @@
 	 * @param ci the content info of the file
 	 * @param bytes the number of bytes downloaded since the previous call
 	 */
-	virtual void OnDownloadProgress(const ContentInfo *ci, uint bytes) {}
+	virtual void OnDownloadProgress(const ContentInfo *ci, int bytes) {}
 	/**
 	 * We have finished downloading a file
@@ @@ -89,7 +89,7 @@ protected: @@
 	void OnConnect(bool success);
 	void OnDisconnect();
 	void OnReceiveContentInfo(const ContentInfo *ci);
-	void OnDownloadProgress(const ContentInfo *ci, uint bytes);
+	void OnDownloadProgress(const ContentInfo *ci, int bytes);
 	void OnDownloadComplete(ContentID cid);
 	void OnFailure();

src/network/network_content_gui.cpp

➞

Show inline comments

@@ @@ -175,7 +175,7 @@ public: @@
+		}
+	}
-	virtual void OnDownloadProgress(const ContentInfo *ci, uint bytes)
+	virtual void OnDownloadProgress(const ContentInfo *ci, int bytes)
+	{
 		if (ci->id != this->cur_id) {
 			strecpy(this->name, ci->filename, lastof(this->name));

0 comments (0 inline, 0 general)