[FFmpeg-devel] [PATCH] avfilter/vf_pad: remove options description from filter description

Michael Niedermayer michaelni at gmx.at
Fri Sep 20 00:39:08 CEST 2013


On Thu, Sep 19, 2013 at 05:55:30PM +0000, Paul B Mahol wrote:
> On 9/19/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Thu, Sep 19, 2013 at 05:40:24PM +0000, Paul B Mahol wrote:
> >> No other filter does that.
> >>
> >> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> >> ---
> >>  libavfilter/vf_pad.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/libavfilter/vf_pad.c b/libavfilter/vf_pad.c
> >> index 45b32e6..23324a9 100644
> >> --- a/libavfilter/vf_pad.c
> >> +++ b/libavfilter/vf_pad.c
> >> @@ -394,7 +394,7 @@ static const AVFilterPad avfilter_vf_pad_outputs[] =
> >> {
> >>
> >>  AVFilter avfilter_vf_pad = {
> >>      .name          = "pad",
> >> -    .description   = NULL_IF_CONFIG_SMALL("Pad input image to
> >> width:height[:x:y[:color]] (default x and y: 0, default color: black)."),
> >> +    .description   = NULL_IF_CONFIG_SMALL("Pad input image."),
> >
> > compare to crop:
> >  .description   = NULL_IF_CONFIG_SMALL("Crop the input video to
> > width:height:x:y."),
> >
> 
> So you are ok to remove that one too, or?

ive no oppinon on the removial of either.
Just looked at crop at the time i read that this is the only one so
wanted to point out there are more

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

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130920/02546f0e/attachment.asc>


More information about the ffmpeg-devel mailing list