[FFmpeg-devel] [PATCH] lavu/opt: misc cosmetics/alignment fixes in opt_list()

Stefano Sabatini stefasab at gmail.com
Wed Oct 17 10:28:46 CEST 2012


On date Sunday 2012-09-02 15:24:17 +0200, Michael Niedermayer encoded:
> On Mon, Aug 13, 2012 at 01:56:47PM +0200, Stefano Sabatini wrote:
> > ---
> >  libavutil/opt.c |   26 +++++++++++++-------------
> >  1 files changed, 13 insertions(+), 13 deletions(-)
> > 
> > diff --git a/libavutil/opt.c b/libavutil/opt.c
> > index 73540ad..eaf36d9 100644
> > --- a/libavutil/opt.c
> > +++ b/libavutil/opt.c
> > @@ -601,44 +601,44 @@ static void opt_list(void *obj, void *av_log_obj, const char *unit,
> >          else if (unit && opt->type==AV_OPT_TYPE_CONST && strcmp(unit, opt->unit))
> >              continue;
> >          else if (unit && opt->type == AV_OPT_TYPE_CONST)
> > -            av_log(av_log_obj, AV_LOG_INFO, "   %-15s ", opt->name);
> > +            av_log(av_log_obj, AV_LOG_INFO, "  %-15s ", opt->name);
> >          else
> 
> > -            av_log(av_log_obj, AV_LOG_INFO, "-%-17s ", opt->name);
> > +            av_log(av_log_obj, AV_LOG_INFO, "%-17s ", opt->name);
> 
> this looks like more than just alignment, can you elaborate ?
> 
> otherwise, if things look better after the patch, sure patch ok

Dropped, since I'm not sure this is a real improvement.
-- 
FFmpeg = Fundamental Fundamental Minimal Portable Eccentric Gargoyle


More information about the ffmpeg-devel mailing list