diff mbox series

[FFmpeg-devel] avfilter/aformat: add shorthand names for options

Message ID 20200107102203.8225-1-ffmpeg@gyani.pro
State Accepted
Headers show
Series [FFmpeg-devel] avfilter/aformat: add shorthand names for options | expand

Checks

Context Check Description
andriy/ffmpeg-patchwork success Make fate finished

Commit Message

Gyan Doshi Jan. 7, 2020, 10:22 a.m. UTC 
---
 doc/filters.texi         | 6 +++---
 libavfilter/af_aformat.c | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)

Comments

Lou Logan Jan. 7, 2020, 8:32 p.m. UTC | #1 
On Tue, Jan 7, 2020, at 1:22 AM, Gyan Doshi wrote:
> ---
>  doc/filters.texi         | 6 +++---
>  libavfilter/af_aformat.c | 3 +++
>  2 files changed, 6 insertions(+), 3 deletions(-)

LGTM
Gyan Doshi Jan. 8, 2020, 5:47 a.m. UTC | #2 
On 08-01-2020 02:02 am, Lou Logan wrote:
> On Tue, Jan 7, 2020, at 1:22 AM, Gyan Doshi wrote:
>> ---
>>   doc/filters.texi         | 6 +++---
>>   libavfilter/af_aformat.c | 3 +++
>>   2 files changed, 6 insertions(+), 3 deletions(-)
> LGTM

Pushed as 5bd001043d91077eabd4cd1ffa71b396cf07c04d

Thanks,
Gyan
diff mbox series

Patch

diff --git a/doc/filters.texi b/doc/filters.texi
index fc2d198077..78ff5294a9 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -1285,13 +1285,13 @@  negotiate the most appropriate format to minimize conversions.
 It accepts the following parameters:
 @table @option
 
-@item sample_fmts
+@item sample_fmts, f
 A '|'-separated list of requested sample formats.
 
-@item sample_rates
+@item sample_rates, r
 A '|'-separated list of requested sample rates.
 
-@item channel_layouts
+@item channel_layouts, cl
 A '|'-separated list of requested channel layouts.
 
 See @ref{channel layout syntax,,the Channel Layout section in the ffmpeg-utils(1) manual,ffmpeg-utils}
diff --git a/libavfilter/af_aformat.c b/libavfilter/af_aformat.c
index e43149561a..1a702778c3 100644
--- a/libavfilter/af_aformat.c
+++ b/libavfilter/af_aformat.c
@@ -50,8 +50,11 @@  typedef struct AFormatContext {
 #define F AV_OPT_FLAG_FILTERING_PARAM
 static const AVOption aformat_options[] = {
     { "sample_fmts",     "A '|'-separated list of sample formats.",  OFFSET(formats_str),         AV_OPT_TYPE_STRING, .flags = A|F },
+    { "f",               "A '|'-separated list of sample formats.",  OFFSET(formats_str),         AV_OPT_TYPE_STRING, .flags = A|F },
     { "sample_rates",    "A '|'-separated list of sample rates.",    OFFSET(sample_rates_str),    AV_OPT_TYPE_STRING, .flags = A|F },
+    { "r",               "A '|'-separated list of sample rates.",    OFFSET(sample_rates_str),    AV_OPT_TYPE_STRING, .flags = A|F },
     { "channel_layouts", "A '|'-separated list of channel layouts.", OFFSET(channel_layouts_str), AV_OPT_TYPE_STRING, .flags = A|F },
+    { "cl",              "A '|'-separated list of channel layouts.", OFFSET(channel_layouts_str), AV_OPT_TYPE_STRING, .flags = A|F },
     { NULL }
 };