(core_info.c) CLeanups

This commit is contained in:
twinaphex 2016-12-14 01:23:08 +01:00
parent c501ee6057
commit bcccc27205
2 changed files with 29 additions and 35 deletions

View File

@ -709,9 +709,11 @@ bool core_info_unsupported_content_path(const char *path)
{ {
const core_info_t *info = &core_info_curr_list->list[i]; const core_info_t *info = &core_info_curr_list->list[i];
if (string_list_find_elem(info->supported_extensions_list, "zip") || if ( !string_list_find_elem(info->supported_extensions_list, "zip")
string_list_find_elem(info->supported_extensions_list, "7z")) && !string_list_find_elem(info->supported_extensions_list, "7z"))
return false; continue;
return false;
} }
} }
@ -794,12 +796,15 @@ bool core_info_list_get_display_name(core_info_list_t *core_info_list,
for (i = 0; i < core_info_list->count; i++) for (i = 0; i < core_info_list->count; i++)
{ {
const core_info_t *info = &core_info_list->list[i]; const core_info_t *info = &core_info_list->list[i];
if (string_is_equal(path_basename(info->path), path_basename(path))
&& info->display_name) if (!string_is_equal(path_basename(info->path), path_basename(path)))
{ continue;
strlcpy(s, info->display_name, len);
return true; if (!info->display_name)
} continue;
strlcpy(s, info->display_name, len);
return true;
} }
return false; return false;
@ -807,40 +812,25 @@ bool core_info_list_get_display_name(core_info_list_t *core_info_list,
bool core_info_get_display_name(const char *path, char *s, size_t len) bool core_info_get_display_name(const char *path, char *s, size_t len)
{ {
char *core_name = NULL; bool ret = true;
char *display_name = NULL; char *display_name = NULL;
config_file_t *conf = NULL; config_file_t *conf = config_file_new(path);
if (!path_file_exists(path))
return false;
conf = config_file_new(path);
if (!conf) if (!conf)
return false; {
ret = false;
config_get_string(conf, "corename",
&core_name);
config_get_string(conf, "display_name",
&display_name);
if (!core_name || !display_name)
goto error; goto error;
}
config_file_free(conf); config_get_string(conf, "display_name", &display_name);
snprintf(s, len,"%s",display_name); if (display_name)
snprintf(s, len, "%s", display_name);
free(core_name);
free(display_name);
return true;
error: error:
config_file_free(conf); config_file_free(conf);
if (core_name)
free(core_name);
if (display_name) if (display_name)
free(display_name); free(display_name);
return false;
return ret;
} }

View File

@ -181,16 +181,20 @@ static void print_buf_lines(file_list_t *list, char *buf,
path_remove_extension(core_path); path_remove_extension(core_path);
last = (char*)strrchr(core_path, '_'); last = (char*)strrchr(core_path, '_');
if (!string_is_empty(last)) if (!string_is_empty(last))
{ {
if (!string_is_equal(last, "_libretro")) if (!string_is_equal(last, "_libretro"))
*last = '\0'; *last = '\0';
} }
strlcat(core_path, strlcat(core_path,
file_path_str(FILE_PATH_CORE_INFO_EXTENSION), file_path_str(FILE_PATH_CORE_INFO_EXTENSION),
sizeof(core_path)); sizeof(core_path));
if (core_info_get_display_name( if (
path_file_exists(core_path)
&& core_info_get_display_name(
core_path, display_name, sizeof(display_name))) core_path, display_name, sizeof(display_name)))
menu_entries_set_alt_at_offset(list, j, display_name); menu_entries_set_alt_at_offset(list, j, display_name);
} }