[FFmpeg-devel] [PATCH] BFI demuxer

[Michael Niedermayer]

On Sun, Apr 13, 2008 at 02:25:40AM +0530, Sisir Koppaka wrote:
> On Sun, Apr 13, 2008 at 2:15 AM, Michael Niedermayer <michaelni at gmx.at>
> wrote:
> 
> > On Sun, Apr 13, 2008 at 01:34:32AM +0530, Sisir Koppaka wrote:
> > > Updated patch attached. I removed the close function definition as well,
> > > since if it's not declared in the structure, then I assume no other code
> > is
> > > going to find the definition.
> >
> > [...]
> > > +static int bfi_read_packet(AVFormatContext * s, AVPacket * pkt)
> > > +{
> > > +    BFIContext *bfi = s->priv_data;
> > > +    ByteIOContext *pb = s->pb;
> > > +    int ret, audio_offset, video_offset, chunk_size, audio_size = 0;
> > > +    if (bfi->nframes == 0 || url_feof(pb)) {
> > > +        return AVERROR(EIO);
> > > +    }
> > > +
> > > +    /* If all previous chunks were completely read, then find a new
> > one... */
> > > +    if (!bfi->avflag) {
> > > +       uint32_t state = 0;
> > > +       while(state != MKTAG('S','A','V','I')){
> > > +           if (url_feof(pb))
> > > +               return AVERROR(EIO);
> > > +           state = 256*state + get_byte(pb);
> > > +        }
> > > +        /* Now that the chunk's location is confirmed, we proceed... */
> > > +        chunk_size      = get_le32(pb);
> >
> > inconsistant indention
> >
> > Fixed.

looks ok

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20080412/df69acb2/attachment.pgp>