On 5/13/11 11:00 PM, Ronald S. Bultje wrote:
Hi,
On Fri, May 13, 2011 at 3:59 AM, Vittorio Giovara
<[email protected]> wrote:
weekly ping :)
as I explained in the first email this patch is not doing the right thing as
the "right thing"(tm) would be to actually crop the resulting frame to the
dimensions specified in the cropping rectangle (tested on the JM); however
this patch takes a saner approach for the current handling in which the
"unusual" cropping values are always ignored and don't modify the final
resolution.
So this breaks fate-h264-interlace-crop, and not for the better.
Before the patch, the values are properly used and the bottom of the
image is cropped off. After the patch, the bottom looks like crap. I
don't think that's right.
Vittorio you should keep this patch on your local tree and either prod
somebody to complete the crop support or extend the patch a little to
make the test pass again.
lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
