- Status Closed
- Percent Complete
- Task Type Patches
- Category Drivers
- 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
FS#12079 - Support for new commands in iap.c
This is my first consequent patch to rockbox, so I may have done a lot of mistakes, please be indulgent.
This patch implements several new commands in iap.c to support some stuff related to playlists. Basically, this allows me to access quickly my playlists from my iPod dock.
Closed by bertrik
2011-04-27 06:25
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2011-04-27 06:25
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Committed as SVN r29788, thanks!
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
I didn’t join the patch…
I cleaned up the patch a bit:
* removed tabs, replaced them by spaces (please read docs/CONTRIBUTING)
* renamed current_item to cur_dbrecord (I think this is more descriptive)
* commented commands according to specification
* fixed calculation of item_offset (bytes not shifted properly)
* fixed conflict with lingo0/cmd6
I’d like to have this reviewed by another rockbox developer with regards to the playlist logic.
Wow, you have been very fast. You motivated me to uget_playlist_namepdate my patch again.
- I changed get_playlist_name() to avoid buffer overflow,
- added a check to verify that the files returned are actually playlists
- and prevented the remote device to print files extensions
Again, thanks for your interest…
Hmmm; Again! I didn’t join the patch! It should be possible to edit previous posts!