<div dir="ltr">2014-02-11 9:26 GMT+01:00 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span>:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div class="">On Mon, 10 Feb 2014 19:14:50 +0100, Felix Abecassis<br>
<<a href="mailto:felix.abecassis@gmail.com">felix.abecassis@gmail.com</a>> wrote:<br>
> ---<br>
>  include/vlc_vout_display.h         |  5 +++++<br>
>  include/vlc_vout_wrapper.h         |  4 ----<br>
>  modules/visualization/glspectrum.c |  4 ++--<br>
>  src/video_output/display.c         | 16 ++++++++++++----<br>
>  src/video_output/video_output.c    | 19 +++++++++++++++----<br>
>  5 files changed, 34 insertions(+), 14 deletions(-)<br>
><br>
> diff --git a/include/vlc_vout_display.h b/include/vlc_vout_display.h<br>
> index d99cf7e..870e207 100644<br>
> --- a/include/vlc_vout_display.h<br>
> +++ b/include/vlc_vout_display.h<br>
> @@ -110,6 +110,11 @@ typedef struct {<br>
>          int den;<br>
>      } zoom;<br>
><br>
> +    /* Aspect ratio */<br>
> +    struct {<br>
> +        int num;<br>
> +        int den;<br>
> +    } sar;<br>
>  } vout_display_cfg_t;<br>
<br>
</div>Err, how is that different from display.sar already in the same structure?<br></blockquote><div>This is display aspect ratio, initialized from variable "monitor-par". <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


<br>
And if it is different, you presumably need to patch every vout display<br>
plugin to take the value into account :-(<span class=""><font color="#888888"><br></font></span></blockquote><div>Why? any change to the SAR will still be forwarded to the vout module through the control event VOUT_DISPLAY_CHANGE_SOURCE_ASPECT.<br>

</div></div></div><div class="gmail_extra">What I don't understand is why this variable was in vout_display_state_t and not in the vout_display_cfg_t structure with other variables.<br clear="all"><br>-- <br>Félix Abecassis<div>

<a href="http://felix.abecassis.me" target="_blank">http://felix.abecassis.me</a></div>
</div></div>