diff options
| author | Junio C Hamano <gitster@pobox.com> | 2017-08-23 14:33:43 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-08-23 14:33:44 -0700 |
| commit | 0cb526e031bd28f33750ed7beef1d7556ef387ea (patch) | |
| tree | 3e7c8f360a71a1b3fd6f7fdefcb6f537d5de0183 /builtin/log.c | |
| parent | 72140a73198b2fdbf6fc83ff679d3f553ed5faed (diff) | |
| parent | de239446b69f3b453050af8091e07aa5433421cc (diff) | |
| download | git-0cb526e031bd28f33750ed7beef1d7556ef387ea.tar.gz git-0cb526e031bd28f33750ed7beef1d7556ef387ea.tar.xz | |
Merge branch 'jk/reflog-walk' into maint
Numerous bugs in walking of reflogs via "log -g" and friends have
been fixed.
* jk/reflog-walk:
reflog-walk: apply --since/--until to reflog dates
reflog-walk: stop using fake parents
rev-list: check reflog_info before showing usage
get_revision_1(): replace do-while with an early return
log: do not free parents when walking reflog
log: clarify comment about reflog cycles
revision: disallow reflog walking with revs->limited
t1414: document some reflog-walk oddities
Diffstat (limited to 'builtin/log.c')
| -rw-r--r-- | builtin/log.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/builtin/log.c b/builtin/log.c index c6362cf92..5ffc380bd 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -372,11 +372,14 @@ static int cmd_log_walk(struct rev_info *rev) */ rev->max_count++; if (!rev->reflog_info) { - /* we allow cycles in reflog ancestry */ + /* + * We may show a given commit multiple times when + * walking the reflogs. + */ free_commit_buffer(commit); + free_commit_list(commit->parents); + commit->parents = NULL; } - free_commit_list(commit->parents); - commit->parents = NULL; if (saved_nrl < rev->diffopt.needed_rename_limit) saved_nrl = rev->diffopt.needed_rename_limit; if (rev->diffopt.degraded_cc_to_c) |