[vlc-devel] [PATCH 04/10] video_filter:deinterlace: use a union to store phosphor and ivtc parameters

[Rémi Denis-Courmont]

Le 26 juin 2017 19:20:13 GMT+02:00, Steve Lhomme <robux4 at videolabs.io> a écrit :
>since they are mutually exclusive
>---
> modules/video_filter/deinterlace/deinterlace.h | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/modules/video_filter/deinterlace/deinterlace.h
>b/modules/video_filter/deinterlace/deinterlace.h
>index 93ae1d255b..00fddb74ec 100644
>--- a/modules/video_filter/deinterlace/deinterlace.h
>+++ b/modules/video_filter/deinterlace/deinterlace.h
>@@ -119,8 +119,10 @@ struct filter_sys_t
>     picture_t *pp_history[HISTORY_SIZE];
> 
>     /* Algorithm-specific substructures */
>-    phosphor_sys_t phosphor; /**< Phosphor algorithm state. */
>-    ivtc_sys_t ivtc;         /**< IVTC algorithm state. */
>+    union {
>+        phosphor_sys_t phosphor; /**< Phosphor algorithm state. */
>+        ivtc_sys_t ivtc;         /**< IVTC algorithm state. */
>+    };
> };
> 
>/*****************************************************************************
>-- 
>2.12.1
>
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

Looks OK, bjt I haven't checked if initialization is still fine.
-- 
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/a7664e60/attachment.html>