[FFmpeg-devel] [PATCH] DVCPRO HD [1/3] dynamic DV macroblock lookup

Roman Shaposhnik rvs
Thu Jan 3 20:40:55 CET 2008


On Thu, 2008-01-03 at 20:09 +0100, Michael Niedermayer wrote:
> This is duplicated, please clean and simplify your code before submitting!!!
> 
> Also note to roman, please dont apply this before it has been simplified and
> cleaned up, the author should do this before it reaches svn!

  Sure. I just took the slower approach of asking just a couple of
questions at a time ;-) Also, sometimes I clean the patches myself 
before submitting them to SVN so that they follow all the rules like
no tabs, doxygen comments, etc.

> Also please split the table removial out of the addition+changes
> so you dont need to compress the patch (the actual table removial can be
> compressed but thats likely going to be approved quickly while the rest looks
> like it will need a few iterations)

  That's actually a good suggestion. I haven't thought of it. 

Thanks,
Roman.





More information about the ffmpeg-devel mailing list