[FFmpeg-devel] [PATCH] Enhance pad filter command-line description.

John Calcote john.calcote
Fri Jul 2 00:29:26 CEST 2010

On 7/1/2010 4:08 PM, Michael Niedermayer wrote:
> On Thu, Jul 01, 2010 at 03:50:29PM -0600, John Calcote wrote:
>> Index: vf_pad.c
>> ===================================================================
>> --- vf_pad.c    (revision 23945)
>> +++ vf_pad.c    (working copy)
>> @@ -306,7 +306,7 @@
>>  AVFilter avfilter_vf_pad = {
>>      .name          = "pad",
>> -    .description   = NULL_IF_CONFIG_SMALL("Add paddings to the input image."),
>> +    .description   = NULL_IF_CONFIG_SMALL("Pad input image to width:height:x:y[:color] (def color: black)."),
> interresting, iam not against this

Thanks Michael.

I sent this earlier, but it seemed to get lost in the ether somewhere.
I've also noticed that one of the big problems in user space right now
involves people running late versions of ffmpeg under GUIs that aren't
aware of the changeover from -padleft, etc to -vf pad= syntax. Late
versions of ffmpeg terminate with an error message indicating that the
new pad filter option should be used instead. In that case, should we
not remove the old command-line syntax from the help text? And a
documentation update is in order as well, I think.

I'd be happy to do this if no one cares but I wanted to start with this
teaser to see who's listening. :)


More information about the ffmpeg-devel mailing list