[FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

Michael Niedermayer michael at niedermayer.cc
Thu May 5 05:11:50 CEST 2016


On Wed, May 04, 2016 at 08:40:04AM +0300, Andrey Utkin wrote:
> On Wed, May 04, 2016 at 05:07:10AM +0200, Michael Niedermayer wrote:
> > doesnt this require an update to teh docs ?
> 
> Indeed, thanks for commenting.
> 
> But I wonder how should I format reference to "aspect" correctly and pretty.
> Now I have come up with this, which I think is not perfect.  Any help is very
> appreciated.
> 
> --- a/doc/codecs.texi
> +++ b/doc/codecs.texi
> @@ -456,6 +456,9 @@ Possible values:
>  @item aspect @var{rational number} (@emph{encoding,video})
>  Set sample aspect ratio.
>  
> + at item sar @var{rational number} (@emph{encoding,video})
> +Set sample aspect ratio. Alias to @var{aspect}.

i think this is ok if noone has a better idea (i dont have one)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160505/694826f3/attachment.sig>


More information about the ffmpeg-devel mailing list