[vlc-devel] [PATCH 1/2] include: use vlc_rational_t for display zoom

Alexandre Janniaux ajanni at videolabs.io
Tue Feb 9 16:20:55 UTC 2021


Hi,

Patchset LGTM.

Regards,
--
Alexandre Janniaux
Videolabs

On Tue, Feb 09, 2021 at 06:13:48PM +0200, remi at remlab.net wrote:
> From: RĂ©mi Denis-Courmont <remi at remlab.net>
>
> ---
>  include/vlc_vout_display.h | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/include/vlc_vout_display.h b/include/vlc_vout_display.h
> index 58a0c2e430..36018a9542 100644
> --- a/include/vlc_vout_display.h
> +++ b/include/vlc_vout_display.h
> @@ -119,10 +119,7 @@ typedef struct vout_display_cfg {
>      bool is_display_filled;
>
>      /** Zoom ratio */
> -    struct {
> -        unsigned num;
> -        unsigned den;
> -    } zoom;
> +    vlc_rational_t zoom;
>
>      vlc_viewpoint_t viewpoint;
>  } vout_display_cfg_t;
> --
> 2.30.0
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list