Looks like unnecessary logic

This commit is contained in:
twinaphex 2020-02-02 17:32:14 +01:00
parent 28b2a34191
commit cde2d7c18c
3 changed files with 6 additions and 20 deletions

View File

@ -6546,13 +6546,9 @@ static int materialui_menu_entry_action(
menu_entry_t *entry_ptr = entry;
size_t selection = i;
size_t new_selection;
enum menu_action new_action;
if (!mui)
generic_menu_entry_action(userdata, entry, i, action);
/* Process input action */
new_action = materialui_parse_menu_entry_action(mui, action);
enum menu_action new_action = materialui_parse_menu_entry_action(
mui, action);
/* NOTE: It would make sense to stop here if the
* resultant action is a NOOP (MENU_ACTION_NOOP),

View File

@ -2498,14 +2498,9 @@ static int ozone_menu_entry_action(
void *userdata, menu_entry_t *entry,
size_t i, enum menu_action action)
{
ozone_handle_t *ozone = (ozone_handle_t*)userdata;
enum menu_action new_action;
if (!ozone)
generic_menu_entry_action(userdata, entry, i, action);
ozone_handle_t *ozone = (ozone_handle_t*)userdata;
/* Process input action */
new_action = ozone_parse_menu_entry_action(ozone, action);
enum menu_action new_action = ozone_parse_menu_entry_action(ozone, action);
/* Call standard generic_menu_entry_action() function */
return generic_menu_entry_action(userdata, entry, i, new_action);

View File

@ -6577,14 +6577,9 @@ static int xmb_menu_entry_action(
void *userdata, menu_entry_t *entry,
size_t i, enum menu_action action)
{
xmb_handle_t *xmb = (xmb_handle_t*)userdata;
enum menu_action new_action;
if (!xmb)
generic_menu_entry_action(userdata, entry, i, action);
xmb_handle_t *xmb = (xmb_handle_t*)userdata;
/* Process input action */
new_action = xmb_parse_menu_entry_action(xmb, action);
enum menu_action new_action = xmb_parse_menu_entry_action(xmb, action);
/* Call standard generic_menu_entry_action() function */
return generic_menu_entry_action(userdata, entry, i, new_action);