[FFmpeg-devel] [PATCH] Lego Mindstorms RSO de/muxer (copied from au.c)

[Rafaël Carré]

On Wed, 14 Jul 2010 23:44:02 +0200
Diego Biurrun <diego at biurrun.de> wrote:

> On Wed, Jul 14, 2010 at 09:54:31PM +0200, Rafa?l Carr? wrote:
> > - format from http://bricxcc.sourceforge.net/
> 
> This is missing a minor version bump and a documentation update.

I added the bump and an entry in general.texi

> > --- /dev/null
> > +++ b/libavformat/rso.c
> > @@ -0,0 +1,203 @@
> > +/*
> > + * RSO muxer and demuxer
> > + * Copyright (c) 2001 Fabrice Bellard (original AU code)
> > + * Copyright ?? 2010 Rafa??l Carr??
> 
> Please use UTF-8

Changed to ASCII (which is still UTF-8) with Mans suggestion

> > +#if CONFIG_RSO_MUXER
> 
> Split muxer and demuxer into separate files and avoid the #ifdefs.

Splitted in rsoenc.c and rsodec.c

> > +static int rso_read_packet(AVFormatContext *s,
> > +                          AVPacket *pkt)
> 
> Indentation is off; just don't break this short line.

Done


Also added a probe function with Mike suggestion (details in the patch
commit log)

Also make coded_file_size unsigned


2nd patch in attachment

-- 
? Rafa?l Carr? ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Lego-Mindstorms-RSO-de-muxer-copied-from-au.c.patch
Type: text/x-patch
Size: 12747 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100715/1caaa997/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100715/1caaa997/attachment.pgp>