Browse Source

2008-05-09 Tatsuhiro Tsujikawa <tujikawa at rednoah dot com>

	Call Command::transitStatus() before calling Command::execute(),
	so that the status change made in execute() is not cleared.
	* src/DownloadEngine.cc
Tatsuhiro Tsujikawa 17 years ago
parent
commit
0b5e827ead
2 changed files with 7 additions and 2 deletions
  1. 6 0
      ChangeLog
  2. 1 2
      src/DownloadEngine.cc

+ 6 - 0
ChangeLog

@@ -1,3 +1,9 @@
+2008-05-09  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
+
+	Call Command::transitStatus() before calling Command::execute(),
+	so that the status change made in execute() is not cleared.
+	* src/DownloadEngine.cc
+
 2008-05-09  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
 
 	Removed forward declaration of SocketCore class.

+ 1 - 2
src/DownloadEngine.cc

@@ -117,10 +117,9 @@ static void executeCommand(std::deque<Command*>& commands,
     Command* com = commands.front();
     commands.pop_front();
     if(com->statusMatch(statusFilter)) {
+      com->transitStatus();
       if(com->execute()) {
 	delete com;
-      } else {
-	com->transitStatus();
       }
     } else {
       commands.push_back(com);