[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