On 05/15/2013 09:30 AM, Martin Storsjö wrote:
> From: Michael Niedermayer <[email protected]>
> 
> This replaces a large number of checks for the second field by
> fixing the pointers when they are setup.
> 
> This should also fix I/BI field pictures.
> 
> Changes checksums for vc1_sa10143, the file becomes slightly closer
> to what the reference decoder outputs.
> 
> Based on "vc1dec: the second field is written wrong to the picture"
> by Sebastian Sandberg <[email protected]>.
> 

Basically it factorizes off-by-one-line machinery that was scattered in
different points, doesn't look bad. Just I wonder why the *should* in
the comment.

Could you please replace "fix" with "update" when you push?

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to