Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8656 - Error saving non-current playlist file

Attached to Project: Rockbox
Opened by Steve (TheBashar) - Wednesday, 27 February 2008, 19:22 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Thursday, 21 July 2011, 08:41 GMT+2
Task Type Bugs
Category Playlists
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

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.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Thursday, 21 July 2011, 08:41 GMT+2
Reason for closing:  Fixed
Additional comments about closing:  in r30184
Comment by Marc Guay (Marc_Guay) - Thursday, 03 April 2008, 18:00 GMT+2
Confirmed with Sansa e200 & r16929.
Comment by Jonathan Gordon (jdgordon) - Wednesday, 20 July 2011, 16:58 GMT+2
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.
Comment by Michael Sevakis (MikeS) - Thursday, 21 July 2011, 02:33 GMT+2
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?
Comment by Jonathan Gordon (jdgordon) - Thursday, 21 July 2011, 02:35 GMT+2
or that :)

Loading...