From def41a573f1e23d0e723e01b8949d55dd1862acd Mon Sep 17 00:00:00 2001 From: twinaphex Date: Mon, 20 Oct 2014 19:18:18 +0200 Subject: [PATCH] Avoid using 'index' for local variables --- frontend/menu/backend/menu_backend.h | 2 +- frontend/menu/backend/menu_common_backend.c | 20 ++++++++++---------- frontend/menu/disp/rmenu_xui.cpp | 4 ++-- frontend/menu/disp/xmb.c | 8 ++++---- frontend/menu/menu_common.c | 4 ++-- frontend/menu/menu_entries_cbs.h | 2 +- frontend/menu/menu_input_line_cb.c | 6 +++--- frontend/menu/menu_list.c | 16 ++++++++-------- frontend/menu/menu_list.h | 8 ++++---- 9 files changed, 35 insertions(+), 35 deletions(-) diff --git a/frontend/menu/backend/menu_backend.h b/frontend/menu/backend/menu_backend.h index e93dae9535..8b47c35dd1 100644 --- a/frontend/menu/backend/menu_backend.h +++ b/frontend/menu/backend/menu_backend.h @@ -10,7 +10,7 @@ typedef struct menu_file_list_cbs int (*action_deferred_push)(void *data, void *userdata, const char *path, const char *label, unsigned type); int (*action_ok)(const char *path, const char *label, unsigned type, - size_t index); + size_t idx); int (*action_start)(unsigned type, const char *label, unsigned action); int (*action_toggle)(unsigned type, const char *label, unsigned action); } menu_file_list_cbs_t; diff --git a/frontend/menu/backend/menu_common_backend.c b/frontend/menu/backend/menu_common_backend.c index f8cafa1d66..0c58b26644 100644 --- a/frontend/menu/backend/menu_common_backend.c +++ b/frontend/menu/backend/menu_common_backend.c @@ -674,34 +674,34 @@ static int menu_common_iterate(unsigned action) static void menu_common_setting_set_label(char *type_str, size_t type_str_size, unsigned *w, unsigned type, - const char *menu_label, const char *label, unsigned index) + const char *menu_label, const char *label, unsigned idx) { setting_data_get_label(type_str, type_str_size, w, - type, menu_label, label, index); + type, menu_label, label, idx); } static void menu_common_list_insert(void *data, const char *path, const char *label, - unsigned type, size_t index) + unsigned type, size_t idx) { file_list_t *list = (file_list_t*)data; if (!list) return; - list->list[index].actiondata = (menu_file_list_cbs_t*) + list->list[idx].actiondata = (menu_file_list_cbs_t*) calloc(1, sizeof(menu_file_list_cbs_t)); - if (!list->list[index].actiondata) + if (!list->list[idx].actiondata) { RARCH_ERR("Action data could not be allocated.\n"); return; } - menu_entries_cbs_init(list, path, label, type, index); + menu_entries_cbs_init(list, path, label, type, idx); } -static void menu_common_list_delete(void *data, size_t index, +static void menu_common_list_delete(void *data, size_t idx, size_t list_size) { menu_file_list_cbs_t *cbs = NULL; @@ -710,7 +710,7 @@ static void menu_common_list_delete(void *data, size_t index, if (!list) return; - cbs = (menu_file_list_cbs_t*)list->list[index].actiondata; + cbs = (menu_file_list_cbs_t*)list->list[idx].actiondata; if (cbs) { @@ -718,9 +718,9 @@ static void menu_common_list_delete(void *data, size_t index, cbs->action_ok = NULL; cbs->action_toggle = NULL; cbs->action_deferred_push = NULL; - free(list->list[index].actiondata); + free(list->list[idx].actiondata); } - list->list[index].actiondata = NULL; + list->list[idx].actiondata = NULL; } static void menu_common_list_clear(void *data) diff --git a/frontend/menu/disp/rmenu_xui.cpp b/frontend/menu/disp/rmenu_xui.cpp index 13ccb27871..afbcaa7300 100644 --- a/frontend/menu/disp/rmenu_xui.cpp +++ b/frontend/menu/disp/rmenu_xui.cpp @@ -498,11 +498,11 @@ static void rmenu_xui_list_insert(void *data, XuiListSetText(m_menulist, list_size, buf); } -static void rmenu_xui_list_delete(void *data, size_t index, +static void rmenu_xui_list_delete(void *data, size_t idx, size_t list_size) { (void)data; - (void)index; + (void)idx; XuiListDeleteItems(m_menulist, 0, list_size); } diff --git a/frontend/menu/disp/xmb.c b/frontend/menu/disp/xmb.c index 9025984d78..295ad856d1 100644 --- a/frontend/menu/disp/xmb.c +++ b/frontend/menu/disp/xmb.c @@ -914,7 +914,7 @@ static void xmb_list_insert(void *data, node->y = iy; } -static void xmb_list_delete(void *data, size_t index, +static void xmb_list_delete(void *data, size_t idx, size_t list_size) { file_list_t *list = (file_list_t*)data; @@ -922,9 +922,9 @@ static void xmb_list_delete(void *data, size_t index, if (!list) return; - if (list->list[index].userdata) - free(list->list[index].userdata); - list->list[index].userdata = NULL; + if (list->list[idx].userdata) + free(list->list[idx].userdata); + list->list[idx].userdata = NULL; } static void xmb_list_clear(void *data) diff --git a/frontend/menu/menu_common.c b/frontend/menu/menu_common.c index 913db718c6..c577ae03a4 100644 --- a/frontend/menu/menu_common.c +++ b/frontend/menu/menu_common.c @@ -238,7 +238,7 @@ void menu_free(void *data) free(data); } -void menu_ticker_line(char *buf, size_t len, unsigned index, +void menu_ticker_line(char *buf, size_t len, unsigned idx, const char *str, bool selected) { size_t str_len = strlen(str); @@ -259,7 +259,7 @@ void menu_ticker_line(char *buf, size_t len, unsigned index, { /* Wrap long strings in options with some kind of ticker line. */ unsigned ticker_period = 2 * (str_len - len) + 4; - unsigned phase = index % ticker_period; + unsigned phase = idx % ticker_period; unsigned phase_left_stop = 2; unsigned phase_left_moving = phase_left_stop + (str_len - len); diff --git a/frontend/menu/menu_entries_cbs.h b/frontend/menu/menu_entries_cbs.h index 17b97523d1..98e6d6ba2c 100644 --- a/frontend/menu/menu_entries_cbs.h +++ b/frontend/menu/menu_entries_cbs.h @@ -24,7 +24,7 @@ extern "C" { void menu_entries_cbs_init(void *data, const char *path, const char *label, - unsigned type, size_t index); + unsigned type, size_t idx); #ifdef __cplusplus } diff --git a/frontend/menu/menu_input_line_cb.c b/frontend/menu/menu_input_line_cb.c index dceaa802f5..c621f67b0a 100644 --- a/frontend/menu/menu_input_line_cb.c +++ b/frontend/menu/menu_input_line_cb.c @@ -63,10 +63,10 @@ static void menu_key_end_line(void *data) static void menu_search_callback(void *userdata, const char *str) { menu_handle_t *menu = (menu_handle_t*)userdata; - size_t index; + size_t idx; - if (str && *str && file_list_search(menu->menu_list->selection_buf, str, &index)) - menu_navigation_set(menu, index); + if (str && *str && file_list_search(menu->menu_list->selection_buf, str, &idx)) + menu_navigation_set(menu, idx); menu_key_end_line(menu); } diff --git a/frontend/menu/menu_list.c b/frontend/menu/menu_list.c index 6469a67f06..0d592a0ed6 100644 --- a/frontend/menu/menu_list.c +++ b/frontend/menu/menu_list.c @@ -74,10 +74,10 @@ size_t menu_list_get_stack_size(menu_list_t *list) return file_list_get_size(list->menu_stack); } -void menu_list_get_at_offset(const file_list_t *list, size_t index, +void menu_list_get_at_offset(const file_list_t *list, size_t idx, const char **path, const char **label, unsigned *file_type) { - file_list_get_at_offset(list, index, path, label, file_type); + file_list_get_at_offset(list, idx, path, label, file_type); } size_t menu_list_get_size(menu_list_t *list) @@ -103,10 +103,10 @@ void menu_list_get_last_stack(const menu_list_t *list, file_list_get_last(list->menu_stack, path, label, file_type); } -void *menu_list_get_actiondata_at_offset(const file_list_t *list, size_t index) +void *menu_list_get_actiondata_at_offset(const file_list_t *list, size_t idx) { if (list) - return file_list_get_actiondata_at_offset(list, index); + return file_list_get_actiondata_at_offset(list, idx); return NULL; } @@ -281,16 +281,16 @@ void menu_list_push_stack_refresh(menu_list_t *list, driver.menu->need_refresh = true; } -void menu_list_set_alt_at_offset(file_list_t *list, size_t index, +void menu_list_set_alt_at_offset(file_list_t *list, size_t idx, const char *alt) { - file_list_set_alt_at_offset(list, index, alt); + file_list_set_alt_at_offset(list, idx, alt); } -void menu_list_get_alt_at_offset(const file_list_t *list, size_t index, +void menu_list_get_alt_at_offset(const file_list_t *list, size_t idx, const char **alt) { - file_list_get_alt_at_offset(list, index, alt); + file_list_get_alt_at_offset(list, idx, alt); } void menu_list_sort_on_alt(file_list_t *list) diff --git a/frontend/menu/menu_list.h b/frontend/menu/menu_list.h index 692cdc95ea..3f78d0f02d 100644 --- a/frontend/menu/menu_list.h +++ b/frontend/menu/menu_list.h @@ -48,10 +48,10 @@ void menu_list_pop_stack(menu_list_t *list); void menu_list_pop_stack_by_needle(menu_list_t *list, const char *needle); -void menu_list_get_at_offset(const file_list_t *list, size_t index, +void menu_list_get_at_offset(const file_list_t *list, size_t idx, const char **path, const char **label, unsigned *file_type); -void *menu_list_get_actiondata_at_offset(const file_list_t *list, size_t index); +void *menu_list_get_actiondata_at_offset(const file_list_t *list, size_t idx); size_t menu_list_get_stack_size(menu_list_t *list); @@ -87,10 +87,10 @@ void menu_list_push_stack_refresh(menu_list_t *list, const char *path, const char *label, unsigned type, size_t directory_ptr); -void menu_list_get_alt_at_offset(const file_list_t *list, size_t index, +void menu_list_get_alt_at_offset(const file_list_t *list, size_t idx, const char **alt); -void menu_list_set_alt_at_offset(file_list_t *list, size_t index, +void menu_list_set_alt_at_offset(file_list_t *list, size_t idx, const char *alt); #ifdef __cplusplus