[vlc-devel] [PATCH 05/10] video_filter:deinterlace: don't use the field parameter in RenderDiscard()

[Rémi Denis-Courmont]

Le 26 juin 2017 19:20:14 GMT+02:00, Steve Lhomme <robux4 at videolabs.io> a écrit :
>it's always 0
>---
> modules/video_filter/deinterlace/algo_basic.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/modules/video_filter/deinterlace/algo_basic.c
>b/modules/video_filter/deinterlace/algo_basic.c
>index 858ae93da8..e7f32baac2 100644
>--- a/modules/video_filter/deinterlace/algo_basic.c
>+++ b/modules/video_filter/deinterlace/algo_basic.c
>@@ -44,6 +44,7 @@
> 
>void RenderDiscard( picture_t *p_outpic, picture_t *p_pic, int i_field
>)
> {
>+    VLC_UNUSED(i_field);
>     int i_plane;
> 
>     /* Copy image and skip lines */
>@@ -51,8 +52,7 @@ void RenderDiscard( picture_t *p_outpic, picture_t
>*p_pic, int i_field )
>     {
>         uint8_t *p_in, *p_out_end, *p_out;
> 
>-        p_in = p_pic->p[i_plane].p_pixels
>-                   + i_field * p_pic->p[i_plane].i_pitch;
>+        p_in = p_pic->p[i_plane].p_pixels;
> 
>         p_out = p_outpic->p[i_plane].p_pixels;
>         p_out_end = p_out + p_outpic->p[i_plane].i_pitch
>-- 
>2.12.1
>
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

Use assert then.
-- 
Rémi Denis-Courmont
Typed on an inconvenient virtual keyboard
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170626/053fb5b5/attachment.html>