Tests are actually not needed thanks to guarding statement

This commit is contained in:
Stephan Hadinger 2022-09-20 14:45:59 +02:00
parent 28920fdeb7
commit 44e03f014a

View File

@ -561,7 +561,7 @@ bool Z_attribute::equalsKey(const Z_attribute & attr2, bool ignore_key_suffix) c
bool Z_attribute::equalsId(uint16_t _cluster, uint16_t _attr_id, uint8_t suffix) const {
if (key_is_cmd || key_is_str) { return false; }
if ((this->cluster == _cluster) && (this->attr_id == _attr_id) && (!this->key_is_cmd)) {
if ((this->cluster == _cluster) && (this->attr_id == _attr_id)) {
if (suffix) {
if (key_suffix == suffix) { return true; }
} else {
@ -574,7 +574,7 @@ bool Z_attribute::equalsId(uint16_t _cluster, uint16_t _attr_id, uint8_t suffix)
bool Z_attribute::equalsCmd(uint16_t _cluster, uint8_t _cmd_id, bool _direction, bool cmd_general, uint8_t suffix) const {
if (!key_is_cmd || key_is_str) { return false; }
uint16_t _attr_id = _cmd_id | (_direction ? 0x100 : 0x000) | (cmd_general ? 0x200 : 0x000);
if ((this->cluster == _cluster) && (this->attr_id == _attr_id) && (this->key_is_cmd)) {
if ((this->cluster == _cluster) && (this->attr_id == _attr_id)) {
if (suffix) {
if (key_suffix == suffix) { return true; }
} else {