[FFmpeg-devel] [PATCH/RFC] honor DAR aspect ratio in nuv files.

elupus elupus
Sun Jun 8 20:50:05 CEST 2008


>> Anyway, here is a updated patch that behavies just like official myth
>> player (and actually works). It considers a value of 1.0, which the old
>> muxers wrote, a DAR of 4:3. 
> [...]
>> +        av_reduce(&vst->codec->sample_aspect_ratio.num,
>> +                  &vst->codec->sample_aspect_ratio.den,
>> +                  height * aspect,
>> +                  width, 255);
> 
> this is wrong
> converting height * aspect to an integer is nonsense
> 
> [...]

Right, missed that. This should be better, but i'm not sure what I should
set the max num/den in av_reduce/av_d2q.

Joakim
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-The-aspect-stored-in-nuv-files-is-not-PAR-it-s-DAR.patch
Type: text/x-patch
Size: 1896 bytes
Desc: Attached file: 0001-The-aspect-stored-in-nuv-files-is-not-PAR-it-s-DAR.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080608/bd99823e/attachment.bin>



More information about the ffmpeg-devel mailing list