Revert "picture: copy more than the visible lines"
authorSteve Lhomme <robux4@ycbcr.xyz>
Thu, 9 Aug 2018 14:19:20 +0000 (16:19 +0200)
committerSteve Lhomme <robux4@ycbcr.xyz>
Thu, 9 Aug 2018 14:19:20 +0000 (16:19 +0200)
This reverts commit 4594dedf117e40a1ba783ef55df995e12ea8e5fe.

This is the wrong way to do it. And breaks other things like magnify.
Either the visible area should account for the x/y offsets or there should be
and offset stored per plane.

We also need to decide in a picture copy copies only the visible area (and not
event the part in the x/y offset) or some of the decoder/crop padding too. The
current code leans towards the latter but still copies too much.

src/misc/picture.c

index 6fe9437..0740f06 100644 (file)
@@ -332,7 +332,8 @@ void plane_CopyPixels( plane_t *p_dst, const plane_t *p_src )
 {
     const unsigned i_width  = __MIN( p_dst->i_visible_pitch,
                                      p_src->i_visible_pitch );
-    const unsigned i_height = __MIN( p_dst->i_lines, p_src->i_lines );
+    const unsigned i_height = __MIN( p_dst->i_visible_lines,
+                                     p_src->i_visible_lines );
 
     /* The 2x visible pitch check does two things:
        1) Makes field plane_t's work correctly (see the deinterlacer module)