- Status Closed
- Percent Complete
- Task Type Patches
- Category
- Assigned To No-one
- Operating System
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Opened by rdjackso - 2005-07-05
Last edited by linusnielsen - 2005-07-05
FS#2561 - Seek support for Ogg Vorbis
This patch adds support to Rockbox for the following:
- Seeking within vorbis files
- Vorbis comments
- Mono encodings and encodings at sampling rates other
than 44100Hz.
- Implementation of memchr() (needed for seek support)
Currently the comment support isn’t as clean as I’d
like it. The comments must all exist in the first Ogg
page (which is true 99% of the time). Patch number
1228478 implements better metadata support. Perhaps
this patch could be used with that one?
Also, currently only Ogg files with a single logical
vorbis bitstream are supported. Again, the vast
majority of encodings would fit this category, so this
may not be a problem. If we extended the mp3entry
structure to include the relevant vorbis metadata
(bitstream offsets, vorbis info structs, etc) then we
could easily have support for multiple bitstreams.
2005-07-05 08:44
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
Logged In: YES
user_id=259137
Thanks a bunch, Ryan!
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
the memchr.c file goes in firmware/common (but this should
be obvious)
Thanks a bunch, Ryan!