Opened 12 years ago

Closed 11 years ago

#5275 closed defect (fixed)

dvd problems with the matrix dvds.

Reported by: skamithi Owned by: skamithi
Priority: minor Milestone: 0.21.1
Component: mythtv Version: 0.21-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

recently purchased the matrix set of dvds. Found some problems. this ticket to document problems resolved using this dvd set.

1st problem is a crash because audioOutput is called in the video playing thread and it may be null when audio is disabled in the decoding thread. moving audioOutput call out of the video thread.

Change History (12)

comment:1 Changed 12 years ago by skamithi

Owner: changed from Isaac Richards to skamithi
Status: newassigned

comment:2 Changed 12 years ago by skamithi

(In [17178]) Refs #5275. internal dvd player. move audioOutput call out of the video playing thread. prevents a recently seen crash when changing from dvd title to the dvd root menu.

comment:3 Changed 12 years ago by skamithi

(In [17199]) Refs #5275. internal dvd player: undo [17178]. it was causing problems on other dvd. objective is to prevent audio from been paused in the dvd menu without calling audioOutput->Pause(false) in the video thread. this seems to do the trick.

comment:4 Changed 12 years ago by skamithi

(In [17261]) Refs #5275. apply audioOutput->drain() only to still frames. fixes dvd menu lockup in the matrix dvd and also in the sahara(2005) dvd.

comment:5 Changed 12 years ago by skamithi

(In [17265]) Refs #5275. replace dvdPriv::IsInMenu? with a more accurate definition. matrix bonus disc uses title zero for the main feature.

comment:6 Changed 12 years ago by skamithi

(In [17266]) Refs #5275. internal dvd player. remove some unnecessary code when navigating the dvd menu.

comment:7 Changed 12 years ago by skamithi

(In [17267]) Refs #5275. internal dvd player: fix the cell change detection code and total title length detection code. fixes problem where the wrong title length is reported and can cause ff/rew issues.

comment:8 Changed 12 years ago by skamithi

(In [17270]) Refs #5275. internal dvd player. allow buttons to work in the "super burly brawl" section of matrix revolutions bonus disc. ffw/rw in this section will cause a crash in this section.looking into it. let me know if this change causes major differences navigating some dvd menus. i know some dvds have "hidden" buttons in some dvd menu sections. this patch may prevent you from jumping through those sections.

comment:9 Changed 12 years ago by skamithi

(In [17271]) Refs #5275. forgot one small addition to [17270]

comment:10 Changed 12 years ago by skamithi

(In [17285]) Refs #5275. internal dvd player. make dvd menu button reappear after a video flushing event (NVP::clearafterseek).

comment:11 Changed 12 years ago by skamithi

(In [17305]) Refs #5275. internal dvd player fixes: commit [17178], [17199], [17261], [17265], [17266], [17267], [17270], [17271], [17285] from trunk to fixes.

comment:12 Changed 11 years ago by skamithi

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.