From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Sat, 22 Jun 2013 12:05:01 GMT
Subject: Re: [PATCH] help: introduce man.viewer = eman
Message-ID: <CALkWK0m6V9ULgeA81yK2WuoLgf6n4q8_BS-D4VHpTYJMQiG95Q@mail.gmail.com>
In-Reply-To: <20130622115938.GF4676@serenity.lan>

John Keeping wrote:
> Would it be nicer to pass a string in here instead of a flag?  Then this
> becomes:
>
>         strbuf_addf(&man_page, "(%s \"%s\")", command, page);
>
> You should probably also rename this function to "exec_emacsclient" or
> something as well now that it doesn't just launch woman.

Sure, will do both.