Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Applications
  • 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 emeraldd - 2007-10-20
Last edited by bertrik - 2011-04-17

FS#7998 - Meaningful Error messages when Bookmark Creation Fails

I recently installed Rockbox on my Sansa C250 and attempted to use the bookmarking feature. The current behavior is to do nothing if the creation of a bookmark fails, don’t tell the user anything and ignore the input. Because of this lack of meaningful error messages I was beginning to think that bookmark did not function at all. After adding a few error messages to the create bookmark routine, I was able to determine what I had wrong and that there was not bug.

So, to make the next newbie’s life a little easier, this patch will add some reasonable error messages to apps/bookmark.c. It is by no means a comprehensive list of possible errors ( there may be a latter version of this ) but it should cover several of the basic conditions that could pop. Hopefully, the messages are descriptive enough to allow the user to determine that they have something wrong and there is nothing wrong with Rockbox.

The patch was generated against revision 15216 of the source. It should only make modifications to apps/bookmark.c and apps/lang/english.lang.

Closed by  bertrik
2011-04-17 10:31
Reason for closing:  Wont Fix
Additional comments about closing:  

I don't see the point of having super-detailed error messages for this. The patch no longer applies anyway.

I agree on the general idea, but I think the messages you propose result in even more confusion.
“Bookmark Real Create Failed!” or “Bad Info Playing!” is still not very clear to me.

Good point, I’ll clean those up a bit and resubmit.

Ok, try this on for size. The messages should be more helpful, if a little verbose.

Whoops, forgot to mention that patch is against revision 15258

While adding a few failure messages would be useful, I think this is a bit too detailed. A few of the messages are for what is essentially defensive programming, i.e., a protection against bugs. As I see it, two messages are enough. One stating that bookmarking failed because the playlist is modified (which is the main reason bookmarking fails), and one generic error message for other failures (possibly with a number code to help debugging, but only if can be easily done).

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing