[FFmpeg-devel] [RCF] lavfi aspect ratio setting path

Baptiste Coudurier baptiste.coudurier
Mon Nov 29 12:26:40 CET 2010


On 11/29/10 3:12 AM, Baptiste Coudurier wrote:
> On 11/29/10 2:55 AM, Stefano Sabatini wrote:
>> On date Sunday 2010-11-28 14:51:14 -0800, Baptiste Coudurier encoded:
>>> On 11/28/10 2:41 PM, Stefano Sabatini wrote:
>>>> On date Wednesday 2010-11-24 00:50:17 -0800, Baptiste Coudurier encoded:
>> [...]
>>>>> diff --git a/ffmpeg.c b/ffmpeg.c
>>>>> index e58e7b5..c85c81b 100644
>>>>> --- a/ffmpeg.c
>>>>> +++ b/ffmpeg.c
>>>>> @@ -351,6 +351,7 @@ static int configure_filters(AVInputStream *ist, AVOutputStream *ost)
>>>>>      AVCodecContext *codec = ost->st->codec;
>>>>>      AVCodecContext *icodec = ist->st->codec;
>>>>>      FFSinkContext ffsink_ctx = { .pix_fmt = codec->pix_fmt };
>>>>> +    AVRational pixel_aspect_ratio;
>>>>
>>>> I suggest sample_aspect_ratio, for consistency with setsar and
>>>> AVCodecContext (AVFilterLink.pixel_aspect also should be renamed).
>>>
>>> Humm, ffmpeg displays PAR for it and I think that in this case I would
>>> prefer naming it pixel_aspect_ratio in AVFilterLink as well.
>>
>> http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/121220
> 
> Hummm, rofl, ok :)
> 

Patch updated.
Btw do we want to keep pixel_aspect in AVFilterBufferRefVideoProps ?
Otherwise I'll just remove it afterwards.

-- 
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/20101129/90120248/attachment.asc>



More information about the ffmpeg-devel mailing list