diff options
author | Christian Soffke <christian.soffke@gmail.com> | 2023-10-15 02:48:30 +0200 |
---|---|---|
committer | Christian Soffke <christian.soffke@gmail.com> | 2023-10-17 12:05:57 -0400 |
commit | f2af0711b01a5aa66a0533504bb88e142c67ef80 (patch) | |
tree | 415b7ca673fc9be37d0346d0f95f994fb42e4c4e | |
parent | ebb7f33a9ff1ebe4feded12697b69704f9d192c8 (diff) | |
download | rockbox-f2af0711b01a5aa66a0533504bb88e142c67ef80.tar.gz rockbox-f2af0711b01a5aa66a0533504bb88e142c67ef80.zip |
Delete existing bookmark file when saving a new playlist
Deleting a playlist leaves its bookmark file behind.
When a new playlist was saved under the same name as an
existing bookmark file, unrelated bookmarks were shown
for the new playlist
Change-Id: I7332460a5f488c354f41195c8fff4cf4d66f4bbb
-rw-r--r-- | apps/playlist_catalog.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/apps/playlist_catalog.c b/apps/playlist_catalog.c index f91f8c07bb..0cf9d2e09d 100644 --- a/apps/playlist_catalog.c +++ b/apps/playlist_catalog.c | |||
@@ -420,10 +420,10 @@ bool catalog_pick_new_playlist_name(char *pl_name, size_t buf_size, | |||
420 | apply_playlist_extension(pl_name, buf_size); | 420 | apply_playlist_extension(pl_name, buf_size); |
421 | 421 | ||
422 | /* warn before overwriting existing (different) playlist */ | 422 | /* warn before overwriting existing (different) playlist */ |
423 | if ((!curr_pl_name || strcmp(curr_pl_name, pl_name)) && | 423 | if (!curr_pl_name || strcmp(curr_pl_name, pl_name)) |
424 | file_exists(pl_name)) | ||
425 | { | 424 | { |
426 | do_save = confirm_overwrite_yesno() == YESNO_YES; | 425 | if (file_exists(pl_name)) |
426 | do_save = confirm_overwrite_yesno() == YESNO_YES; | ||
427 | 427 | ||
428 | if (do_save) /* delete bookmark file unrelated to new playlist */ | 428 | if (do_save) /* delete bookmark file unrelated to new playlist */ |
429 | { | 429 | { |