diff options
author | Aidan MacDonald <amachronic@protonmail.com> | 2022-11-30 14:50:18 +0000 |
---|---|---|
committer | Aidan MacDonald <amachronic@protonmail.com> | 2022-12-17 13:36:38 +0000 |
commit | 701e262d3d622898279167ba909da631ac460bc1 (patch) | |
tree | 763be2f80bd12dde2b6c743a023ad331959d891b /apps/gui/skin_engine/skin_parser.c | |
parent | 767ddef550ec193c087064291b78d0e1db5e8fb0 (diff) | |
download | rockbox-701e262d3d622898279167ba909da631ac460bc1.tar.gz rockbox-701e262d3d622898279167ba909da631ac460bc1.zip |
settings: Remove setting ID return from find_setting()
Now that all users have replaced setting IDs with direct pointers,
find_setting() and friends don't need to return an ID value.
Change-Id: I8c5c31bb68d3bca5350d43538335265a55fd5517
Diffstat (limited to 'apps/gui/skin_engine/skin_parser.c')
-rw-r--r-- | apps/gui/skin_engine/skin_parser.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/apps/gui/skin_engine/skin_parser.c b/apps/gui/skin_engine/skin_parser.c index 435bad9e1f..ad5fa48756 100644 --- a/apps/gui/skin_engine/skin_parser.c +++ b/apps/gui/skin_engine/skin_parser.c | |||
@@ -823,7 +823,7 @@ static int parse_setting_and_lang(struct skin_element *element, | |||
823 | else | 823 | else |
824 | { | 824 | { |
825 | #ifndef __PCTOOL__ | 825 | #ifndef __PCTOOL__ |
826 | const struct settings_list *setting = find_setting_by_cfgname(temp, NULL); | 826 | const struct settings_list *setting = find_setting_by_cfgname(temp); |
827 | if (!setting) | 827 | if (!setting) |
828 | return WPS_ERROR_INVALID_PARAM; | 828 | return WPS_ERROR_INVALID_PARAM; |
829 | 829 | ||
@@ -1158,7 +1158,7 @@ static int parse_progressbar_tag(struct skin_element* element, | |||
1158 | param++; | 1158 | param++; |
1159 | text = SKINOFFSETTOPTR(skin_buffer, param->data.text); | 1159 | text = SKINOFFSETTOPTR(skin_buffer, param->data.text); |
1160 | #ifndef __PCTOOL__ | 1160 | #ifndef __PCTOOL__ |
1161 | pb->setting = find_setting_by_cfgname(text, NULL); | 1161 | pb->setting = find_setting_by_cfgname(text); |
1162 | if (!pb->setting) | 1162 | if (!pb->setting) |
1163 | return WPS_ERROR_INVALID_PARAM; | 1163 | return WPS_ERROR_INVALID_PARAM; |
1164 | #endif | 1164 | #endif |
@@ -1548,7 +1548,7 @@ static int touchregion_setup_setting(struct skin_element *element, int param_no, | |||
1548 | #ifndef __PCTOOL__ | 1548 | #ifndef __PCTOOL__ |
1549 | int p = param_no; | 1549 | int p = param_no; |
1550 | char *name = get_param_text(element, p++); | 1550 | char *name = get_param_text(element, p++); |
1551 | const struct settings_list *setting = find_setting_by_cfgname(name, NULL); | 1551 | const struct settings_list *setting = find_setting_by_cfgname(name); |
1552 | if (!setting) | 1552 | if (!setting) |
1553 | return WPS_ERROR_INVALID_PARAM; | 1553 | return WPS_ERROR_INVALID_PARAM; |
1554 | 1554 | ||