Tidy some method names per reviewer comment

This commit is contained in:
Simon Walton 2018-08-20 17:50:29 -07:00
parent a891b74fac
commit 8d6aaf3fb5
3 changed files with 8 additions and 8 deletions

View file

@ -112,7 +112,7 @@ void OctreePacketProcessor::processPacket(QSharedPointer<ReceivedMessage> messag
auto renderer = qApp->getEntities();
if (renderer) {
renderer->processDatagram(*message, sendingNode);
_safeLanding->sequenceNumberReceived(renderer->getLastOctreeMessageSequence());
_safeLanding->noteReceivedsequenceNumber(renderer->getLastOctreeMessageSequence());
}
}
} break;

View file

@ -95,7 +95,7 @@ void SafeLanding::setCompletionSequenceNumbers(int first, int last) {
}
}
void SafeLanding::sequenceNumberReceived(int sequenceNumber) {
void SafeLanding::noteReceivedsequenceNumber(int sequenceNumber) {
if (_trackingEntities) {
Locker lock(_lock);
_sequenceNumbers.insert(sequenceNumber);
@ -103,7 +103,7 @@ void SafeLanding::sequenceNumberReceived(int sequenceNumber) {
}
bool SafeLanding::isLoadSequenceComplete() {
if (entityPhysicsComplete() && sequenceNumbersComplete()) {
if (isEntityPhysicsComplete() && isSequenceNumbersComplete()) {
Locker lock(_lock);
_trackedEntities.clear();
_initialStart = INVALID_SEQUENCE;
@ -116,7 +116,7 @@ bool SafeLanding::isLoadSequenceComplete() {
return !_trackingEntities;
}
bool SafeLanding::sequenceNumbersComplete() {
bool SafeLanding::isSequenceNumbersComplete() {
if (_initialStart != INVALID_SEQUENCE) {
Locker lock(_lock);
int sequenceSize = _initialStart <= _initialEnd ? _initialEnd - _initialStart:
@ -134,7 +134,7 @@ bool SafeLanding::sequenceNumbersComplete() {
return false;
}
bool SafeLanding::entityPhysicsComplete() {
bool SafeLanding::isEntityPhysicsComplete() {
Locker lock(_lock);
for (auto entityMapIter = _trackedEntities.begin(); entityMapIter != _trackedEntities.end(); ++entityMapIter) {
auto entity = entityMapIter->second;

View file

@ -27,7 +27,7 @@ public:
void startEntitySequence(QSharedPointer<EntityTreeRenderer> entityTreeRenderer);
void stopEntitySequence();
void setCompletionSequenceNumbers(int first, int last);
void sequenceNumberReceived(int sequenceNumber);
void noteReceivedsequenceNumber(int sequenceNumber);
bool isLoadSequenceComplete();
private slots:
@ -35,9 +35,9 @@ private slots:
void deleteTrackedEntity(const EntityItemID& entityID);
private:
bool sequenceNumbersComplete();
bool isSequenceNumbersComplete();
void debugDumpSequenceIDs() const;
bool entityPhysicsComplete();
bool isEntityPhysicsComplete();
std::mutex _lock;
using Locker = std::lock_guard<std::mutex>;