Michael Niedermayer <[email protected]> added the comment:

On Fri, Jul 31, 2009 at 09:28:28PM +0000, Jeff Downs wrote:
> 
> New patch: mpeg2-edgeemu-1.diff
> 
> This changes the edge emulation calls for mpeg_motion_internal() so that mvs 
> in
> field pictures are adjusted back in to reference frame buffer bounds and edge
> pixels replicated as expected.  The patch makes all arguments to
> ff_emulated_edge_mc in terms of full frame units (never field units), which, 
> if
> I am understanding it properly, is what it expects.
> 
> If this is an acceptable fix, I'll prepare a second patch that fixes any other
> similar cases in the mpeg decoder (similar code appears in several other 
> places
> which may have the same issue).

does reverting r18344 fix this bug?
if so, does it cause a regression with issue961?

if the awnsers are yes+no then reverting it seems the correct solution
maybe we could replace return by setting the mv to 0,0 but thats a seperate
thing

[...]

_____________________________________________________
FFmpeg issue tracker <[email protected]>
<https://roundup.ffmpeg.org/roundup/ffmpeg/issue1221>
_____________________________________________________

Reply via email to