From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 20 Jan 2011 20:59:45 GMT
Subject: Re: [PATCH 1/2] rebase -i: reword in-editor documentation of "exec"
Message-ID: <7vy66fqoji.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20110120200949.GB11702@burratino>

Jonathan Nieder <jrnieder@gmail.com> writes:

> Maybe "exec" should be explained outside this table?  For example,
> maybe something along the lines of
>
> 	 x, exec = run an arbitrary command (see below)

Ok, none of the other insns in the insn sheet mention what the argument to
the command means anyway (e.g. "p, pick = replay the commit" doesn't say
explicitly where the commit comes from), so I think the original patch is
probably fine.

If we wanted to be more helpful, perhaps s/(see below)/specified on the
rest of the line/ should be sufficient without adding extra lines.