Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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 TheBashar - 2008-02-27
Last edited by jdgordon - 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  jdgordon
2011-07-21 06:41
Reason for closing:  Fixed
Additional comments about closing:  

in r30184

Marc_Guay commented on 2008-04-03 16:00

Confirmed with Sansa e200 & r16929.

jdgordon commented on 2011-07-20 14:58

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.

MikeS commented on 2011-07-21 00:33

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?

jdgordon commented on 2011-07-21 00:35

or that :)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing