[FFmpeg-devel] [PATCH] Electronic Arts MAD Decoder
Peter Ross
pross
Wed Jun 10 12:52:58 CEST 2009
On Tue, Jun 09, 2009 at 11:07:53PM +0200, Michael Niedermayer wrote:
> On Mon, Jun 08, 2009 at 12:06:53PM +1000, Peter Ross wrote:
> > Updated patch enclosed. Thanks to Diego and Reimar for the review.
> >
> > On Sun, Jun 07, 2009 at 11:53:38AM +0200, Reimar D?ffinger wrote:
> > > On Sun, Jun 07, 2009 at 04:24:59PM +1000, Peter Ross wrote:
> > >
> > > except that I think your code is wrong and you should be using
> > > ref_linesize for the ref_y offset.
> >
> > Well spotted.
> >
> > So I have left this verbose low-level code in with a comment. Suggestions
> > on how to better reuse the code without bloat are welcome.
>
> i wish i had an idea on how to factorize that, sadly i have no clean idea
>
Patch updated.
-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mad-decoder-r105.diff
Type: text/x-diff
Size: 13159 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090610/3d990acb/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090610/3d990acb/attachment.pgp>
More information about the ffmpeg-devel
mailing list