mirror of
https://github.com/esphome/esphome.git
synced 2025-10-07 12:23:47 +01:00
[esp32_ble_server] Use early returns in is_created() and is_failed() methods
This commit is contained in:
@@ -125,26 +125,25 @@ bool BLECharacteristic::is_created() {
|
||||
if (this->state_ != CREATING_DEPENDENTS)
|
||||
return false;
|
||||
|
||||
bool created = true;
|
||||
for (auto *descriptor : this->descriptors_) {
|
||||
created &= descriptor->is_created();
|
||||
if (!descriptor->is_created())
|
||||
return false;
|
||||
}
|
||||
if (created)
|
||||
this->state_ = CREATED;
|
||||
return this->state_ == CREATED;
|
||||
this->state_ = CREATED;
|
||||
return true;
|
||||
}
|
||||
|
||||
bool BLECharacteristic::is_failed() {
|
||||
if (this->state_ == FAILED)
|
||||
return true;
|
||||
|
||||
bool failed = false;
|
||||
for (auto *descriptor : this->descriptors_) {
|
||||
failed |= descriptor->is_failed();
|
||||
if (descriptor->is_failed()) {
|
||||
this->state_ = FAILED;
|
||||
return true;
|
||||
}
|
||||
}
|
||||
if (failed)
|
||||
this->state_ = FAILED;
|
||||
return this->state_ == FAILED;
|
||||
return false;
|
||||
}
|
||||
|
||||
void BLECharacteristic::set_broadcast_property(bool value) {
|
||||
|
Reference in New Issue
Block a user