Undone an out-of-scope change

Apart from Downloader, there are several other doneAll() methods
needing refactoring as well (e.g., ConnOpener::doneAll()). We
should refactor them all at once (probably in a separate PR).

diff --git a/src/Downloader.cc b/src/Downloader.cc
index de4038d36ca13e8985488ba362df8b7dc885b8f3..548b243045a08c4377f1846b4e3a6a37bb243c65 100644 (file)
--- a/src/Downloader.cc
+++ b/src/Downloader.cc
@@ -94,7 +94,7 @@ Downloader::swanSong()
 bool
 Downloader::doneAll() const
 {
-    return !callback_;
+    return (!callback_ || callback_->canceled()) && AsyncJob::doneAll();
 }
 
 static void