From: Chris R. <ran...@ya...> - 2011-08-28 00:21:57
Attachments:
DEMUX-TS.diff
|
Hi, Here's a better patch for removing the stream autodetection, as I've realised that I'd left the pes_stream_id variable behind. Cheers, Chris |
From: Petri H. <phi...@us...> - 2011-08-31 18:12:42
|
Chris Rankin wrote: > Hi, > > Here's a better patch for removing the stream autodetection, as I've realised > that I'd left the pes_stream_id variable behind. Looks good. Those who need pid autodetection, rise your hands... > + if (payload_unit_start_indicator && (this->media_num < MAX_PIDS) && (pid == 0)) { > + demux_ts_parse_pat(this, originalPkt, originalPkt+data_offset-4, > + payload_unit_start_indicator); > + return; > } I think we should always parse PAT, but that may require some fixes to pat/pmt parsing (like clearing this->media[] when program changes etc.). - Petri |
From: Chris R. <ran...@ya...> - 2011-08-31 22:39:17
|
On 31/08/11 13:39, Petri Hintukainen wrote: > I think we should always parse PAT, but that may require some fixes to > pat/pmt parsing (like clearing this->media[] when program changes etc.). I'll have to leave such decisions to you. But do you have any other pending changes to the MPEG-TS demuxer, please? I am currently investigating this bug: https://bugs.xine-project.org/show_bug.cgi?id=403 It's actually a little more complicated than just "xine not understanding AAC LATM" because I've patched in FFmpeg's AAC LATM support and xine still doesn't play the bug's attached TS files correctly. (It seems to be trying to initialise the audio codec plugin too late, after BUF_FLAG_HEADER has been removed from the decoder flags). So if you already have demuxer patches that might help then they'd be gratefully received ;-). Thanks, Chris |