Opened 16 years ago
Closed 16 years ago
#6749 closed defect (fixed)
Invalid audio streams caused by changset 20971
Reported by: | Owned by: | Janne Grunau | |
---|---|---|---|
Priority: | minor | Milestone: | 0.22 |
Component: | MythTV - Video Playback | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Changeset 20971 causes invalid audio streams to appear along side valid ones. Reverting 20971 fixes the problem.
The problem seems to be that the decision if a stream is an audio stream or not, is made upon the existence of a language description which leads to some streams wrongfully labeled as audio. The following patch adds a proper check if the stream is indeed an audio stream or not. This fixes the problem and so far hasn't caused any new ones.
Hope that helps...
Attachments (2)
Change History (10)
Changed 16 years ago by
Attachment: | fix-wrong-audio-streams added |
---|
Changed 16 years ago by
Attachment: | fix-wrong-audio-streams.2 added |
---|
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
Component: | MythTV - General → MythTV - Video Playback |
---|---|
Milestone: | unknown → 0.22 |
Owner: | changed from Isaac Richards to Janne Grunau |
Status: | new → assigned |
comment:3 Changed 16 years ago by
comment:4 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:5 Changed 16 years ago by
Janne, unfortunately [20978] does not fix the issue. Things are still unchanged.
comment:6 Changed 16 years ago by
Resolution: | fixed |
---|---|
Status: | closed → new |
I need a small sample, or at least the PMT of the recording.
comment:7 Changed 16 years ago by
Janne, please close. I found a stale libmythavformat* from a just a few revisions back (remainings from when I was doing debugging) which portage didn't consider to overwrite for some reason. So, everything is detecte fine. On the bright side: After reading through mpegts.c the last hour, at least I understand it a lot more.
I apologize for any inconvenience caused...
comment:8 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Forgot to remove one comment line, otherwise identical. Please remove the first patch as I am not allowed to do it myself. Thanks.