On Mon, Feb 02, 2009 at 10:46:30PM +0100, Thomas Rast wrote:
> +interactive.readkey::
> + Attempt to use Term::ReadKey facilities to allow typing
> + one-letter input with a single key. Currently only used by
> + the `\--patch` mode of linkgit:git-add[1]. Silently disabled
> + if Term::ReadKey is not available.
Minor nit: the name of this variable implies that it will be used across
all interactive commands (including any future ones). But the
description is intimately linked with perl. Maybe structure it like
"here is what this does in general, but here are some specific caveats".
Something like:
interactive.readkey::
In interactive programs, allow the user to provide one-letter
input with a single key (i.e., without hitting enter). Currently
this is used only by the `\--patch` mode of linkgit:git-add[1].
Note that this feature is silently disabled for Perl programs
(like git-add) if Term::ReadKey is not available.
But since there is only _one_ place it is used now, and since that
program _is_ a Perl program, maybe this is overengineering.
Other than that, your series looks OK to me. I think it would be nice to
have a 5/4 that extends the feature to other menus for the sake of
consistency. But like you, I really just use "git add -p" myself.
-Peff
--
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
