I am not a big fan of doing assignment in a return statement. Split it into two lines.
Showing the changes to scripts (1 of 1 file). Show all
Unified · Split
| @@ -65,7 +65,8 @@ void menu_add_entry(struct symbol *sym) · 64 unmodified lines | |||
| 65 | struct menu *menu_add_menu(void) | 65 | struct menu *menu_add_menu(void) |
| 66 | { | 66 | { |
| 67 | last_entry_ptr = ¤t_entry->list; | 67 | last_entry_ptr = ¤t_entry->list; |
| 68 | return current_menu = current_entry; | 68 | current_menu = current_entry; |
| 69 | return current_menu; | ||
| 69 | } | 70 | } |
| 70 | 71 | ||
| 71 | void menu_end_menu(void) | 72 | void menu_end_menu(void) |