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

[Michael Niedermayer]

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?

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20090308/0be957f3/attachment.pgp>