[FFmpeg-devel] [PATCH 1/2] Add sample_aspect_ratio to AVFilterLink

[Ronald S. Bultje]

Hi,

On Fri, Feb 4, 2011 at 3:34 AM, Stefano Sabatini
<stefano.sabatini-lala at poste.it> wrote:
> On date Thursday 2011-02-03 20:46:03 -0500, Ronald S. Bultje encoded:
>> > - ? ?/* These two parameters apply only to video */
>> > + ? ?/* These parameters apply only to video */
>> > ? ? int w; ? ? ? ? ? ? ? ? ? ? ?///< agreed upon image width
>> > ? ? int h; ? ? ? ? ? ? ? ? ? ? ?///< agreed upon image height
>> > + ? ?AVRational sample_aspect_ratio; ///< agreed upon sample aspect ratio
>> > ? ? /* These two parameters apply only to audio */
>> > ? ? int64_t channel_layout; ? ? ///< channel layout of current buffer (see libavcore/audioconvert.h)
>> > ? ? int64_t sample_rate; ? ? ? ?///< samples per second
>>
>> /**
>> ?* @defgroup video_props Video properties
>> ?* These parameters apply only to video
>> ?* @{
>> ?*/
>> int w; ///< ...
>> int h; ///< ...
>> AVRational sar; ///< ...
>> /**
>> ?* @}
>> ?*/
>>
>> /**
>> ?* @defgroup audio_props Audio properties
>> ?* These parameters apply only to audio
>> ?* @{
>> ?*/
>> int channels; ///< ...
>> int sample_rate; ///< ...
>> /**
>> ?* @}
>> ?*/
>>
>> This way it groups correctly in doxygen output also. You can also
>> remove some newlines if you care...
>
> I'll do as a separate patch, I want to avoid to do more modifications
> to this patch which was already committed to ffmpeg at videolan.

Makes sense, I'll leave it for Mans to commit since he already has it
in his tree.

Ronald