- Status Closed
- Percent Complete
- Task Type Bugs
- Category Playlists
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by Steve - 2008-02-27
Last edited by Jonathan Gordon - 2011-07-21
Opened by Steve - 2008-02-27
Last edited by Jonathan Gordon - 2011-07-21
FS#8656 - Error saving non-current playlist file
I was trying to remove a track from a non-current (not playing) playlist using the playlist viewer. I can remove the track but when I return to the playlist, the change is not persisted. While in the viewer I try forcing a save to the same file name as was loaded and I get the following error message: “error accessing playlist file”.
Because of this problem, the only way I remove tracks from a non-current playlist is through “Open with” text_editor. And that is pretty inconvenient.
Closed by Jonathan Gordon
2011-07-21 06:41
Reason for closing: Fixed
Additional comments about closing:
2011-07-21 06:41
Reason for closing: Fixed
Additional comments about closing:
in r30184
Confirmed with Sansa e200 & r16929.
This is still an open and confirmed bug. The problem is we dont necessarily load the whole playlist into ram so we cant easily write over the same file.
playlist.c line 3388 is the culprit (playlist_save()).
Possible fix is to try to grab the plugin buffer here which should be enough for most playlists.
Why not save a temp file with the new playlist, delete the original and then rename the temp file the same name as the original?
or that :)