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

Rafaël Carré rafael.carre
Sun Jul 18 19:36:56 CEST 2010


On Sun, 18 Jul 2010 19:13:26 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> On Sun, Jul 18, 2010 at 05:37:13PM +0200, Rafa?l Carr? wrote:
> [...]
> > +static int rso_read_header(AVFormatContext *s, AVFormatParameters

> > +    st->duration            = (size * 8) >> av_log2(bps);
> 
> why not 
> st->duration            = (size * 8) / bps ?

I thought I could spare an expensive division since bps is always 4 or 8

But I'll try to make some demuxers use raw_read_packet() from raw.c and
then it won't be the case anymore so let's just drop that.

Attached patch use / bps

-- 
? 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: 14035 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100718/4c66ae1f/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/20100718/4c66ae1f/attachment.pgp>



More information about the ffmpeg-devel mailing list