Re: [PATCH] help: introduce man.viewer = eman
- From
Ramkumar Ramachandra <artagnon@gmail.com>
- Date
- Jun 22, 2013, 12:05 UTC
- Message-ID
- <CALkWK0m6V9ULgeA81yK2WuoLgf6n4q8_BS-D4VHpTYJMQiG95Q@mail.gmail.com>
- In-Reply-To
- <20130622115938.GF4676@serenity.lan>
John Keeping wrote:
Show 7 quoted lines
> 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.