| commit | 18bc8eb7b51f9249fd2d57aa09f2aa959dae14a2 | |
| author | Jeff King <peff@peff.net> | |
| Mon, 9 Aug 2021 22:47:42 +0000 (9 18:47 -0400) | ||
| committer | Johannes Schindelin <johannes.schindelin@gmx.de> | |
| Sun, 12 Mar 2023 19:31:54 +0000 (12 20:31 +0100) | ||
| tree | da2e4a245d039f959821232c30972ede9219240a | treesnapshot (tar.gz zip) |
| parent | 394a759d2b5f0a1a1908c820cf142f45cb78718c | commitdiff |
range-diff: drop useless "offset" variable from read_patches()
The "offset" variable was was introduced in 44b67cb62b (range-diff:
split lines manually, 2019-07-11), but it has never done anything
useful. We use it to count up the number of bytes we've consumed, but we
never look at the result. It was probably copied accidentally from an
almost-identical loop in apply.c:find_header() (and the point of that
commit was to make use of the parse_git_diff_header() function which
underlies both).
Because the variable was set but not used, most compilers didn't seem to
notice, but the upcoming clang-14 does complain about it, via its
-Wunused-but-set-variable warning.
Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
The "offset" variable was was introduced in 44b67cb62b (range-diff:
split lines manually, 2019-07-11), but it has never done anything
useful. We use it to count up the number of bytes we've consumed, but we
never look at the result. It was probably copied accidentally from an
almost-identical loop in apply.c:find_header() (and the point of that
commit was to make use of the parse_git_diff_header() function which
underlies both).
Because the variable was set but not used, most compilers didn't seem to
notice, but the upcoming clang-14 does complain about it, via its
-Wunused-but-set-variable warning.
Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| range-diff.c | diffblobblamehistory |