Remove PeerStorage:scheduleForcedChokeRound since it is not effective

src/DefaultBtInteractive.cc

@@ -189,7 +189,6 @@ void DefaultBtInteractive::doPostHandshakeProcessing()
   if (!metadataGetMode_) {
     addAllowedFastMessageToQueue();
   }
-  peerStorage_->scheduleForcedChokeRound();
   sendPendingMessage();
 }
 

src/DefaultPeerStorage.cc

@@ -62,8 +62,7 @@ DefaultPeerStorage::DefaultPeerStorage()
     : maxPeerListSize_(MAX_PEER_LIST_SIZE),
       seederStateChoke_(make_unique<BtSeederStateChoke>()),
       leecherStateChoke_(make_unique<BtLeecherStateChoke>()),
-      lastTransferStatMapUpdated_(Timer::zero()),
-      forceChokeRound_(false)
+      lastTransferStatMapUpdated_(Timer::zero())
 {
 }
 
@@ -285,21 +284,16 @@ void DefaultPeerStorage::returnPeer(const std::shared_ptr<Peer>& peer)
 bool DefaultPeerStorage::chokeRoundIntervalElapsed()
 {
   constexpr auto CHOKE_ROUND_INTERVAL = 10_s;
-  auto forceChokeRound = forceChokeRound_;
-  forceChokeRound_ = false;
 
   if (pieceStorage_->downloadFinished()) {
-    auto interval = forceChokeRound ? 1_s : CHOKE_ROUND_INTERVAL;
     return seederStateChoke_->getLastRound().difference(global::wallclock()) >=
-           interval;
+           CHOKE_ROUND_INTERVAL;
   }
 
   return leecherStateChoke_->getLastRound().difference(global::wallclock()) >=
          CHOKE_ROUND_INTERVAL;
 }
 
-void DefaultPeerStorage::scheduleForcedChokeRound() { forceChokeRound_ = true; }
-
 void DefaultPeerStorage::executeChoke()
 {
   if (pieceStorage_->downloadFinished()) {

src/DefaultPeerStorage.h

@@ -74,8 +74,6 @@ private:
   std::map<std::string, Timer> badPeers_;
   Timer lastBadPeerCleaned_;
 
-  bool forceChokeRound_;
-
   bool isPeerAlreadyAdded(const std::shared_ptr<Peer>& peer);
   void addUniqPeer(const std::shared_ptr<Peer>& peer);
 
@@ -115,8 +113,6 @@ public:
 
   virtual bool chokeRoundIntervalElapsed() CXX11_OVERRIDE;
 
-  virtual void scheduleForcedChokeRound() CXX11_OVERRIDE;
-
   virtual void executeChoke() CXX11_OVERRIDE;
 
   void deleteUnusedPeer(size_t delSize);

src/PeerStorage.h

@@ -113,11 +113,6 @@ public:
 
   virtual bool chokeRoundIntervalElapsed() = 0;
 
-  /**
-   * Schedules choke round forcibly.
-   */
-  virtual void scheduleForcedChokeRound() = 0;
-
   virtual void executeChoke() = 0;
 };
 

test/MockPeerStorage.h

@@ -84,8 +84,6 @@ public:
 
   virtual bool chokeRoundIntervalElapsed() CXX11_OVERRIDE { return false; }
 
-  virtual void scheduleForcedChokeRound() CXX11_OVERRIDE {}
-
   virtual void executeChoke() CXX11_OVERRIDE { ++numChokeExecuted_; }
 
   int getNumChokeExecuted() const { return numChokeExecuted_; }