summaryrefslogtreecommitdiff
path: root/builtin
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2017-07-07 05:07:16 -0400
committerLibravatar Junio C Hamano <gitster@pobox.com>2017-07-07 10:04:53 -0700
commit82fd0f4a4b2c394ddba279eab7072fe87b750d2e (patch)
treedbbf242d07813060d816cc14a661b7bb1f50c075 /builtin
parentt1414: document some reflog-walk oddities (diff)
downloadtgif-82fd0f4a4b2c394ddba279eab7072fe87b750d2e.tar.xz
revision: disallow reflog walking with revs->limited
The reflog-walk code doesn't work with limit_list(). That function traverses down the real history graph, not the fake reflog history that get_revision() returns. So it's not going to actually examine all of the commits we're going to show, because we'd add them to the pending list only during the actual traversal. In practice this limitation doesn't really matter, because the options that require list-limiting generally need UNINTERESTING endpoints or symmetric ranges, which already are forbidden for reflog walks. Still, there are likely some corner cases that would behave oddly. We're better off to warn the user that we can't fulfill their request than to generate potentially wrong output. This will also make it easier to refactor the reflog-walking code, because it eliminates a whole area of corner cases we'd have to consider (that already don't work anyway). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
0 files changed, 0 insertions, 0 deletions