From 3a19b65dd77ff7a71c8ad277ec9ba3e17de7ee2d Mon Sep 17 00:00:00 2001 From: Tatsuhiro Tsujikawa Date: Sat, 26 Jan 2013 18:50:48 +0900 Subject: [PATCH] Rename /BadCondition/ in Peer to /Drop/ --- src/DefaultPeerStorage.cc | 2 +- src/Peer.cc | 6 +++--- src/Peer.h | 8 ++++---- src/UTPexExtensionMessage.cc | 3 +-- test/UTPexExtensionMessageTest.cc | 24 ++++++++++++------------ 5 files changed, 21 insertions(+), 22 deletions(-) diff --git a/src/DefaultPeerStorage.cc b/src/DefaultPeerStorage.cc index b8a0ce5a..a6bb908d 100644 --- a/src/DefaultPeerStorage.cc +++ b/src/DefaultPeerStorage.cc @@ -238,7 +238,7 @@ void DefaultPeerStorage::onReturningPeer(const SharedHandle& peer) { if(peer->isActive()) { if(peer->isDisconnectedGracefully() && !peer->isIncomingPeer()) { - peer->startBadCondition(); + peer->startDrop(); addDroppedPeer(peer); } // Execute choking algorithm if unchoked and interested peer is diff --git a/src/Peer.cc b/src/Peer.cc index 1ca02d91..4edcebb9 100644 --- a/src/Peer.cc +++ b/src/Peer.cc @@ -51,7 +51,7 @@ Peer::Peer(std::string ipaddr, uint16_t port, bool incoming): origPort_(port), cuid_(0), firstContactTime_(global::wallclock()), - badConditionStartTime_(0), + dropStartTime_(0), seeder_(false), res_(0), incoming_(incoming), @@ -317,9 +317,9 @@ void Peer::setAllBitfield() { updateSeeder(); } -void Peer::startBadCondition() +void Peer::startDrop() { - badConditionStartTime_ = global::wallclock(); + dropStartTime_ = global::wallclock(); } uint8_t Peer::getExtensionMessageID(int key) const diff --git a/src/Peer.h b/src/Peer.h index 072764ea..ec6a22ec 100644 --- a/src/Peer.h +++ b/src/Peer.h @@ -71,7 +71,7 @@ private: Timer firstContactTime_; - Timer badConditionStartTime_; + Timer dropStartTime_; bool seeder_; @@ -145,7 +145,7 @@ public: return seeder_; } - void startBadCondition(); + void startDrop(); void allocateSessionResource(int32_t pieceLength, int64_t totalLength); @@ -160,9 +160,9 @@ public: void setFirstContactTime(const Timer& time); - const Timer& getBadConditionStartTime() const + const Timer& getDropStartTime() const { - return badConditionStartTime_; + return dropStartTime_; } // Before calling following member functions, make sure that diff --git a/src/UTPexExtensionMessage.cc b/src/UTPexExtensionMessage.cc index 3de9e8fc..a5245772 100644 --- a/src/UTPexExtensionMessage.cc +++ b/src/UTPexExtensionMessage.cc @@ -151,8 +151,7 @@ bool UTPexExtensionMessage::freshPeersAreFull() const bool UTPexExtensionMessage::addDroppedPeer(const SharedHandle& peer) { if(!peer->isIncomingPeer() && - peer->getBadConditionStartTime(). - difference(global::wallclock()) < interval_) { + peer->getDropStartTime().difference(global::wallclock()) < interval_) { droppedPeers_.push_back(peer); return true; } else { diff --git a/test/UTPexExtensionMessageTest.cc b/test/UTPexExtensionMessageTest.cc index 04412fe7..e9589c71 100644 --- a/test/UTPexExtensionMessageTest.cc +++ b/test/UTPexExtensionMessageTest.cc @@ -77,17 +77,17 @@ void UTPexExtensionMessageTest::testGetBencodedData() SharedHandle p2(new Peer("10.1.1.2", 9999)); CPPUNIT_ASSERT(msg.addFreshPeer(p2)); SharedHandle p3(new Peer("192.168.0.2", 6882)); - p3->startBadCondition(); + p3->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p3)); SharedHandle p4(new Peer("10.1.1.3", 10000)); - p4->startBadCondition(); + p4->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p4)); SharedHandle p5(new Peer("1002:1035:4527:3546:7854:1237:3247:3217", 6881)); CPPUNIT_ASSERT(msg.addFreshPeer(p5)); SharedHandle p6(new Peer("2001:db8:bd05:1d2:288a:1fc0:1:10ee", 6882)); - p6->startBadCondition(); + p6->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p6)); unsigned char c1[COMPACT_LEN_IPV6]; @@ -126,10 +126,10 @@ void UTPexExtensionMessageTest::testToString() SharedHandle p2(new Peer("10.1.1.2", 9999)); msg.addFreshPeer(p2); SharedHandle p3(new Peer("192.168.0.2", 6882)); - p3->startBadCondition(); + p3->startDrop(); msg.addDroppedPeer(p3); SharedHandle p4(new Peer("10.1.1.3", 10000)); - p4->startBadCondition(); + p4->startDrop(); msg.addDroppedPeer(p4); CPPUNIT_ASSERT_EQUAL(std::string("ut_pex added=2, dropped=2"), msg.toString()); } @@ -145,10 +145,10 @@ void UTPexExtensionMessageTest::testDoReceivedAction() 9999)); msg.addFreshPeer(p2); SharedHandle p3(new Peer("192.168.0.2", 6882)); - p3->startBadCondition(); + p3->startDrop(); msg.addDroppedPeer(p3); SharedHandle p4(new Peer("2001:db8:bd05:1d2:288a:1fc0:1:10ee", 10000)); - p4->startBadCondition(); + p4->startDrop(); msg.addDroppedPeer(p4); msg.setPeerStorage(peerStorage_); @@ -259,10 +259,10 @@ void UTPexExtensionMessageTest::testAddDroppedPeer() SharedHandle p1(new Peer("192.168.0.1", 6881)); CPPUNIT_ASSERT(!msg.addDroppedPeer(p1)); SharedHandle p2(new Peer("10.1.1.2", 9999)); - p2->startBadCondition(); + p2->startDrop(); CPPUNIT_ASSERT(msg.addFreshPeer(p2)); SharedHandle p3(new Peer("10.1.1.3", 9999, true)); - p3->startBadCondition(); + p3->startDrop(); CPPUNIT_ASSERT(!msg.addDroppedPeer(p3)); } @@ -288,15 +288,15 @@ void UTPexExtensionMessageTest::testDroppedPeersAreFull() CPPUNIT_ASSERT_EQUAL((size_t)50, msg.getMaxDroppedPeer()); msg.setMaxDroppedPeer(2); SharedHandle p1(new Peer("192.168.0.1", 6881)); - p1->startBadCondition(); + p1->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p1)); CPPUNIT_ASSERT(!msg.droppedPeersAreFull()); SharedHandle p2(new Peer("10.1.1.2", 9999)); - p2->startBadCondition(); + p2->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p2)); CPPUNIT_ASSERT(msg.droppedPeersAreFull()); SharedHandle p3(new Peer("10.1.1.3", 9999)); - p3->startBadCondition(); + p3->startDrop(); CPPUNIT_ASSERT(msg.addDroppedPeer(p3)); CPPUNIT_ASSERT(msg.droppedPeersAreFull()); }