Jonathan Nieder <jrnieder@xxxxxxxxx> 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. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html