From 879b8a2e45ecd3e7b41a7bae4e6f9ccef31fa67c Mon Sep 17 00:00:00 2001 From: Minje Ahn Date: Tue, 27 Oct 2020 13:05:18 +0900 Subject: [PATCH] Use g_list_free_full() instead Change-Id: Icc79163cef871034a2ee33579f61176f1de34e0c Signed-off-by: Minje Ahn --- utils/mm_file_util_tag.c | 30 +++++++++++------------------- 1 file changed, 11 insertions(+), 19 deletions(-) diff --git a/utils/mm_file_util_tag.c b/utils/mm_file_util_tag.c index b699778..050cf28 100644 --- a/utils/mm_file_util_tag.c +++ b/utils/mm_file_util_tag.c @@ -3537,31 +3537,23 @@ void mm_file_id3tag_restore_content_info(AvFileContentInfo *pInfo) } } -void mm_file_free_synclyrics_list(GList *synclyrics_list) +static void __free_synclyrics(gpointer data) { - int list_len = 0; - int idx = 0; - AvSynclyricsInfo *synclyrics_info = NULL; + AvSynclyricsInfo *info = (AvSynclyricsInfo *) data; - if (synclyrics_list == NULL) { + if (!info) return; - } - list_len = g_list_length(synclyrics_list); - for (idx = 0; idx < list_len; idx++) { - synclyrics_info = g_list_nth_data(synclyrics_list, idx); + mmfile_free(info->lyric_info); + mmfile_free(info); +} - if (synclyrics_info != NULL) { - mmfile_free(synclyrics_info->lyric_info); - mmfile_free(synclyrics_info); - } - } - if (synclyrics_list != NULL) { - g_list_free(synclyrics_list); - synclyrics_list = NULL; - } +void mm_file_free_synclyrics_list(GList *synclyrics_list) +{ + if (!synclyrics_list) + return; - return; + g_list_free_full(synclyrics_list, __free_synclyrics); } -- 2.7.4