Opened 12 years ago
Closed 12 years ago
#3937 closed enhancement (fixed)
Mythmusic time display and pause seeking enhancements
| Reported by: | otto at kolsi dot fi | Owned by: | paulh |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.21 |
| Component: | mythmusic | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
MythMusic player calculates displayed playback time in three places, two of these are wrong. This can be seen e.g. if you simply press STOP while playing -> song time display is incorrect.
First attached patch (mythmusic-time-fix.patch) fixes time display issue and cleans up the code a bit.
Currently while playback is paused, you can only seek one step back or forward, not more. Also the time display and progress bar are not updated. Attached second patch (mythmusic-pause-seek.patch) fixes these and allows full seeking and updates time and progress bar.
With these patches, LCD display is NOT updated while seeking when paused. I also noticed that when you resume playback after pause&seek, there's maybe 0.5s "previous" audio in some buffer that is played before playback is continued from the correct place. It would be nice if someone knows how to fix this so that playback is always resumed cleanly without this "distortion".
Second patch is produced with "diff -u" since I don't know how to create incremential SVN diffs. If someone knows, please advice.
Attachments (2)
Change History (5)
Changed 12 years ago by
| Attachment: | mythmusic-time-fix.patch added |
|---|
Changed 12 years ago by
| Attachment: | mythmusic-pause-seek.patch added |
|---|
comment:1 Changed 12 years ago by
| Milestone: | unknown → 0.21 |
|---|---|
| Owner: | changed from Isaac Richards to paulh |
(In [14715]) Fix MythMusic's time display and clean up the code a bit. Patch by Otto. Refs #3937.