[FFmpeg-devel] [PATCH] Make RM demuxer behave better with -an option

Kostya kostya.shishkov
Sun Mar 8 21:42:19 CET 2009


On Sun, Mar 08, 2009 at 07:22:56PM +0100, Michael Niedermayer wrote:
> On Sun, Mar 08, 2009 at 10:44:27AM -0400, Ronald S. Bultje wrote:
> > Hi,
> > 
> > On Mon, Mar 2, 2009 at 8:35 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> > > On Mon, Mar 2, 2009 at 2:02 AM, Kostya <kostya.shishkov at gmail.com> wrote:
> > >> The only obvious simplification I see for now is merging old and new format
> > >> demuxing - the only difference is that old format does not contain packet
> > >> headers and audio only (exactly like .wav and .avi).
> > >
> > > that's funny, I just noticed that yesterday and was planning some
> > > testing today to see if it's really that simple as replacing that
> > > whole block by a single ff_rm_parse_packet(s, s->priv_data,
> > > s->streams[0], s->streams[0]->priv_data, pkt);. That be the most
> > > beautiful simplification of all.
> > 
> > Attached patch does it, tested with "old_format" .ra files and works
> > same as before. It's a few more lines of code than I would've liked,
> > but still a good simplification and removal of duplicate code.
> 
> do we have a rm demuxer maintainer volunteer?

me, for instance
 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB




More information about the ffmpeg-devel mailing list