From fdbaf7df597b404be04cecbdc83dbc0551a5b996 Mon Sep 17 00:00:00 2001 From: William Wilgus Date: Fri, 5 Jul 2024 17:13:10 -0400 Subject: [Feature] playlist_viewer id3 title display Not sure this is a great idea from disk and battery standpoint but there is no reason you can't.. using the name buffer to fill title data prevent hitting the disk for each screen scroll add get_metadata_ex to allow flags METADATA_EXCLUDE_ID3_PATH prevent copying the filename to the ID3 struct METADATA_CLOSE_FD_ON_EXIT instead of seeking to the beginning the file is closed before get_metadata returns add logic to allow a invalid fd to signal that get_metadata should open and close the file within its call bugfix per Chris_s don't use the tagcache for the trackinfo Change-Id: Ic7a595b39a8d7a57f975312bc9c8bb4111f22a88 --- apps/iap/iap-core.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'apps/iap/iap-core.c') diff --git a/apps/iap/iap-core.c b/apps/iap/iap-core.c index da04a67311..3faf8237bf 100644 --- a/apps/iap/iap-core.c +++ b/apps/iap/iap-core.c @@ -685,7 +685,6 @@ bool iap_getc(const unsigned char x) void iap_get_trackinfo(const unsigned int track, struct mp3entry* id3) { int tracknum; - int fd; struct playlist_track_info info; tracknum = track; @@ -699,10 +698,8 @@ void iap_get_trackinfo(const unsigned int track, struct mp3entry* id3) if(playlist_next(0) != tracknum) { playlist_get_track_info(NULL, tracknum, &info); - fd = open(info.filename, O_RDONLY); - memset(id3, 0, sizeof(*id3)); - get_metadata(id3, fd, info.filename); - close(fd); + /* memset(id3, 0, sizeof(*id3)) --get_metadata does this for us */ + get_metadata(id3, -1, info.filename); } else { memcpy(id3, audio_current_track(), sizeof(*id3)); } -- cgit v1.2.3