Cleanup AbstractCommand take 2

Decided to keep the 'if(' -> 'if (' stuff, as I formatted the whole file
and so it is internally consistent within the file. Also, too much
hassle.

1 files changed, 505 insertions, 439 deletions

diff --git a/src/AbstractCommand.cc b/src/AbstractCommand.cc
index 313155ba..4c34e038 100644
--- a/src/AbstractCommand.cc
+++ b/src/AbstractCommand.cc
@@ -74,13 +74,15 @@
 
 namespace aria2 {
 
-AbstractCommand::AbstractCommand(cuid_t cuid,
-                                 const std::shared_ptr<Request>& req,
-                                 const std::shared_ptr<FileEntry>& fileEntry,
-                                 RequestGroup* requestGroup, DownloadEngine* e,
-                                 const std::shared_ptr<SocketCore>& s,
-                                 const std::shared_ptr<SocketRecvBuffer>& socketRecvBuffer,
-                                 bool incNumConnection)
+AbstractCommand::AbstractCommand(
+    cuid_t cuid,
+    const std::shared_ptr<Request>& req,
+    const std::shared_ptr<FileEntry>& fileEntry,
+    RequestGroup* requestGroup,
+    DownloadEngine* e,
+    const std::shared_ptr<SocketCore>& s,
+    const std::shared_ptr<SocketRecvBuffer>& socketRecvBuffer,
+    bool incNumConnection)
   : Command(cuid),
     req_(req),
     fileEntry_(fileEntry),
@@ -98,10 +100,10 @@ AbstractCommand::AbstractCommand(cuid_t cuid,
     checkSocketIsWritable_(false),
     incNumConnection_(incNumConnection)
 {
-  if(socket_ && socket_->isOpen()) {
+  if (socket_ && socket_->isOpen()) {
     setReadCheckSocket(socket_);
   }
-  if(incNumConnection_) {
+  if (incNumConnection_) {
     requestGroup->increaseStreamConnection();
   }
   requestGroup_->increaseStreamCommand();
@@ -111,7 +113,8 @@ AbstractCommand::AbstractCommand(cuid_t cuid,
 #endif // ENABLE_ASYNC_DNS
 }
 
-AbstractCommand::~AbstractCommand() {
+AbstractCommand::~AbstractCommand()
+{
   disableReadCheckSocket();
   disableWriteCheckSocket();
 #ifdef ENABLE_ASYNC_DNS
@@ -119,13 +122,13 @@ AbstractCommand::~AbstractCommand() {
 #endif // ENABLE_ASYNC_DNS
   requestGroup_->decreaseNumCommand();
   requestGroup_->decreaseStreamCommand();
-  if(incNumConnection_) {
+  if (incNumConnection_) {
     requestGroup_->decreaseStreamConnection();
   }
 }
 
-void AbstractCommand::useFasterRequest
-(const std::shared_ptr<Request>& fasterRequest)
+void
+AbstractCommand::useFasterRequest(const std::shared_ptr<Request>& fasterRequest)
 {
   A2_LOG_INFO(fmt("CUID#%" PRId64 " - Use faster Request hostname=%s, port=%u",
                   getCuid(),
@@ -134,234 +137,265 @@ void AbstractCommand::useFasterRequest
   // Cancel current Request object and use faster one.
   fileEntry_->removeRequest(req_);
   e_->setNoWait(true);
-  e_->addCommand(InitiateConnectionCommandFactory::
-                 createInitiateConnectionCommand
-                 (getCuid(), fasterRequest, fileEntry_, requestGroup_, e_));
+  e_->addCommand(
+      InitiateConnectionCommandFactory::createInitiateConnectionCommand(
+          getCuid(), fasterRequest, fileEntry_, requestGroup_, e_));
 }
 
-bool AbstractCommand::execute() {
-  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - socket: read:%d, write:%d, hup:%d, err:%d",
+bool AbstractCommand::shouldProcess() const
+{
+  if (checkSocketIsReadable_) {
+    if (readEventEnabled()) {
+      return true;
+    }
+
+    if (socketRecvBuffer_ && !socketRecvBuffer_->bufferEmpty()) {
+      return true;
+    }
+  }
+
+  if (checkSocketIsWritable_ && writeEventEnabled()) {
+    return true;
+  }
+
+#ifdef ENABLE_ASYNC_DNS
+  const auto resolverChecked = asyncNameResolverMan_->resolverChecked();
+  if (resolverChecked && asyncNameResolverMan_->getStatus() != 0) {
+    return true;
+  }
+
+  if (!checkSocketIsReadable_ && !checkSocketIsWritable_ && !resolverChecked) {
+    return true;
+  }
+#else // ENABLE_ASYNC_DNS
+  if (!checkSocketIsReadable_ && !checkSocketIsWritable_) {
+    return true;
+  }
+#endif // ENABLE_ASYNC_DNS
+
+  return noCheck();
+}
+
+bool AbstractCommand::execute()
+{
+  A2_LOG_DEBUG(fmt("CUID#%" PRId64
+                   " - socket: read:%d, write:%d, hup:%d, err:%d",
                    getCuid(),
                    readEventEnabled(),
                    writeEventEnabled(),
                    hupEventEnabled(),
                    errorEventEnabled()));
   try {
-    if(requestGroup_->downloadFinished() || requestGroup_->isHaltRequested()) {
+    if (requestGroup_->downloadFinished() || requestGroup_->isHaltRequested()) {
       return true;
     }
-    if(req_ && req_->removalRequested()) {
-      A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Discard original URI=%s because it is"
+
+    if (req_ && req_->removalRequested()) {
+      A2_LOG_DEBUG(fmt("CUID#%" PRId64
+                       " - Discard original URI=%s because it is"
                        " requested.",
-                       getCuid(), req_->getUri().c_str()));
+                       getCuid(),
+                       req_->getUri().c_str()));
       return prepareForRetry(0);
     }
-    if(getPieceStorage()) {
+
+    auto sm = getSegmentMan();
+
+    if (getPieceStorage()) {
       segments_.clear();
-      getSegmentMan()->getInFlightSegment(segments_, getCuid());
-      if(req_ && segments_.empty()) {
+      sm->getInFlightSegment(segments_, getCuid());
+
+      if (req_ && segments_.empty()) {
         // This command previously has assigned segments, but it is
         // canceled. So discard current request chain.  Plus, if no
         // segment is available when http pipelining is used.
-        A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - It seems previously assigned segments"
+        A2_LOG_DEBUG(fmt("CUID#%" PRId64
+                         " - It seems previously assigned segments"
                          " are canceled. Restart.",
                          getCuid()));
         // Request::isPipeliningEnabled() == true means aria2
         // accessed the remote server and discovered that the server
         // supports pipelining.
-        if(req_ && req_->isPipeliningEnabled()) {
+        if (req_ && req_->isPipeliningEnabled()) {
           e_->poolSocket(req_, createProxyRequest(), socket_);
         }
         return prepareForRetry(0);
       }
+
       // TODO it is not needed to check other PeerStats every time.
       // Find faster Request when no segment split is allowed.
-      if(req_ && fileEntry_->countPooledRequest() > 0 &&
-         requestGroup_->getTotalLength()-requestGroup_->getCompletedLength()
-         < calculateMinSplitSize()*2) {
-        std::shared_ptr<Request> fasterRequest = fileEntry_->findFasterRequest(req_);
-        if(fasterRequest) {
+      if (req_ && fileEntry_->countPooledRequest() > 0 &&
+          requestGroup_->getPendingLength() < calculateMinSplitSize() * 2) {
+        auto fasterRequest = fileEntry_->findFasterRequest(req_);
+        if (fasterRequest) {
           useFasterRequest(fasterRequest);
           return true;
         }
       }
       // Don't use this feature if PREF_MAX_{OVERALL_}DOWNLOAD_LIMIT
       // is used or total length is unknown.
-      if(req_ && fileEntry_->getLength() > 0 &&
-         e_->getRequestGroupMan()->getMaxOverallDownloadSpeedLimit() == 0 &&
-         requestGroup_->getMaxDownloadSpeedLimit() == 0 &&
-         serverStatTimer_.difference(global::wallclock()) >= 10) {
+      if (req_ && fileEntry_->getLength() > 0 &&
+          e_->getRequestGroupMan()->getMaxOverallDownloadSpeedLimit() == 0 &&
+          requestGroup_->getMaxDownloadSpeedLimit() == 0 &&
+          serverStatTimer_.difference(global::wallclock()) >= 10) {
         serverStatTimer_ = global::wallclock();
-        std::vector<std::pair<size_t, std::string> > usedHosts;
-        if(getOption()->getAsBool(PREF_SELECT_LEAST_USED_HOST)) {
+        std::vector<std::pair<size_t, std::string>> usedHosts;
+        if (getOption()->getAsBool(PREF_SELECT_LEAST_USED_HOST)) {
           getDownloadEngine()->getRequestGroupMan()->getUsedHosts(usedHosts);
         }
-        std::shared_ptr<Request> fasterRequest =
-          fileEntry_->findFasterRequest
-          (req_, usedHosts, e_->getRequestGroupMan()->getServerStatMan());
-        if(fasterRequest) {
+        auto fasterRequest = fileEntry_->findFasterRequest(
+            req_, usedHosts, e_->getRequestGroupMan()->getServerStatMan());
+        if (fasterRequest) {
           useFasterRequest(fasterRequest);
           return true;
         }
       }
     }
-    if((checkSocketIsReadable_ &&
-        (readEventEnabled() ||
-         (socketRecvBuffer_ && !socketRecvBuffer_->bufferEmpty()))) ||
-       (checkSocketIsWritable_ && writeEventEnabled()) ||
-       hupEventEnabled() ||
-#ifdef ENABLE_ASYNC_DNS
-       (asyncNameResolverMan_->resolverChecked() &&
-        asyncNameResolverMan_->getStatus() != 0) ||
-#endif // ENABLE_ASYNC_DNS
-       (!checkSocketIsReadable_ && !checkSocketIsWritable_
-#ifdef ENABLE_ASYNC_DNS
-        && !asyncNameResolverMan_->resolverChecked()
-#endif // ENABLE_ASYNC_DNS
-        ) || noCheck()) {
+
+    if (shouldProcess()) {
       checkPoint_ = global::wallclock();
-      if(getPieceStorage()) {
-        if(!req_ || req_->getMaxPipelinedRequest() == 1 ||
-           // Why the following condition is necessary? That's because
-           // For single file download, SegmentMan::getSegment(cuid)
-           // is more efficient.
-           getDownloadContext()->getFileEntries().size() == 1) {
-          size_t maxSegments = req_?req_->getMaxPipelinedRequest():1;
-          size_t minSplitSize = calculateMinSplitSize();
-          while(segments_.size() < maxSegments) {
-            std::shared_ptr<Segment> segment =
-              getSegmentMan()->getSegment(getCuid(), minSplitSize);
-            if(!segment) {
-              break;
-            } else {
-              segments_.push_back(segment);
-            }
-          }
-          if(segments_.empty()) {
-            // TODO socket could be pooled here if pipelining is
-            // enabled...  Hmm, I don't think if pipelining is enabled
-            // it does not go here.
-            A2_LOG_INFO(fmt(MSG_NO_SEGMENT_AVAILABLE,
-                            getCuid()));
-            // When all segments are ignored in SegmentMan, there are
-            // no URIs available, so don't retry.
-            if(getSegmentMan()->allSegmentsIgnored()) {
-              A2_LOG_DEBUG("All segments are ignored.");
-              // This will execute other idle Commands and let them
-              // finish quickly.
-              e_->setRefreshInterval(0);
-              return true;
-            } else {
-              return prepareForRetry(1);
-            }
-          }
-        } else {
-          // For multi-file downloads
-          size_t minSplitSize = calculateMinSplitSize();
-          size_t maxSegments = req_->getMaxPipelinedRequest();
-          if(segments_.size() < maxSegments) {
-            getSegmentMan()->getSegment
-              (segments_, getCuid(), minSplitSize, fileEntry_, maxSegments);
+
+      if (!getPieceStorage()) {
+        return executeInternal();
+      }
+
+      if (!req_ || req_->getMaxPipelinedRequest() == 1 ||
+          // Why the following condition is necessary? That's because
+          // For single file download, SegmentMan::getSegment(cuid)
+          // is more efficient.
+          getDownloadContext()->getFileEntries().size() == 1) {
+        size_t maxSegments = req_ ? req_->getMaxPipelinedRequest() : 1;
+        size_t minSplitSize = calculateMinSplitSize();
+        while (segments_.size() < maxSegments) {
+          auto segment = sm->getSegment(getCuid(), minSplitSize);
+          if (!segment) {
+            break;
           }
-          if(segments_.empty()) {
-            return prepareForRetry(0);
+          segments_.push_back(segment);
+        }
+        if (segments_.empty()) {
+          // TODO socket could be pooled here if pipelining is
+          // enabled...  Hmm, I don't think if pipelining is enabled
+          // it does not go here.
+          A2_LOG_INFO(fmt(MSG_NO_SEGMENT_AVAILABLE, getCuid()));
+          // When all segments are ignored in SegmentMan, there are
+          // no URIs available, so don't retry.
+          if (sm->allSegmentsIgnored()) {
+            A2_LOG_DEBUG("All segments are ignored.");
+            // This will execute other idle Commands and let them
+            // finish quickly.
+            e_->setRefreshInterval(0);
+            return true;
           }
+
+          return prepareForRetry(1);
         }
       }
-      return executeInternal();
-    } else if(errorEventEnabled()) {
-      throw DL_RETRY_EX(fmt(MSG_NETWORK_PROBLEM,
-                            socket_->getSocketError().c_str()));
-    } else {
-      if(checkPoint_.difference(global::wallclock()) >= timeout_) {
-        // timeout triggers ServerStat error state.
-        std::shared_ptr<ServerStat> ss =
-          e_->getRequestGroupMan()->getOrCreateServerStat(req_->getHost(),
-                                                          req_->getProtocol());
-        ss->setError();
-        // When DNS query was timeout, req_->getConnectedAddr() is
-        // empty.
-        if(!req_->getConnectedAddr().empty()) {
-          // Purging IP address cache to renew IP address.
-          A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Marking IP address %s as bad",
-                           getCuid(),
-                           req_->getConnectedAddr().c_str()));
-          e_->markBadIPAddress(req_->getConnectedHostname(),
-                               req_->getConnectedAddr(),
-                               req_->getConnectedPort());
+      else {
+        // For multi-file downloads
+        size_t minSplitSize = calculateMinSplitSize();
+        size_t maxSegments = req_->getMaxPipelinedRequest();
+        if (segments_.size() < maxSegments) {
+          sm->getSegment(
+              segments_, getCuid(), minSplitSize, fileEntry_, maxSegments);
         }
-        if(e_->findCachedIPAddress
-           (req_->getConnectedHostname(), req_->getConnectedPort()).empty()) {
-          A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - All IP addresses were marked bad."
-                           " Removing Entry.",
-                           getCuid()));
-          e_->removeCachedIPAddress
-            (req_->getConnectedHostname(), req_->getConnectedPort());
+        if (segments_.empty()) {
+          return prepareForRetry(0);
         }
-        throw DL_RETRY_EX2(EX_TIME_OUT, error_code::TIME_OUT);
       }
-      addCommandSelf();
-      return false;
+
+      return executeInternal();
     }
-  } catch(DlAbortEx& err) {
+
+    if (errorEventEnabled()) {
+      throw DL_RETRY_EX(
+          fmt(MSG_NETWORK_PROBLEM, socket_->getSocketError().c_str()));
+    }
+
+    if (checkPoint_.difference(global::wallclock()) >= timeout_) {
+      // timeout triggers ServerStat error state.
+      auto ss = e_->getRequestGroupMan()->getOrCreateServerStat(
+          req_->getHost(), req_->getProtocol());
+      ss->setError();
+      // When DNS query was timeout, req_->getConnectedAddr() is
+      // empty.
+      if (!req_->getConnectedAddr().empty()) {
+        // Purging IP address cache to renew IP address.
+        A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Marking IP address %s as bad",
+                         getCuid(),
+                         req_->getConnectedAddr().c_str()));
+        e_->markBadIPAddress(req_->getConnectedHostname(),
+                             req_->getConnectedAddr(),
+                             req_->getConnectedPort());
+      }
+      if (e_->findCachedIPAddress(req_->getConnectedHostname(),
+                                  req_->getConnectedPort()).empty()) {
+        A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - All IP addresses were marked bad."
+                         " Removing Entry.",
+                         getCuid()));
+        e_->removeCachedIPAddress(req_->getConnectedHostname(),
+                                  req_->getConnectedPort());
+      }
+      throw DL_RETRY_EX2(EX_TIME_OUT, error_code::TIME_OUT);
+    }
+
+    addCommandSelf();
+    return false;
+  }
+  catch (DlAbortEx& err) {
     requestGroup_->setLastErrorCode(err.getErrorCode());
-    if(req_) {
-      A2_LOG_ERROR_EX(fmt(MSG_DOWNLOAD_ABORTED,
-                          getCuid(),
-                          req_->getUri().c_str()),
-                      DL_ABORT_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()),
-                                   err));
+    if (req_) {
+      A2_LOG_ERROR_EX(
+          fmt(MSG_DOWNLOAD_ABORTED, getCuid(), req_->getUri().c_str()),
+          DL_ABORT_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()), err));
       fileEntry_->addURIResult(req_->getUri(), err.getErrorCode());
-      if(err.getErrorCode() == error_code::CANNOT_RESUME) {
+      if (err.getErrorCode() == error_code::CANNOT_RESUME) {
         requestGroup_->increaseResumeFailureCount();
       }
-    } else {
+    }
+    else {
       A2_LOG_DEBUG_EX(EX_EXCEPTION_CAUGHT, err);
     }
     onAbort();
     tryReserved();
     return true;
-  } catch(DlRetryEx& err) {
+  }
+  catch (DlRetryEx& err) {
     assert(req_);
-    A2_LOG_INFO_EX(fmt(MSG_RESTARTING_DOWNLOAD,
-                       getCuid(), req_->getUri().c_str()),
-                   DL_RETRY_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()),
-                                err));
+    A2_LOG_INFO_EX(
+        fmt(MSG_RESTARTING_DOWNLOAD, getCuid(), req_->getUri().c_str()),
+        DL_RETRY_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()), err));
     req_->addTryCount();
     req_->resetRedirectCount();
     req_->resetUri();
+
     const int maxTries = getOption()->getAsInt(PREF_MAX_TRIES);
     bool isAbort = maxTries != 0 && req_->getTryCount() >= maxTries;
-    if(isAbort) {
-      A2_LOG_INFO(fmt(MSG_MAX_TRY,
-                      getCuid(),
-                      req_->getTryCount()));
-      A2_LOG_ERROR_EX(fmt(MSG_DOWNLOAD_ABORTED,
-                          getCuid(),
-                          req_->getUri().c_str()),
-                      err);
+    if (isAbort) {
+      A2_LOG_INFO(fmt(MSG_MAX_TRY, getCuid(), req_->getTryCount()));
+      A2_LOG_ERROR_EX(
+          fmt(MSG_DOWNLOAD_ABORTED, getCuid(), req_->getUri().c_str()), err);
       fileEntry_->addURIResult(req_->getUri(), err.getErrorCode());
       requestGroup_->setLastErrorCode(err.getErrorCode());
-      if(err.getErrorCode() == error_code::CANNOT_RESUME) {
+      if (err.getErrorCode() == error_code::CANNOT_RESUME) {
         requestGroup_->increaseResumeFailureCount();
       }
       onAbort();
       tryReserved();
       return true;
-    } else {
-      Timer wakeTime(global::wallclock());
-      wakeTime.advance(getOption()->getAsInt(PREF_RETRY_WAIT));
-      req_->setWakeTime(wakeTime);
-      return prepareForRetry(0);
     }
-  } catch(DownloadFailureException& err) {
+
+    Timer wakeTime(global::wallclock());
+    wakeTime.advance(getOption()->getAsInt(PREF_RETRY_WAIT));
+    req_->setWakeTime(wakeTime);
+    return prepareForRetry(0);
+  }
+  catch (DownloadFailureException& err) {
     requestGroup_->setLastErrorCode(err.getErrorCode());
-    if(req_) {
-      A2_LOG_ERROR_EX(fmt(MSG_DOWNLOAD_ABORTED,
-                          getCuid(),
-                          req_->getUri().c_str()),
-                      DL_ABORT_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()),
-                                   err));
+    if (req_) {
+      A2_LOG_ERROR_EX(
+          fmt(MSG_DOWNLOAD_ABORTED, getCuid(), req_->getUri().c_str()),
+          DL_ABORT_EX2(fmt("URI=%s", req_->getCurrentUri().c_str()), err));
       fileEntry_->addURIResult(req_->getUri(), err.getErrorCode());
     } else {
       A2_LOG_ERROR_EX(EX_EXCEPTION_CAUGHT, err);
@@ -372,14 +406,14 @@ bool AbstractCommand::execute() {
   }
 }
 
-void AbstractCommand::tryReserved() {
-  if(getDownloadContext()->getFileEntries().size() == 1) {
-    const std::shared_ptr<FileEntry>& entry =
-      getDownloadContext()->getFirstFileEntry();
+void AbstractCommand::tryReserved()
+{
+  if (getDownloadContext()->getFileEntries().size() == 1) {
+    const auto& entry = getDownloadContext()->getFirstFileEntry();
     // Don't create new command if currently file length is unknown
     // and there are no URI left. Because file length is unknown, we
     // can assume that there are no in-flight request object.
-    if(entry->getLength() == 0 && entry->getRemainingUris().empty()) {
+    if (entry->getLength() == 0 && entry->getRemainingUris().empty()) {
       A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Not trying next request."
                        " No reserved/pooled request is remaining and"
                        " total length is still unknown.",
@@ -387,19 +421,20 @@ void AbstractCommand::tryReserved() {
       return;
     }
   }
-  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Trying reserved/pooled request.",
-                   getCuid()));
+  A2_LOG_DEBUG(
+      fmt("CUID#%" PRId64 " - Trying reserved/pooled request.", getCuid()));
   std::vector<std::unique_ptr<Command>> commands;
   requestGroup_->createNextCommand(commands, e_, 1);
   e_->setNoWait(true);
   e_->addCommand(std::move(commands));
 }
 
-bool AbstractCommand::prepareForRetry(time_t wait) {
-  if(getPieceStorage()) {
+bool AbstractCommand::prepareForRetry(time_t wait)
+{
+  if (getPieceStorage()) {
     getSegmentMan()->cancelSegment(getCuid());
   }
-  if(req_) {
+  if (req_) {
     // Reset persistentConnection and maxPipelinedRequest to handle
     // the situation where remote server returns Connection: close
     // after several pipelined requests.
@@ -408,17 +443,19 @@ bool AbstractCommand::prepareForRetry(time_t wait) {
 
     fileEntry_->poolRequest(req_);
     A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Pooling request URI=%s",
-                     getCuid(), req_->getUri().c_str()));
-    if(getSegmentMan()) {
+                     getCuid(),
+                     req_->getUri().c_str()));
+    if (getSegmentMan()) {
       getSegmentMan()->recognizeSegmentFor(fileEntry_);
     }
   }
 
-  auto command = make_unique<CreateRequestCommand>(getCuid(),
-                                                   requestGroup_, e_);
-  if(wait == 0) {
+  auto command =
+      make_unique<CreateRequestCommand>(getCuid(), requestGroup_, e_);
+  if (wait == 0) {
     e_->setNoWait(true);
-  } else {
+  }
+  else {
     // We don't use wait so that Command can be executed by
     // DownloadEngine::setRefreshInterval(0).
     command->setStatus(Command::STATUS_INACTIVE);
@@ -427,131 +464,154 @@ bool AbstractCommand::prepareForRetry(time_t wait) {
   return true;
 }
 
-void AbstractCommand::onAbort() {
-  if(req_) {
+void AbstractCommand::onAbort()
+{
+  if (req_) {
     fileEntry_->removeIdenticalURI(req_->getUri());
     fileEntry_->removeRequest(req_);
   }
-  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Aborting download",
+
+  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Aborting download", getCuid()));
+  if (!getPieceStorage()) {
+    return;
+  }
+
+  getSegmentMan()->cancelSegment(getCuid());
+  // Don't do following process if BitTorrent is involved or files
+  // in DownloadContext is more than 1. The latter condition is
+  // limitation of current implementation.
+  if (getOption()->getAsBool(PREF_ALWAYS_RESUME) || !fileEntry_ ||
+      getDownloadContext()->getNetStat().getSessionDownloadLength() != 0 ||
+      requestGroup_->p2pInvolved() ||
+      getDownloadContext()->getFileEntries().size() != 1) {
+    return;
+  }
+
+  const int maxTries = getOption()->getAsInt(PREF_MAX_RESUME_FAILURE_TRIES);
+  if (!(maxTries > 0 && requestGroup_->getResumeFailureCount() >= maxTries) &&
+      !fileEntry_->emptyRequestUri()) {
+    return;
+  }
+  // Local file exists, but given servers(or at least contacted
+  // ones) doesn't support resume. Let's restart download from
+  // scratch.
+  A2_LOG_NOTICE(fmt(_("CUID#%" PRId64 " - Failed to resume download."
+                      " Download from scratch."),
+                    getCuid()));
+  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Gathering URIs that has CANNOT_RESUME"
+                   " error",
                    getCuid()));
-  if(getPieceStorage()) {
-    getSegmentMan()->cancelSegment(getCuid());
-    // Don't do following process if BitTorrent is involved or files
-    // in DownloadContext is more than 1. The latter condition is
-    // limitation of current implementation.
-    if(!getOption()->getAsBool(PREF_ALWAYS_RESUME) &&
-       fileEntry_ &&
-       getDownloadContext()->getNetStat().getSessionDownloadLength() == 0 &&
-       !requestGroup_->p2pInvolved() &&
-       getDownloadContext()->getFileEntries().size() == 1) {
-      const int maxTries = getOption()->getAsInt(PREF_MAX_RESUME_FAILURE_TRIES);
-      if((maxTries > 0 && requestGroup_->getResumeFailureCount() >= maxTries)||
-         fileEntry_->emptyRequestUri()) {
-        // Local file exists, but given servers(or at least contacted
-        // ones) doesn't support resume. Let's restart download from
-        // scratch.
-        A2_LOG_NOTICE(fmt(_("CUID#%" PRId64 " - Failed to resume download."
-                            " Download from scratch."),
-                          getCuid()));
-        A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Gathering URIs that has CANNOT_RESUME"
-                         " error",
-                         getCuid()));
-        // Set PREF_ALWAYS_RESUME to A2_V_TRUE to avoid repeating this
-        // process.
-        getOption()->put(PREF_ALWAYS_RESUME, A2_V_TRUE);
-        std::deque<URIResult> res;
-        fileEntry_->extractURIResult(res, error_code::CANNOT_RESUME);
-        if(!res.empty()) {
-          getSegmentMan()->cancelAllSegments();
-          getSegmentMan()->eraseSegmentWrittenLengthMemo();
-          getPieceStorage()->markPiecesDone(0);
-          std::vector<std::string> uris;
-          uris.reserve(res.size());
-          std::transform(res.begin(), res.end(), std::back_inserter(uris),
-                         std::mem_fn(&URIResult::getURI));
-          A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - %lu URIs found.",
-                           getCuid(),
-                           static_cast<unsigned long int>(uris.size())));
-          fileEntry_->addUris(uris.begin(), uris.end());
-          getSegmentMan()->recognizeSegmentFor(fileEntry_);
-        }
-      }
-    }
+  // Set PREF_ALWAYS_RESUME to A2_V_TRUE to avoid repeating this
+  // process.
+  getOption()->put(PREF_ALWAYS_RESUME, A2_V_TRUE);
+  std::deque<URIResult> res;
+  fileEntry_->extractURIResult(res, error_code::CANNOT_RESUME);
+  if (res.empty()) {
+    return;
   }
+
+  getSegmentMan()->cancelAllSegments();
+  getSegmentMan()->eraseSegmentWrittenLengthMemo();
+  getPieceStorage()->markPiecesDone(0);
+  std::vector<std::string> uris;
+  uris.reserve(res.size());
+  std::transform(res.begin(),
+                 res.end(),
+                 std::back_inserter(uris),
+                 std::mem_fn(&URIResult::getURI));
+  A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - %lu URIs found.",
+                   getCuid(),
+                   static_cast<unsigned long int>(uris.size())));
+  fileEntry_->addUris(uris.begin(), uris.end());
+  getSegmentMan()->recognizeSegmentFor(fileEntry_);
 }
 
-void AbstractCommand::disableReadCheckSocket() {
-  if(checkSocketIsReadable_) {
-    e_->deleteSocketForReadCheck(readCheckTarget_, this);
-    checkSocketIsReadable_ = false;
-    readCheckTarget_.reset();
+void AbstractCommand::disableReadCheckSocket()
+{
+  if (!checkSocketIsReadable_) {
+    return;
   }
+
+  e_->deleteSocketForReadCheck(readCheckTarget_, this);
+  checkSocketIsReadable_ = false;
+  readCheckTarget_.reset();
 }
 
-void AbstractCommand::setReadCheckSocket
-(const std::shared_ptr<SocketCore>& socket) {
-  if(!socket->isOpen()) {
+void
+AbstractCommand::setReadCheckSocket(const std::shared_ptr<SocketCore>& socket)
+{
+  if (!socket->isOpen()) {
     disableReadCheckSocket();
-  } else {
-    if(checkSocketIsReadable_) {
-      if(*readCheckTarget_ != *socket) {
-        e_->deleteSocketForReadCheck(readCheckTarget_, this);
-        e_->addSocketForReadCheck(socket, this);
-        readCheckTarget_ = socket;
-      }
-    } else {
+    return;
+  }
+
+  if (checkSocketIsReadable_) {
+    if (*readCheckTarget_ != *socket) {
+      e_->deleteSocketForReadCheck(readCheckTarget_, this);
       e_->addSocketForReadCheck(socket, this);
-      checkSocketIsReadable_ = true;
       readCheckTarget_ = socket;
     }
+    return;
   }
+
+  e_->addSocketForReadCheck(socket, this);
+  checkSocketIsReadable_ = true;
+  readCheckTarget_ = socket;
 }
 
-void AbstractCommand::setReadCheckSocketIf
-(const std::shared_ptr<SocketCore>& socket, bool pred)
+void
+AbstractCommand::setReadCheckSocketIf(const std::shared_ptr<SocketCore>& socket,
+                                      bool pred)
 {
-  if(pred) {
+  if (pred) {
     setReadCheckSocket(socket);
-  } else {
-    disableReadCheckSocket();
+    return;
   }
+
+  disableReadCheckSocket();
 }
 
-void AbstractCommand::disableWriteCheckSocket() {
-  if(checkSocketIsWritable_) {
-    e_->deleteSocketForWriteCheck(writeCheckTarget_, this);
-    checkSocketIsWritable_ = false;
-    writeCheckTarget_.reset();
+void AbstractCommand::disableWriteCheckSocket()
+{
+  if (!checkSocketIsWritable_) {
+    return;
   }
+  e_->deleteSocketForWriteCheck(writeCheckTarget_, this);
+  checkSocketIsWritable_ = false;
+  writeCheckTarget_.reset();
 }
 
-void AbstractCommand::setWriteCheckSocket
-(const std::shared_ptr<SocketCore>& socket) {
-  if(!socket->isOpen()) {
+void
+AbstractCommand::setWriteCheckSocket(const std::shared_ptr<SocketCore>& socket)
+{
+  if (!socket->isOpen()) {
     disableWriteCheckSocket();
-  } else {
-    if(checkSocketIsWritable_) {
-      if(*writeCheckTarget_ != *socket) {
-        e_->deleteSocketForWriteCheck(writeCheckTarget_, this);
-        e_->addSocketForWriteCheck(socket, this);
-        writeCheckTarget_ = socket;
-      }
-    } else {
+    return;
+  }
+
+  if (checkSocketIsWritable_) {
+    if (*writeCheckTarget_ != *socket) {
+      e_->deleteSocketForWriteCheck(writeCheckTarget_, this);
       e_->addSocketForWriteCheck(socket, this);
-      checkSocketIsWritable_ = true;
       writeCheckTarget_ = socket;
     }
+    return;
   }
+
+  e_->addSocketForWriteCheck(socket, this);
+  checkSocketIsWritable_ = true;
+  writeCheckTarget_ = socket;
 }
 
-void AbstractCommand::setWriteCheckSocketIf
-(const std::shared_ptr<SocketCore>& socket, bool pred)
+void AbstractCommand::setWriteCheckSocketIf(
+    const std::shared_ptr<SocketCore>& socket, bool pred)
 {
-  if(pred) {
+  if (pred) {
     setWriteCheckSocket(socket);
-  } else {
-    disableWriteCheckSocket();
+    return;
   }
+
+  disableWriteCheckSocket();
 }
 
 void AbstractCommand::swapSocket(std::shared_ptr<SocketCore>& socket)
@@ -564,76 +624,72 @@ void AbstractCommand::swapSocket(std::shared_ptr<SocketCore>& socket)
 namespace {
 // Constructs proxy URI, merging username and password if they are
 // defined.
-std::string makeProxyUri
-(PrefPtr proxyPref,
- PrefPtr proxyUser,
- PrefPtr proxyPasswd,
- const Option* option)
+std::string makeProxyUri(PrefPtr proxyPref,
+                         PrefPtr proxyUser,
+                         PrefPtr proxyPasswd,
+                         const Option* option)
 {
   uri::UriStruct us;
-  if(uri::parse(us, option->get(proxyPref))) {
-    if(option->defined(proxyUser)) {
-      us.username = option->get(proxyUser);
-    }
-    if(option->defined(proxyPasswd)) {
-      us.password = option->get(proxyPasswd);
-      us.hasPassword = true;
-    }
-    return uri::construct(us);
-  } else {
+  if (!uri::parse(us, option->get(proxyPref))) {
     return "";
   }
+  if (option->defined(proxyUser)) {
+    us.username = option->get(proxyUser);
+  }
+  if (option->defined(proxyPasswd)) {
+    us.password = option->get(proxyPasswd);
+    us.hasPassword = true;
+  }
+  return uri::construct(us);
 }
 } // namespace
 
 namespace {
 // Returns proxy option value for the given protocol.
-std::string getProxyOptionFor
-(PrefPtr proxyPref,
- PrefPtr proxyUser,
- PrefPtr proxyPasswd,
- const Option* option)
+std::string getProxyOptionFor(PrefPtr proxyPref,
+                              PrefPtr proxyUser,
+                              PrefPtr proxyPasswd,
+                              const Option* option)
 {
   std::string uri = makeProxyUri(proxyPref, proxyUser, proxyPasswd, option);
-  if(uri.empty()) {
-    return makeProxyUri(PREF_ALL_PROXY, PREF_ALL_PROXY_USER,
-                        PREF_ALL_PROXY_PASSWD, option);
-  } else {
-    return uri;
+  if (uri.empty()) {
+    return makeProxyUri(
+        PREF_ALL_PROXY, PREF_ALL_PROXY_USER, PREF_ALL_PROXY_PASSWD, option);
   }
+
+  return uri;
 }
 } // namespace
 
 // Returns proxy URI for given protocol.  If no proxy URI is defined,
 // then returns an empty string.
-std::string getProxyUri
-(const std::string& protocol, const Option* option)
+std::string getProxyUri(const std::string& protocol, const Option* option)
 {
-  if(protocol == "http") {
-    return getProxyOptionFor(PREF_HTTP_PROXY,
-                             PREF_HTTP_PROXY_USER,
-                             PREF_HTTP_PROXY_PASSWD,
-                             option);
-  } else if(protocol == "https") {
+  if (protocol == "http") {
+    return getProxyOptionFor(
+        PREF_HTTP_PROXY, PREF_HTTP_PROXY_USER, PREF_HTTP_PROXY_PASSWD, option);
+  }
+
+  if (protocol == "https") {
     return getProxyOptionFor(PREF_HTTPS_PROXY,
                              PREF_HTTPS_PROXY_USER,
                              PREF_HTTPS_PROXY_PASSWD,
                              option);
-  } else if(protocol == "ftp") {
-    return getProxyOptionFor(PREF_FTP_PROXY,
-                             PREF_FTP_PROXY_USER,
-                             PREF_FTP_PROXY_PASSWD,
-                             option);
-  } else {
-    return A2STR::NIL;
   }
+
+  if (protocol == "ftp") {
+    return getProxyOptionFor(
+        PREF_FTP_PROXY, PREF_FTP_PROXY_USER, PREF_FTP_PROXY_PASSWD, option);
+  }
+
+  return A2STR::NIL;
 }
 
 namespace {
 // Returns true if proxy is defined for the given protocol. Otherwise
 // returns false.
-bool isProxyRequest
-(const std::string& protocol, const std::shared_ptr<Option>& option)
+bool isProxyRequest(const std::string& protocol,
+                    const std::shared_ptr<Option>& option)
 {
   std::string proxyUri = getProxyUri(protocol, option.get());
   return !proxyUri.empty();
@@ -641,38 +697,35 @@ bool isProxyRequest
 } // namespace
 
 namespace {
-bool inNoProxy(const std::shared_ptr<Request>& req,
-               const std::string& noProxy)
+bool inNoProxy(const std::shared_ptr<Request>& req, const std::string& noProxy)
 {
   std::vector<Scip> entries;
-  util::splitIter(noProxy.begin(), noProxy.end(), std::back_inserter(entries),
-                  ',', true);
-  if(entries.empty()) {
+  util::splitIter(
+      noProxy.begin(), noProxy.end(), std::back_inserter(entries), ',', true);
+  if (entries.empty()) {
     return false;
   }
-  for(std::vector<Scip>::const_iterator i = entries.begin(),
-        eoi = entries.end(); i != eoi; ++i) {
-    std::string::const_iterator slashpos =
-      std::find((*i).first, (*i).second, '/');
-    if(slashpos == (*i).second) {
-      if(util::noProxyDomainMatch
-         (req->getHost(), std::string((*i).first, (*i).second))) {
-        return true;
-      }
-    } else {
-      // TODO We don't resolve hostname here.  More complete
-      // implementation is that we should first resolve
-      // hostname(which may result in several IP addresses) and
-      // evaluates against all of them
-      std::string ip((*i).first, slashpos);
-      uint32_t bits;
-      if(!util::parseUIntNoThrow(bits, std::string(slashpos+1, (*i).second))) {
-        continue;
-      }
-      if(util::inSameCidrBlock(ip, req->getHost(), bits)) {
+
+  for (const auto& e : entries) {
+    const auto slashpos = std::find(e.first, e.second, '/');
+    if (slashpos == e.second) {
+      if (util::noProxyDomainMatch(req->getHost(),
+                                   std::string(e.first, e.second))) {
         return true;
       }
     }
+    // TODO We don't resolve hostname here.  More complete
+    // implementation is that we should first resolve
+    // hostname(which may result in several IP addresses) and
+    // evaluates against all of them
+    std::string ip(e.first, slashpos);
+    uint32_t bits;
+    if (!util::parseUIntNoThrow(bits, std::string(slashpos + 1, e.second))) {
+      continue;
+    }
+    if (util::inSameCidrBlock(ip, req->getHost(), bits)) {
+      return true;
+    }
   }
   return false;
 }
@@ -681,147 +734,157 @@ bool inNoProxy(const std::shared_ptr<Request>& req,
 bool AbstractCommand::isProxyDefined() const
 {
   return isProxyRequest(req_->getProtocol(), getOption()) &&
-    !inNoProxy(req_, getOption()->get(PREF_NO_PROXY));
+         !inNoProxy(req_, getOption()->get(PREF_NO_PROXY));
 }
 
 std::shared_ptr<Request> AbstractCommand::createProxyRequest() const
 {
   std::shared_ptr<Request> proxyRequest;
-  if(inNoProxy(req_, getOption()->get(PREF_NO_PROXY))) {
+  if (inNoProxy(req_, getOption()->get(PREF_NO_PROXY))) {
     return proxyRequest;
   }
+
   std::string proxy = getProxyUri(req_->getProtocol(), getOption().get());
-  if(!proxy.empty()) {
+  if (!proxy.empty()) {
     proxyRequest.reset(new Request());
-    if(proxyRequest->setUri(proxy)) {
+    if (proxyRequest->setUri(proxy)) {
       A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Using proxy", getCuid()));
-    } else {
-      A2_LOG_DEBUG(fmt("CUID#%" PRId64 " - Failed to parse proxy string",
-                       getCuid()));
+    }
+    else {
+      A2_LOG_DEBUG(
+          fmt("CUID#%" PRId64 " - Failed to parse proxy string", getCuid()));
       proxyRequest.reset();
     }
   }
   return proxyRequest;
 }
 
-std::string AbstractCommand::resolveHostname
-(std::vector<std::string>& addrs, const std::string& hostname, uint16_t port)
+std::string AbstractCommand::resolveHostname(std::vector<std::string>& addrs,
+                                             const std::string& hostname,
+                                             uint16_t port)
 {
-  if(util::isNumericHost(hostname)) {
+  if (util::isNumericHost(hostname)) {
     addrs.push_back(hostname);
     return hostname;
   }
+
   e_->findAllCachedIPAddresses(std::back_inserter(addrs), hostname, port);
+  if (!addrs.empty()) {
+    auto ipaddr = addrs.front();
+    A2_LOG_INFO(fmt(MSG_DNS_CACHE_HIT,
+                    getCuid(),
+                    hostname.c_str(),
+                    strjoin(addrs.begin(), addrs.end(), ", ").c_str()));
+    return ipaddr;
+  }
+
   std::string ipaddr;
-  if(addrs.empty()) {
 #ifdef ENABLE_ASYNC_DNS
-    if(getOption()->getAsBool(PREF_ASYNC_DNS)) {
-      if(!asyncNameResolverMan_->started()) {
-        asyncNameResolverMan_->startAsync(hostname, e_, this);
+  if (getOption()->getAsBool(PREF_ASYNC_DNS)) {
+    if (!asyncNameResolverMan_->started()) {
+      asyncNameResolverMan_->startAsync(hostname, e_, this);
+    }
+    switch (asyncNameResolverMan_->getStatus()) {
+    case -1:
+      if (!isProxyRequest(req_->getProtocol(), getOption())) {
+        e_->getRequestGroupMan()
+            ->getOrCreateServerStat(req_->getHost(), req_->getProtocol())
+            ->setError();
       }
-      switch(asyncNameResolverMan_->getStatus()) {
-      case -1:
-        if(!isProxyRequest(req_->getProtocol(), getOption())) {
-          e_->getRequestGroupMan()->getOrCreateServerStat
-            (req_->getHost(), req_->getProtocol())->setError();
-        }
-        throw DL_ABORT_EX2
-          (fmt(MSG_NAME_RESOLUTION_FAILED, getCuid(), hostname.c_str(),
-               asyncNameResolverMan_->getLastError().c_str()),
-           error_code::NAME_RESOLVE_ERROR);
-      case 0:
-        return A2STR::NIL;
-      case 1:
-        asyncNameResolverMan_->getResolvedAddress(addrs);
-        if(addrs.empty()) {
-          throw DL_ABORT_EX2
-            (fmt(MSG_NAME_RESOLUTION_FAILED, getCuid(), hostname.c_str(),
-                 "No address returned"),
-             error_code::NAME_RESOLVE_ERROR);
-        }
-        break;
+      throw DL_ABORT_EX2(fmt(MSG_NAME_RESOLUTION_FAILED,
+                             getCuid(),
+                             hostname.c_str(),
+                             asyncNameResolverMan_->getLastError().c_str()),
+                         error_code::NAME_RESOLVE_ERROR);
+    case 0:
+      return A2STR::NIL;
+
+    case 1:
+      asyncNameResolverMan_->getResolvedAddress(addrs);
+      if (addrs.empty()) {
+        throw DL_ABORT_EX2(fmt(MSG_NAME_RESOLUTION_FAILED,
+                               getCuid(),
+                               hostname.c_str(),
+                               "No address returned"),
+                           error_code::NAME_RESOLVE_ERROR);
       }
-    } else
+      break;
+    }
+  }
+  else
 #endif // ENABLE_ASYNC_DNS
-      {
-        NameResolver res;
-        res.setSocktype(SOCK_STREAM);
-        if(e_->getOption()->getAsBool(PREF_DISABLE_IPV6)) {
-          res.setFamily(AF_INET);
-        }
-        res.resolve(addrs, hostname);
-      }
-    A2_LOG_INFO(fmt(MSG_NAME_RESOLUTION_COMPLETE,
-                    getCuid(),
-                    hostname.c_str(),
-                    strjoin(addrs.begin(), addrs.end(), ", ").c_str()));
-    for(std::vector<std::string>::const_iterator i = addrs.begin(),
-          eoi = addrs.end(); i != eoi; ++i) {
-      e_->cacheIPAddress(hostname, *i, port);
+  {
+    NameResolver res;
+    res.setSocktype(SOCK_STREAM);
+    if (e_->getOption()->getAsBool(PREF_DISABLE_IPV6)) {
+      res.setFamily(AF_INET);
     }
-    ipaddr = e_->findCachedIPAddress(hostname, port);
-  } else {
-    ipaddr = addrs.front();
-    A2_LOG_INFO(fmt(MSG_DNS_CACHE_HIT,
-                    getCuid(),
-                    hostname.c_str(),
-                    strjoin(addrs.begin(), addrs.end(), ", ").c_str()));
+    res.resolve(addrs, hostname);
   }
+  A2_LOG_INFO(fmt(MSG_NAME_RESOLUTION_COMPLETE,
+                  getCuid(),
+                  hostname.c_str(),
+                  strjoin(addrs.begin(), addrs.end(), ", ").c_str()));
+  for (const auto& addr : addrs) {
+    e_->cacheIPAddress(hostname, addr, port);
+  }
+  ipaddr = e_->findCachedIPAddress(hostname, port);
   return ipaddr;
 }
 
-void AbstractCommand::prepareForNextAction
-(std::unique_ptr<CheckIntegrityEntry> checkEntry)
+void AbstractCommand::prepareForNextAction(
+    std::unique_ptr<CheckIntegrityEntry> checkEntry)
 {
   std::vector<std::unique_ptr<Command>> commands;
-  requestGroup_->processCheckIntegrityEntry(commands, std::move(checkEntry),
-                                            e_);
+  requestGroup_->processCheckIntegrityEntry(
+      commands, std::move(checkEntry), e_);
   e_->addCommand(std::move(commands));
   e_->setNoWait(true);
 }
 
-bool AbstractCommand::checkIfConnectionEstablished
-(const std::shared_ptr<SocketCore>& socket,
- const std::string& connectedHostname,
- const std::string& connectedAddr,
- uint16_t connectedPort)
+bool AbstractCommand::checkIfConnectionEstablished(
+    const std::shared_ptr<SocketCore>& socket,
+    const std::string& connectedHostname,
+    const std::string& connectedAddr,
+    uint16_t connectedPort)
 {
   std::string error = socket->getSocketError();
-  if(!error.empty()) {
-    // See also InitiateConnectionCommand::executeInternal()
-    e_->markBadIPAddress(connectedHostname, connectedAddr, connectedPort);
-    if(!e_->findCachedIPAddress(connectedHostname, connectedPort).empty()) {
-      A2_LOG_INFO(fmt(MSG_CONNECT_FAILED_AND_RETRY,
-                      getCuid(),
-                      connectedAddr.c_str(), connectedPort));
-      e_->setNoWait(true);
-      e_->addCommand(InitiateConnectionCommandFactory::
-                     createInitiateConnectionCommand
-                     (getCuid(), req_, fileEntry_, requestGroup_, e_));
-      return false;
-    }
+  if (error.empty()) {
+    return true;
+  }
+
+  // See also InitiateConnectionCommand::executeInternal()
+  e_->markBadIPAddress(connectedHostname, connectedAddr, connectedPort);
+  if (e_->findCachedIPAddress(connectedHostname, connectedPort).empty()) {
     e_->removeCachedIPAddress(connectedHostname, connectedPort);
     // Don't set error if proxy server is used and its method is GET.
-    if(resolveProxyMethod(req_->getProtocol()) != V_GET ||
-       !isProxyRequest(req_->getProtocol(), getOption())) {
-      e_->getRequestGroupMan()->getOrCreateServerStat
-        (req_->getHost(), req_->getProtocol())->setError();
+    if (resolveProxyMethod(req_->getProtocol()) != V_GET ||
+        !isProxyRequest(req_->getProtocol(), getOption())) {
+      e_->getRequestGroupMan()
+          ->getOrCreateServerStat(req_->getHost(), req_->getProtocol())
+          ->setError();
     }
-    throw DL_RETRY_EX
-      (fmt(MSG_ESTABLISHING_CONNECTION_FAILED, error.c_str()));
+    throw DL_RETRY_EX(fmt(MSG_ESTABLISHING_CONNECTION_FAILED, error.c_str()));
   }
-  return true;
+
+  A2_LOG_INFO(fmt(MSG_CONNECT_FAILED_AND_RETRY,
+                  getCuid(),
+                  connectedAddr.c_str(),
+                  connectedPort));
+  e_->setNoWait(true);
+  e_->addCommand(
+      InitiateConnectionCommandFactory::createInitiateConnectionCommand(
+          getCuid(), req_, fileEntry_, requestGroup_, e_));
+  return false;
 }
 
-const std::string& AbstractCommand::resolveProxyMethod
-(const std::string& protocol) const
+const std::string&
+AbstractCommand::resolveProxyMethod(const std::string& protocol) const
 {
-  if(getOption()->get(PREF_PROXY_METHOD) == V_TUNNEL ||
-     protocol == "https") {
+  if (getOption()->get(PREF_PROXY_METHOD) == V_TUNNEL || protocol == "https") {
     return V_TUNNEL;
-  } else {
-    return V_GET;
   }
+  return V_GET;
 }
 
 const std::shared_ptr<Option>& AbstractCommand::getOption() const
@@ -836,11 +899,11 @@ void AbstractCommand::createSocket()
 
 int32_t AbstractCommand::calculateMinSplitSize() const
 {
-  if(req_ && req_->isPipeliningEnabled()) {
+  if (req_ && req_->isPipeliningEnabled()) {
     return getDownloadContext()->getPieceLength();
-  } else {
-    return getOption()->getAsInt(PREF_MIN_SPLIT_SIZE);
   }
+
+  return getOption()->getAsInt(PREF_MIN_SPLIT_SIZE);
 }
 
 void AbstractCommand::setRequest(const std::shared_ptr<Request>& request)
@@ -863,7 +926,8 @@ void AbstractCommand::setSocket(const std::shared_ptr<SocketCore>& s)
   socket_ = s;
 }
 
-const std::shared_ptr<DownloadContext>& AbstractCommand::getDownloadContext() const
+const std::shared_ptr<DownloadContext>&
+AbstractCommand::getDownloadContext() const
 {
   return requestGroup_->getDownloadContext();
 }
@@ -880,10 +944,12 @@ const std::shared_ptr<PieceStorage>& AbstractCommand::getPieceStorage() const
 
 void AbstractCommand::checkSocketRecvBuffer()
 {
-  if(!socketRecvBuffer_->bufferEmpty()) {
-    setStatus(Command::STATUS_ONESHOT_REALTIME);
-    e_->setNoWait(true);
+  if (socketRecvBuffer_->bufferEmpty()) {
+    return;
   }
+
+  setStatus(Command::STATUS_ONESHOT_REALTIME);
+  e_->setNoWait(true);
 }
 
 void AbstractCommand::addCommandSelf()