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

Ronald S. Bultje rsbultje
Sun Feb 6 02:43:23 CET 2011


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



More information about the ffmpeg-devel mailing list