On 27/11/2007, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote:
> [commenting on the patch is a little cumbersome when the mailer does not
> quote it for you...]
Yes, I'm sorry. But being at work I'm extremely lucky to have access
to even gmail (it was blocked for almost a month as a virus source. Of
course it is a virus source! It is a damn mailbox!)
> On Tue, 27 Nov 2007, Alex Riesen wrote:
>
> @@ -550,6 +557,8 @@ static int parse_and_validate_options(int argc, const char *argv[])
> free(enc);
> }
>
> + if (all)
> + cached_only = 0;
>
> My reply> I would rather add another !!cached_only to the existing if().
I'm... Not sure. I'm using it like this:
$ alias gci='git commit --cached'
$ gci
So sometimes I actually want to override --cached, and just so happens
it is always --all case, which seem to be in line with proposals to make
--all default commit behavior.
> @@ -678,7 +688,8 @@ int cmd_commit(int argc, const char **argv, const char *prefix)
> }
>
> if (!prepare_log_message(index_file, prefix) && !in_merge) {
> - run_status(stdout, index_file, prefix);
> + if (!cached_only)
> + run_status(stdout, index_file, prefix);
>
> My reply> Would it not make more sense to avoid run_status() when no_edit?
Will try it out this evening at home. I suspect you're right :)
-
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
