[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