From: Jeff King Date: Sun, 26 Jul 2026 08:46:22 +0000 (-0400) Subject: diff-lib: drop stale comment about advancing o->pos X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=640177a987270564a8ef90cdc359677209fbe0fe;p=thirdparty%2Fgit.git diff-lib: drop stale comment about advancing o->pos The comment above oneway_diff() claims that the callback must advance o->pos to skip index entries it has already processed. That stopped being true in da165f470e (unpack-trees.c: prepare for looking ahead in the index, 2010-01-07), which moved that bookkeeping into unpack_trees(). Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- diff --git a/diff-lib.c b/diff-lib.c index 0e868b28b6..bcab7c8500 100644 --- a/diff-lib.c +++ b/diff-lib.c @@ -508,11 +508,9 @@ static void do_oneway_diff(struct unpack_trees_options *o, * For diffing, the index is more important, and we only have a * single tree. * - * We're supposed to advance o->pos to skip what we have already processed. - * * This wrapper makes it all more readable, and takes care of all * the fairly complex unpack_trees() semantic requirements, including - * the skipping, the path matching, the type conflict cases etc. + * the path matching, the type conflict cases etc. */ static int oneway_diff(const struct cache_entry * const *src, struct unpack_trees_options *o)