From 30f5bf82da45b87814562a92d0893d841bf53867 Mon Sep 17 00:00:00 2001
From: Eduard Bagdasaryan <eduard.bagdasaryan@measurement-factory.com>
Date: Tue, 20 Jul 2021 21:24:30 +0300
Subject: [PATCH] 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).
---
 src/Downloader.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Downloader.cc b/src/Downloader.cc
index de4038d36c..548b243045 100644
--- 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
-- 
2.47.2