- Status Closed
- Percent Complete
- Task Type Bugs
- Category ID3 / meta data
- 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#8082 - no/unreliable display of run-time data tags in WPS
I am afraid I have another MoB bug: The display of run-time data tags such as rating and playcount in the wps is unreliable. I have identified the following situations (gigabeat, r15436):
- select from tree/database while nothing is playing → no display
- select from tree/database while a track is playing → display
- skipping forward → sometimes display (I *think* it coincides with buffering)
- skipping backward → display
- fast forward skipping (i.e. track not yet buffered) → display
I have also noticed when forward skipping it sometimes briefly displays invalid info before it is overwritten by the correct one.
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
Isn't this a dupe of 8040?
No.
FS#8040is about the gathering of the data (the on-unbuffer callback) - this is about the display of already gathered data (the on-buffer callback).Okay, the populating of the runtime data fields in the mp3entry struct is done via callbacks from playback.c/pcm.c. AFAIK this is a result of the apps/firmware division and the fact that firmware code can't call apps code directly (which would be required for HWCODEC, as pcm.c is in firmware)
However it would be much more intuitive (and KISS) if the runtime data fields were filled like any other fields in the mp3entry struct during the call to get_metadata() thereby avoiding any inconsistencies as they are happening now.
The patch implements this mechanism and fixes so the
FS#8082bug.For symmetry it would be nice to use the same mechanism for HWCODEC (I believe HWCODEC doesn't use get_metadata()), but I'll leave that to someone with an actual device.
move position of function call to after the setting of the path to make it work for formats other than mp3.