Akshay Aurora <[email protected]> writes: > Signed-off-by: Akshay Aurora <[email protected]> > --- > This is my first patch for a GSoC Microproject, would be great to have your > feedback. > > revision.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/revision.c b/revision.c > index bd027bc..d578652 100644 > --- a/revision.c > +++ b/revision.c > @@ -2248,7 +2248,8 @@ int setup_revisions(int argc, const char **argv, struct > rev_info *revs, struct s > */ > if (revs->reverse && revs->graph) > die("cannot combine --reverse with --graph"); > - > + if (revs->no_walk && revs->graph) > + die("cannot combine --no-walk with --graph"); > if (revs->reflog_info && revs->graph) > die("cannot combine --walk-reflogs with --graph"); > if (!revs->reflog_info && revs->grep_filter.use_reflog_filter)
Needs a new test to make sure "log --no-walk --graph" is rejected, and also you need to make sure all other existing tests that used to pass before this change still pass. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
