[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