[FFmpeg-devel] [PATCH] IFF Amiga Continuous Bitmap (ACBM)decoder

Michael Niedermayer michaelni at gmx.at
Mon Nov 21 23:10:46 CET 2011


On Mon, Nov 21, 2011 at 08:31:47PM +0100, ami_stuff wrote:
> Hi,
> 
> > On Mon, Nov 21, 2011 at 06:46:22PM +0100, ami_stuff wrote:
> > > @@ -166,6 +168,11 @@ static int iff_read_header(AVFormatContext *s,
> > >              }
> > >              break;
> > >  
> > > +        case ID_ABIT:
> > > +            iff->body_pos = avio_tell(pb);
> > > +            iff->body_size = data_size;
> > > +            break;
> > > +
> > >          case ID_BODY:
> > >              iff->body_pos = avio_tell(pb);
> > >              iff->body_size = data_size;
> > 
> > This is exactly the same code, please don't duplicate it.
> 
> Fixed.
> 
> Regards

>  Changelog         |    1 +
>  libavcodec/iff.c  |   13 ++++++++++++-
>  libavformat/iff.c |    5 ++++-
>  3 files changed, 17 insertions(+), 2 deletions(-)
> dc2686fa0e51d63c431b7308e27c943b03ec0f64  __UTF-8_Q_acbm2.diff__

applied, thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111121/e32c0760/attachment.asc>


More information about the ffmpeg-devel mailing list