[MPlayer-dev-eng] [PATCH] initial transport stream demuxer

[Arpi]

Hi,

> Arpi,   the chaining still doesn't recognize audio, instead it sees the

send me your code, i'll check

> PES stream as an ES 
> video stream.

because it's ES. it's quite trivial to make difference between PES/PS and ES

> The format I use for writing to the video fifo (the only one I use) is
> simply a sequence of 
> PES audio and video packets, e.g.
> 	
> 		APES1 | VPES1 |  VPES2 |  VPES3 |  APES2
> and so on.

are you sure?

> Furthermore, there are a couple of things to consider:
> 
> 1) in any case I need to do at least a simple pes parsing  to determine is
> the pes is 
> audio/video (and the right one) or DVB subtitle (that obviously   I won't
> pass along to 
> demux_mpg).

you should do all that based on TS PID.
the user gives PIDs for audio, video streams and you filter out those
TS substreams and pass to the PS demuxer.

> 2) chaining  the demuxers requires more resources than a single demuxer.

why?
maybe a few kb's more memmory.

> So my proposal is to isolate the pes-parsing function in demux_mpg and
> make it non 
> static, so I can call it from demux_ts without duplicating code and
> without using chaining.

cannot be easily done. demuxers call teh stream API functions while you need
it to call your packet reader, you will end up emulation of chaining...

> What's your opinion?

send me the chaining version i'll look at it


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino