[vlc-devel] [PATCH] fix video playback on contextes with width and height differing from coded_width and coded_height

[Vittorio Giovara]

On Thu, Jun 27, 2013 at 9:29 PM, Rémi Denis-Courmont <remi at remlab.net>wrote:

> Le jeudi 27 juin 2013 18:46:16, Vittorio Giovara a écrit :
> > diff --git a/modules/codec/avcodec/video.c
> b/modules/codec/avcodec/video.c
> > index 5338144..d9e5057 100644
> > --- a/modules/codec/avcodec/video.c
> > +++ b/modules/codec/avcodec/video.c
> > @@ -125,8 +125,14 @@ static inline picture_t *ffmpeg_NewPictBuf(
> decoder_t
> > *p_dec, {
> >      decoder_sys_t *p_sys = p_dec->p_sys;
> >
> > -    p_dec->fmt_out.video.i_width = p_context->width;
> > -    p_dec->fmt_out.video.i_height = p_context->height;
> > +    if( p_context->coded_width != p_context->width ||
> > +        p_context->coded_height != p_context->height )
>
> if() should not be required here.
>

Without the if(), in Tools -> Codec Information you'll get always
"Resolution" and "Display resolution" always set, even when they match.
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.

Vittorio
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20130627/a235831f/attachment.html>