Martin von Zweigbergk <martinv...@gmail.com> writes:

> @@ -435,8 +437,9 @@ static int do_pick_commit(struct commit *commit, struct 
> replay_opts *opts)
>       else
>               parent = commit->parents->item;
>  
> -     if (opts->allow_ff && parent && !hashcmp(parent->object.sha1, head))
> -             return fast_forward_to(commit->object.sha1, head);
> +     if (opts->allow_ff &&
> +         (parent && !hashcmp(parent->object.sha1, head) || !parent && 
> unborn))

Style (from GNU); please avoid (A && B || C) and spell the
precedence between && and || explicitly, i.e.

        ((A && B) || C)

> +     git rm --cached -r . &&
> +     rm -rf * &&

Not "git rm -rf ." and as two separate steps?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to