drath/libratbag
1
0
Fork 1
mirror of https://github.com/libratbag/libratbag.git synced 2026-11-07 15:14:38 +01:00
Code Issues Projects Releases Wiki Activity

sinowealth: simply return a bool instead of 0 or 1

Browse Source
This commit is contained in:
Yaroslav Chvanov
2022-11-26 16:23:01 +03:00
parent 007fa34c87
commit f23f2bfbfb
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/driver-sinowealth.c
View File

@@ -490,22 +490,20 @@ static const struct sinowealth_button_mapping sinowealth_button_map[] = {
};
/* Check if two given button data structs are equal.
*
* @return 1 if structs are equal or 0 otherwise.
*/
static int
static bool
sinowealth_button_data_is_equal(const struct sinowealth_button_data *lhs, const struct sinowealth_button_data *rhs)
{
if (lhs->type != rhs->type)
return 0;
return false;
for (unsigned int i = 0; i < sizeof(lhs->data); ++i) {
if (lhs->data[i] != rhs->data[i]) {
return 0;
return false;
}
}
return 1;
return true;
}
/* Convert a button action to raw data using the `sinowealth_button_map`.
@@ -542,7 +540,7 @@ sinowealth_raw_to_button_action(const struct sinowealth_button_data *data)
const struct sinowealth_button_mapping *mapping = NULL;
ARRAY_FOR_EACH(sinowealth_button_map, mapping) {
if (sinowealth_button_data_is_equal(data, &mapping->data) == 0)
if (!sinowealth_button_data_is_equal(data, &mapping->data))
continue;
return &mapping->action;