[FFmpeg-cvslog] r22112 - trunk/libavcodec/h264.c

[Måns Rullgård]

Andreas ?man <andreas at lonelycoder.com> writes:

> cehoyos wrote:
>> Author: cehoyos
>> Date: Sun Feb 28 19:33:33 2010
>> New Revision: 22112
>> Log:
>> Process picture aspect ratio changes in H.264.
>> This fixes playback of such streams with ffplay (but does not affect
>> current ffmpeg).
>
> This breaks when the workaround for a denominator == 0 kicks in (about
> 15 lines down). I see it with an x264 encoded file (dunno what version
> of x264 though)
>
> Patch attached
> Index: libavcodec/h264.c
> ===================================================================
> --- libavcodec/h264.c	(revision 22118)
> +++ libavcodec/h264.c	(working copy)
> @@ -1777,7 +1777,7 @@
>      if (s->context_initialized
>          && (   s->width != s->avctx->width || s->height != s->avctx->height
>                  || h->sps.sar.num != s->avctx->sample_aspect_ratio.num
> -                || h->sps.sar.den != s->avctx->sample_aspect_ratio.den)) {
> +                || (h->sps.sar.den?:1) != s->avctx->sample_aspect_ratio.den)) {

Not C.

-- 
M?ns Rullg?rd
mans at mansr.com