<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 27, 2013 at 9:29 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Le jeudi 27 juin 2013 18:46:16, Vittorio Giovara a écrit :<br>
</div><div class="im">> diff --git a/modules/codec/avcodec/video.c b/modules/codec/avcodec/video.c<br>
> index 5338144..d9e5057 100644<br>
> --- a/modules/codec/avcodec/video.c<br>
> +++ b/modules/codec/avcodec/video.c<br>
> @@ -125,8 +125,14 @@ static inline picture_t *ffmpeg_NewPictBuf( decoder_t<br>
> *p_dec, {<br>
>      decoder_sys_t *p_sys = p_dec->p_sys;<br>
><br>
> -    p_dec->fmt_out.video.i_width = p_context->width;<br>
> -    p_dec->fmt_out.video.i_height = p_context->height;<br>
> +    if( p_context->coded_width != p_context->width ||<br>
> +        p_context->coded_height != p_context->height )<br>
<br>
</div>if() should not be required here.<br></blockquote></div><br></div><div class="gmail_extra">Without the if(), in Tools -> Codec Information you'll get always "Resolution" and "Display resolution" always set, even when they match.<br>
I thought of it for consistency with other modules, but I don't mind removing it if you think it's fine to have the double entry.<br><br>Vittorio<br></div></div>