[MPlayer-dev-eng] [PATCH] PVA demuxer

[Matteo Giani]

Arpi wrote:

>Hi,
>
>  
>
>> PVA_FashionTV_BROKEN.pva
>>sound is strange, sound like 2 mpeg audio track interleaved
>>probably the file contains more than one audio and as your demuxer doesn't
>>do stream id checking, it reads both stream and mix them...
>>    
>>
>
>ok i've found taht you just skip the audio PES header, instead of parsing
>and checking stream id out of it.
>  
>
I skip the PES header because it is written in the PVA format spec at 
the URL I've included in the source that PVA files only support one 
audio stream.
Maybe multiple stream support was included in a later revision of the 
PVA specification, this I don't know as it was the only doc I had access to.
Anyway, I checked the broken files with a test app I had written, and 
the PES headers that are positioned according to the specifications (at 
the beginning of packets with the PTS flag set) are all from a 0xC0 stream.

>  
>
>>it looks unstable, i mean i wouldn't put it before the mpeg ps/pes/es
>>detection but maybe pva files will be misdetected then.
>>    
>>
>i've changed the pva_check_file, to read and check only the first 5 byte
>
>it's commited to cvs.
>
It's ok. I wrote it that way because I remember I read somewhere (don't 
remember where) that some DVB boards create a device node from which a 
PVA stream can be taken. So, I thought that if playing from there we are 
not sure that the stream will begin with a header...
Anyway, I think it's ok by now until someone confirms that I remember 
correctly... (and I cannot test it at the moment, I should set up a 
decent Linux box on my dad's computer which has a DVB board).

>
>
>A'rpi / Astral & ESP-team
>
>--
>Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
>_______________________________________________
>MPlayer-dev-eng mailing list
>MPlayer-dev-eng at mplayerhq.hu
>http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>  
>