[FFmpeg-devel] [PATCH] Bitmap Brothers JV demuxer

[Michael Niedermayer]

On Thu, Mar 10, 2011 at 01:48:38PM +0100, Kostya wrote:
> On Thu, Mar 10, 2011 at 11:29:22PM +1100, Peter Ross wrote:
> > ---
> > On Wed, Mar 09, 2011 at 12:12:27PM +0100, Kostya wrote:
> > > On Wed, Mar 09, 2011 at 09:45:41PM +1100, Peter Ross wrote:
> > [..]
> > > > > BTW, do you initialise state explicitly somewhere?
> > > >
> > > > JvContext.state defaults to zero when priv_data is allocated. In this updated
> > > > patch, i've forced the JV_AUDIO state enum to be zero. That way theres no
> > > > need to specifically initialise it in decode_init().
> > >
> > > I guessed that but for AVCodecs I think it's better to put that explicitly -
> > > makes people wonder less from what state it starts.
> > 
> > Fixed.
> > 
> > > > +        case JV_PADDING:
> > > > +            avio_skip(pb, FFMAX(e->size - jvf->audio_size - jvf->video_size - (jvf->palette ? 768 : 0), 0));
> > >
> > > ...but this one probably should be
> > 
> > And fixed.
>  
> Yep, now it looks fine to me.

applied

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110311/66fa55b8/attachment.pgp>