diff options
| author | Konstantin Ritt <ritt.ks@gmail.com> | 2015-09-14 01:35:38 +0400 |
|---|---|---|
| committer | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-09-16 09:11:36 +0000 |
| commit | ff59294c0d7a7464870a7a7d3bc1449c7a7695a7 (patch) | |
| tree | 083cd922b83fbebfee3ef113e82264e8b5795f67 /src/bluetooth/qlowenergyservice.cpp | |
| parent | aa6cc482b09bf06d9baaaca1a02d600f8203c2a4 (diff) | |
A nano optimization to QLowEnergyService
Rearrange condition to fail earlier for an invalid service object.
Change-Id: I3ee0c611bab5610352ea4a058edd4af810212aaf
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Diffstat (limited to 'src/bluetooth/qlowenergyservice.cpp')
| -rw-r--r-- | src/bluetooth/qlowenergyservice.cpp | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/src/bluetooth/qlowenergyservice.cpp b/src/bluetooth/qlowenergyservice.cpp index 07590943..67bd7c77 100644 --- a/src/bluetooth/qlowenergyservice.cpp +++ b/src/bluetooth/qlowenergyservice.cpp @@ -597,9 +597,7 @@ void QLowEnergyService::readCharacteristic( { Q_D(QLowEnergyService); - if (!contains(characteristic) - || state() != ServiceDiscovered - || !d->controller) { + if (d->controller == Q_NULLPTR || state() != ServiceDiscovered || !contains(characteristic)) { d->setError(QLowEnergyService::OperationError); return; } @@ -648,9 +646,7 @@ void QLowEnergyService::writeCharacteristic( //TODO check behavior when writing to WriteSigned characteristic Q_D(QLowEnergyService); - if (!contains(characteristic) - || state() != ServiceDiscovered - || !d->controller) { + if (d->controller == Q_NULLPTR || state() != ServiceDiscovered || !contains(characteristic)) { d->setError(QLowEnergyService::OperationError); return; } @@ -716,9 +712,7 @@ void QLowEnergyService::readDescriptor( { Q_D(QLowEnergyService); - if (!contains(descriptor) - || state() != ServiceDiscovered - || !d->controller) { + if (d->controller == Q_NULLPTR || state() != ServiceDiscovered || !contains(descriptor)) { d->setError(QLowEnergyService::OperationError); return; } @@ -750,9 +744,7 @@ void QLowEnergyService::writeDescriptor(const QLowEnergyDescriptor &descriptor, { Q_D(QLowEnergyService); - if (!contains(descriptor) - || state() != ServiceDiscovered - || !d->controller) { + if (d->controller == Q_NULLPTR || state() != ServiceDiscovered || !contains(descriptor)) { d->setError(QLowEnergyService::OperationError); return; } |