Opened 12 years ago
Closed 12 years ago
Last modified 12 years ago
#11455 closed Bug Report - General (fixed)
DVD subpictures without an explicit 'end' time are not shown.
Reported by: | Owned by: | peper03 | |
---|---|---|---|
Priority: | minor | Milestone: | 0.27 |
Component: | MythTV - DVD Playback | Version: | Master Head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
The 'end' time for DVD subpictures defaults to zero, which prevents them being shown if they don't include an explicit command to set it.
The attached patch initializes the 'end' time so that the subpicture shows forever (or until the next subpicture). This allows DVDs such as those found here to play (note subtitles must be turned on to see anything!)
Additionally, the also adds minimal handling for the 'CHG_COLCON' command, which doesn't occur often, but is currently not recognised. This could lead to issues due to the code attempting to interpret the data for the command as new commands. The new handling simply skips over the data to the next command.
Attachments (3)
Change History (10)
Changed 12 years ago by
Attachment: | 0001-Ensure-that-DVD-subpictures-that-don-t-explicitly-se.patch added |
---|
Changed 12 years ago by
Attachment: | 0001a-Ensure-that-DVD-subpictures-that-don-t-explicitly-se.patch added |
---|
comment:1 Changed 12 years ago by
The updated patch initializes end_display_time to the same value as start_display_time.
Also, the number of subtitles that can be buffered is increased to 40 to (hopefully) allow as many subtitles to be buffered as video frames (OpenGL on my machine buffers 29 frames, so for full frame-rate subtitles/subpictures, it must be possible to buffer the same number of subtitles).
Note that the patch from #11451 must be applied before either of these patches.
comment:2 Changed 12 years ago by
Owner: | set to Jim Stichnoth |
---|---|
Status: | new → accepted |
Changed 12 years ago by
Attachment: | 0001b-Ensure-that-DVD-subpictures-that-don-t-explicitly-se.patch added |
---|
comment:3 Changed 12 years ago by
New patch (0001b) adds a check to ensure that an end time has actually been set before clearing a menu highlight. Otherwise the highlight would only appear for one frame on menus where no end time was specified.
comment:4 Changed 12 years ago by
Owner: | changed from Jim Stichnoth to peper03 |
---|---|
Status: | accepted → assigned |
comment:5 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:6 Changed 12 years ago by
Milestone: | unknown → 0.27 |
---|
Updated patch