From 2dd96b4f393ef83d680c7964d6c2f906c99c01f5 Mon Sep 17 00:00:00 2001 From: Tomaz Canabrava Date: Sat, 10 May 2014 21:51:38 -0300 Subject: [PATCH] Simplify code. this is not necessary when you don't have a variable that has the exact same name as the 'this->variable' name, also, there's no need to check if a QString is empty before trying to use the == on them. Signed-off-by: Tomaz Canabrava Signed-off-by: Dirk Hohndel --- qthelper.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/qthelper.cpp b/qthelper.cpp index a421d7f66..c51e13628 100644 --- a/qthelper.cpp +++ b/qthelper.cpp @@ -25,11 +25,11 @@ DiveComputerList::~DiveComputerList() bool DiveComputerNode::operator==(const DiveComputerNode &a) const { - return this->model == a.model && - this->deviceId == a.deviceId && - this->firmware == a.firmware && - this->serialNumber == a.serialNumber && - this->nickName == a.nickName; + return model == a.model && + deviceId == a.deviceId && + firmware == a.firmware && + serialNumber == a.serialNumber && + nickName == a.nickName; } bool DiveComputerNode::operator!=(const DiveComputerNode &a) const @@ -39,13 +39,13 @@ bool DiveComputerNode::operator!=(const DiveComputerNode &a) const bool DiveComputerNode::changesValues(const DiveComputerNode &b) const { - if (this->model != b.model || this->deviceId != b.deviceId) { + if (model != b.model || deviceId != b.deviceId) { qDebug("DiveComputerNodes were not for the same DC"); return false; } - return (b.firmware != "" && this->firmware != b.firmware) || - (b.serialNumber != "" && this->serialNumber != b.serialNumber) || - (b.nickName != "" && this->nickName != b.nickName); + return (firmware != b.firmware) || + (serialNumber != b.serialNumber) || + (nickName != b.nickName); } const DiveComputerNode *DiveComputerList::getExact(QString m, uint32_t d)