[FFmpeg-devel] [PATCH] Electronic Arts TGV decoder

pross at xvid.org pross
Tue Jul 29 14:28:24 CEST 2008


On Sun, Jul 20, 2008 at 10:14:19PM +1000, pross at xvid.org wrote:
> On Sun, Jul 20, 2008 at 10:04:44AM +0200, Reimar D?ffinger wrote:
> > Hello,
> > On Sun, Jul 20, 2008 at 12:28:06PM +1000, pross at xvid.org wrote:
> > > On Sat, Jul 19, 2008 at 11:30:38AM +0200, Reimar D?ffinger wrote:
> > > > The diff should be smaller if you change the order of these two
> > > > functions.
> > > 
> > > Okay, but s/smaller/simpler/.
> > 
> > Why did you put the new function below lzo1x_decode? And no, I did mean
> > smaller, the diff should basically consist of an additional { and a new
> > function header + doxy and maybe a few more lines.
> > Anything above that IMO needs an explanation why that would be needed.
> > Oh well, I just did change it myself, see attached patch.
> 
> Thanks!

Reminar, were there any other comments, or should I just go ahead and
commit?

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- 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/20080729/f1129bcd/attachment.pgp>



More information about the ffmpeg-devel mailing list