[FFmpeg-devel] [PATCH][RFC] TechnoTrend PVA Demuxer

Michael Niedermayer michaelni
Mon Dec 31 17:38:49 CET 2007


On Mon, Dec 31, 2007 at 01:12:31AM +0100, Ivo wrote:
> Hi,
> 
> In order to make it possible to deprecate MPlayer's libmpdemux in the 
> future, I started working on a few demuxers that lavf is still missing. 
> This is the first one.
> 
> http://wiki.multimedia.cx/index.php?title=PVA
> http://samples.mplayerhq.hu/PVA/
> 
> All samples play correctly.
> 

> I did not however implement seeking yet. That's the RFC. How should I 
> proceed? There's no index. PVA files are encapsulated MPEG-ES (for video) 
> and MPEG-PES (for audio) packets. Should I just rely on the caller 
> (MPlayer, ...) feeding it a new byte-position and resync on the AV-syncword 
> at the start of read_packet() ? And if I do so, can I use url_fseek or does 
> that ruin the ability to pipe the input to ffmpeg?

use av_add_index_entry() every time a packet is demuxed
and implement read_timestamp()
and feel free to add seeking as a seperate patch if you like


[...]

> +static int pva_read_close(AVFormatContext *s) {
> +    return 0;
> +}

unneeded ...


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071231/d8572361/attachment.pgp>



More information about the ffmpeg-devel mailing list