[Ffmpeg-cvslog] CVS: ffmpeg/libavformat matroska.c,1.21,1.22

Måns Rullgård mru
Wed Mar 22 20:11:09 CET 2006


michael at mplayerhq.hu (Michael Niedermayer CVS) writes:

> +                av_reduce(&st->codec->sample_aspect_ratio.num,
> +                          &st->codec->sample_aspect_ratio.den,
> +                          st->codec->height * videotrack->display_width,
> +                          st->codec-> width * videotrack->display_height,
> +                          255);

Why do you set the limit at 255?  I don't have any videos that require
a higher limit, but I see no reason to restrict it either.

-- 
M?ns Rullg?rd
mru at inprovide.com





More information about the ffmpeg-cvslog mailing list