Address feedback

This commit is contained in:
twinaphex 2021-10-15 13:54:04 +02:00
parent a16df36b28
commit af40baa04d
2 changed files with 8 additions and 8 deletions

View File

@ -6856,9 +6856,8 @@ void retroarch_menu_running(void)
{ {
runloop_state_t *runloop_st = runloop_state_get_ptr(); runloop_state_t *runloop_st = runloop_state_get_ptr();
video_driver_state_t *video_st = video_state_get_ptr(); video_driver_state_t *video_st = video_state_get_ptr();
#if defined(HAVE_MENU) || defined(HAVE_OVERLAY)
settings_t *settings = config_get_ptr(); settings_t *settings = config_get_ptr();
#endif input_driver_state_t *input_st = input_state_get_ptr();
#ifdef HAVE_OVERLAY #ifdef HAVE_OVERLAY
bool input_overlay_hide_in_menu = settings->bools.input_overlay_hide_in_menu; bool input_overlay_hide_in_menu = settings->bools.input_overlay_hide_in_menu;
#endif #endif
@ -6866,7 +6865,6 @@ void retroarch_menu_running(void)
bool audio_enable_menu = settings->bools.audio_enable_menu; bool audio_enable_menu = settings->bools.audio_enable_menu;
bool audio_enable_menu_bgm = settings->bools.audio_enable_menu_bgm; bool audio_enable_menu_bgm = settings->bools.audio_enable_menu_bgm;
#endif #endif
input_driver_state_t *input_st = input_state_get_ptr();
#ifdef HAVE_MENU #ifdef HAVE_MENU
struct menu_state *menu_st = &menu_driver_state; struct menu_state *menu_st = &menu_driver_state;
menu_handle_t *menu = menu_st->driver_data; menu_handle_t *menu = menu_st->driver_data;

View File

@ -11865,7 +11865,7 @@ static bool retroarch_environment_cb(unsigned cmd, void *data)
retro_time_t core_limit = (core_fps retro_time_t core_limit = (core_fps
? (retro_time_t)(1000000.0f / core_fps) ? (retro_time_t)(1000000.0f / core_fps)
: (retro_time_t)0); : (retro_time_t)0);
runloop_state_t *runloop_st = &runloop_state; runloop_state_t *runloop_st = &runloop_state;
retro_time_t frame_limit = runloop_st->frame_limit_minimum_time; retro_time_t frame_limit = runloop_st->frame_limit_minimum_time;
if (abs((int)(core_limit - frame_limit)) > 10) if (abs((int)(core_limit - frame_limit)) > 10)
{ {
@ -12025,6 +12025,7 @@ static void libretro_get_environment_info(
} }
static dylib_t load_dynamic_core( static dylib_t load_dynamic_core(
struct rarch_state *p_rarch,
const char *path, char *buf, size_t size) const char *path, char *buf, size_t size)
{ {
#if defined(ANDROID) #if defined(ANDROID)
@ -12049,7 +12050,7 @@ static dylib_t load_dynamic_core(
RARCH_ERR("This could happen if other modules RetroArch depends on " RARCH_ERR("This could happen if other modules RetroArch depends on "
"link against libretro directly.\n"); "link against libretro directly.\n");
RARCH_ERR("Proceeding could cause a crash. Aborting ...\n"); RARCH_ERR("Proceeding could cause a crash. Aborting ...\n");
return NULL; retroarch_fail(p_rarch, 1, "load_dynamic_core()");
} }
#endif #endif
@ -12233,6 +12234,7 @@ static bool init_libretro_symbols_custom(
path); path);
if (!(p_rarch->lib_handle = load_dynamic_core( if (!(p_rarch->lib_handle = load_dynamic_core(
p_rarch,
path, path,
path_get_ptr(RARCH_PATH_CORE), path_get_ptr(RARCH_PATH_CORE),
path_get_realsize(RARCH_PATH_CORE) path_get_realsize(RARCH_PATH_CORE)
@ -12241,7 +12243,6 @@ static bool init_libretro_symbols_custom(
RARCH_ERR("%s: \"%s\"\nError(s): %s\n", RARCH_ERR("%s: \"%s\"\nError(s): %s\n",
msg_hash_to_str(MSG_FAILED_TO_OPEN_LIBRETRO_CORE), msg_hash_to_str(MSG_FAILED_TO_OPEN_LIBRETRO_CORE),
path, dylib_error()); path, dylib_error());
retroarch_fail(p_rarch, 1, "load_dynamic_core()");
runloop_msg_queue_push(msg_hash_to_str(MSG_FAILED_TO_OPEN_LIBRETRO_CORE), runloop_msg_queue_push(msg_hash_to_str(MSG_FAILED_TO_OPEN_LIBRETRO_CORE),
1, 180, true, NULL, MESSAGE_QUEUE_ICON_DEFAULT, MESSAGE_QUEUE_CATEGORY_INFO); 1, 180, true, NULL, MESSAGE_QUEUE_ICON_DEFAULT, MESSAGE_QUEUE_CATEGORY_INFO);
return false; return false;
@ -20288,10 +20289,11 @@ static enum runloop_state_enum runloop_check_state(
current_bits, RARCH_FAST_FORWARD_KEY); current_bits, RARCH_FAST_FORWARD_KEY);
bool new_hold_button_state = BIT256_GET( bool new_hold_button_state = BIT256_GET(
current_bits, RARCH_FAST_FORWARD_HOLD_KEY); current_bits, RARCH_FAST_FORWARD_HOLD_KEY);
bool check2 = new_button_state && !old_button_state; bool check2 = new_button_state
&& !old_button_state;
if (!check2) if (!check2)
check2 = old_hold_button_state != new_hold_button_state; check2 = old_hold_button_state != new_hold_button_state;
if (check2) if (check2)
{ {