[vlc-devel] [PATCH 2/2] chroma: no need to carry over the offset when the core does it

Steve Lhomme robux4 at videolabs.io
Thu May 12 13:18:50 CEST 2016


---
 modules/video_chroma/i420_nv12.c | 3 ---
 modules/video_chroma/swscale.c   | 7 -------
 2 files changed, 10 deletions(-)

diff --git a/modules/video_chroma/i420_nv12.c b/modules/video_chroma/i420_nv12.c
index da16349..5183e1d 100644
--- a/modules/video_chroma/i420_nv12.c
+++ b/modules/video_chroma/i420_nv12.c
@@ -118,9 +118,6 @@ VIDEO_FILTER_WRAPPER( YV12_NV12 )
 
 static void I420_YUV( filter_sys_t *p_sys, picture_t *p_src, picture_t *p_dst, bool invertUV )
 {
-    p_dst->format.i_x_offset = p_src->format.i_x_offset;
-    p_dst->format.i_y_offset = p_src->format.i_y_offset;
-
     const size_t u_plane = invertUV ? V_PLANE : U_PLANE;
     const size_t v_plane = invertUV ? U_PLANE : V_PLANE;
 
diff --git a/modules/video_chroma/swscale.c b/modules/video_chroma/swscale.c
index 90494d2..d0e3a9b 100644
--- a/modules/video_chroma/swscale.c
+++ b/modules/video_chroma/swscale.c
@@ -652,13 +652,6 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic )
         CopyPad( p_src, p_pic );
     }
 
-    if ( p_src->format.i_visible_width  == p_dst->format.i_visible_width &&
-         p_src->format.i_visible_height == p_dst->format.i_visible_height )
-    {
-        p_dst->format.i_x_offset = p_src->format.i_x_offset;
-        p_dst->format.i_y_offset = p_src->format.i_y_offset;
-    }
-
     if( p_sys->b_copy && p_sys->b_swap_uvi == p_sys->b_swap_uvo )
         picture_CopyPixels( p_dst, p_src );
     else if( p_sys->b_copy )
-- 
2.8.1



More information about the vlc-devel mailing list