[vlc-devel] [PATCH 1/4] vout: move aspect ratio members from vout_display_state_t to vout_display_cfg_t

[Rémi Denis-Courmont]

On Mon, 10 Feb 2014 19:14:50 +0100, Felix Abecassis
<felix.abecassis at gmail.com> wrote:
> ---
>  include/vlc_vout_display.h         |  5 +++++
>  include/vlc_vout_wrapper.h         |  4 ----
>  modules/visualization/glspectrum.c |  4 ++--
>  src/video_output/display.c         | 16 ++++++++++++----
>  src/video_output/video_output.c    | 19 +++++++++++++++----
>  5 files changed, 34 insertions(+), 14 deletions(-)
> 
> diff --git a/include/vlc_vout_display.h b/include/vlc_vout_display.h
> index d99cf7e..870e207 100644
> --- a/include/vlc_vout_display.h
> +++ b/include/vlc_vout_display.h
> @@ -110,6 +110,11 @@ typedef struct {
>          int den;
>      } zoom;
>  
> +    /* Aspect ratio */
> +    struct {
> +        int num;
> +        int den;
> +    } sar;
>  } vout_display_cfg_t;

Err, how is that different from display.sar already in the same structure?

And if it is different, you presumably need to patch every vout display
plugin to take the value into account :-(

-- 
Rémi Denis-Courmont
Sent from my collocated server