len0rd/rockbox
1
0
Fork 1
mirror of https://github.com/Rockbox/rockbox.git synced 2025-11-09 13:12:37 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Playlist Viewer: Eliminate 'dirty' flag

Browse source 
A playlist's explicit 'modified' flag is now used
for keeping track of whether it's been modified
in the Playlist Viewer, not just in case of the
currently playing list, but for other playlists
as well.

When you start playback of a track from the
Playlist Viewer, a playlist's 'modified'
status is now carried over to the current
playlist, so as to produce a warning when
there is an attempt to replace the list at
a later point. This also prevents (auto)
bookmarking of the playlist if it had been
modified in the Playlist Viewer prior to
becoming the current playlist. (Bugfix)

Change-Id: Ibc391fd69285f8a67d6ffb6d8c274df3d223974c
This commit is contained in:
Christian Soffke 2023-10-13 17:49:28 +02:00
parent 9cd4943950
commit 759aaecdff
2 changed files with 12 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

24
apps/playlist_viewer.c
View file

@ -121,8 +121,6 @@ static struct playlist_viewer viewer;
static struct playlist_info temp_playlist;
static bool temp_playlist_init = false;
static bool dirty = false;
static void playlist_buffer_init(struct playlist_buffer *pb, char *names_buffer,
int names_buffer_size);
static void playlist_buffer_load_entries(struct playlist_buffer * pb, int index,
@ -661,8 +659,14 @@ static enum pv_onplay_result onplay_menu(int index)
break;
case 6:
/* save playlist */
if (!save_playlist_screen(viewer.playlist))
dirty = false;
if (!save_playlist_screen(viewer.playlist) && viewer.playlist)
/*
* Set unmodified unless it's the current playlist, which may
* contain queued songs that aren't saved to the playlist file
* TODO: This can be removed once g5192 is merged,
* "playlist: Rewrite playlist_save(), optimization & fixes"
*/
playlist_set_modified(viewer.playlist, false);
ret = PV_ONPLAY_UNCHANGED;
break;
case 7:
@ -802,8 +806,7 @@ static bool update_viewer_with_changes(struct gui_synclist *playlist_lists, enum
if (res == PV_ONPLAY_CHANGED ||
res == PV_ONPLAY_ITEM_REMOVED)
{
if (!viewer.playlist)
playlist_set_modified(NULL, true);
playlist_set_modified(viewer.playlist, true);
if (res == PV_ONPLAY_ITEM_REMOVED)
gui_synclist_del_item(playlist_lists);
@ -815,8 +818,6 @@ static bool update_viewer_with_changes(struct gui_synclist *playlist_lists, enum
if (viewer.selected_track >= viewer.num_tracks)
viewer.selected_track = viewer.num_tracks-1;
dirty = true;
}
/* the show_icons option in the playlist viewer settings
@ -956,13 +957,11 @@ enum playlist_viewer_result playlist_viewer_ex(const char* filename,
str(LANG_FAILED));
}
if (!viewer.playlist)
playlist_set_modified(NULL, true);
playlist_set_modified(viewer.playlist, true);
update_playlist(true);
viewer.moving_track = -1;
viewer.moving_playlist_index = -1;
dirty = true;
}
else if (!viewer.playlist)
{
@ -1128,11 +1127,10 @@ static void close_playlist_viewer(void)
if (viewer.initial_selection)
*(viewer.initial_selection) = viewer.selected_track;
if(dirty && yesno_pop(ID2P(LANG_SAVE_CHANGES)))
if(playlist_modified(viewer.playlist) && yesno_pop(ID2P(LANG_SAVE_CHANGES)))
save_playlist_screen(viewer.playlist);
playlist_close(viewer.playlist);
}
dirty = false;
}
static const char* playlist_search_callback_name(int selected_item, void * data,