RE: [PATCH v6] help: colorize man pages

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Jun 24, 2021, 01:44 UTC
Message-ID: <60d3e399945d_2cc520855@natae.notmuch>
In-Reply-To: <20210523054454.1188757-1-felipe.contreras@gmail.com>

> --- a/builtin/help.c
> +++ b/builtin/help.c

Show 5 quoted lines

> @@ -264,6 +289,7 @@ static void exec_man_man(const char *path, const char *page)
>  static void exec_man_cmd(const char *cmd, const char *page)
>  {
>  	struct strbuf shell_cmd = STRBUF_INIT;
> +	colorize_man();

On further reflection I don't think this colorize belongs here. exec_man_cmd() is meant to execute any custom command, not necessarily man.

-- 
Felipe Contreras