On Sun, Feb 11, 2018 at 05:15:54PM +0900, Nekopanda wrote: > In 16x8 motion compensation, for lower 16x8 region, the input to > mpeg_motion() for motion_y was "motion_y + 16", which causes wrong rounding. > For 4:2:0, chroma scaling for y is dividing by two and rounding toward zero. > When motion_y < 0 and motion_y + 16 > 0, the rounding direction of "motion_y" > and "motion_y + 16" is different and rounding "motion_y + 16" would be > incorrect. > > We should input "motion_y" as is to round correctly. I add "is_16x8" flag to > do that. > --- > libavcodec/mpegvideo_motion.c | 30 ++++----- > tests/ref/fate/filter-mcdeint-fast | 60 +++++++++--------- > tests/ref/fate/filter-mcdeint-medium | 60 +++++++++--------- > tests/ref/fate/filter-w3fdif-complex | 60 +++++++++--------- > tests/ref/fate/filter-w3fdif-simple | 60 +++++++++--------- > tests/ref/fate/filter-yadif-mode0 | 60 +++++++++--------- > tests/ref/fate/filter-yadif-mode1 | 118 > +++++++++++++++++------------------ > tests/ref/fate/filter-yadif10 | 60 +++++++++--------- > tests/ref/fate/filter-yadif16 | 60 +++++++++--------- > tests/ref/fate/mpeg2-field-enc | 60 +++++++++--------- > tests/ref/fate/mpeg2-ticket6677 | 12 ++-- > 11 files changed, 321 insertions(+), 319 deletions(-)
will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Into a blind darkness they enter who follow after the Ignorance, they as if into a greater darkness enter who devote themselves to the Knowledge alone. -- Isha Upanishad
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
