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

Michael Niedermayer michaelni at gmx.at
Sun Sep 2 15:24:17 CEST 2012


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

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20120902/4093787a/attachment.asc>


More information about the ffmpeg-devel mailing list