[FFmpeg-devel] [RCF] lavfi aspect ratio setting path
Baptiste Coudurier
baptiste.coudurier
Sat Dec 4 06:53:16 CET 2010
On 12/2/10 6:28 PM, Michael Niedermayer wrote:
> On Mon, Nov 29, 2010 at 03:26:40AM -0800, Baptiste Coudurier wrote:
> [...]
>> @@ -419,6 +430,10 @@
>>
>> codec->width = ist->output_video_filter->inputs[0]->w;
>> codec->height = ist->output_video_filter->inputs[0]->h;
>> + ost->st->sample_aspect_ratio = codec->sample_aspect_ratio =
>
>> + frame_aspect_ratio == 0 ? // overriden by the -aspect cli option
>> + av_d2q(frame_aspect_ratio*codec->height/codec->width, 255) :
>> + ist->output_video_filter->inputs[0]->sample_aspect_ratio;
>
> that looks odd if frame_aspect_ratio == 0 then
> av_d2q(frame_aspect_ratio*codec->height/codec->width, 255)
> will be used
> but thats av_d2q(0*codec->height/codec->width, 255)=0
Yes you are right, it required more modifications to make it work.
Updated patch.
--
Baptiste COUDURIER
Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer http://www.ffmpeg.org
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: avfilter_par.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101203/1f75d8fa/attachment.txt>
More information about the ffmpeg-devel
mailing list