[FFmpeg-devel] [PATCH] BFI demuxer

[Michael Niedermayer]

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


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

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- 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/ef7259e4/attachment.pgp>