[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