[PATCH v4 1/3] diff: avoid redundantly clearing a flag
- From
- Jonathan Tan <jonathantanmy@google.com>
- Date
- Aug 16, 2017, 01:27 UTC
- Message-ID
- <d199b62322a29e55a8ae21636aa7076231352ad7.1502846454.git.jonathantanmy@google.com>
- In-Reply-To
- <cover.1502846454.git.jonathantanmy@google.com>
No code in diff.c sets DIFF_SYMBOL_MOVED_LINE except in mark_color_as_moved(), so it is redundant to clear it for the current line. Therefore, clear it only for previous lines.
This makes a refactoring in a subsequent patch easier.
Signed-off-by: Jonathan Tan <jonathantanmy@google.com> --- diff.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/diff.c b/diff.c index 4af73a7e0..23311f9c0 100644 --- a/diff.c +++ b/diff.c @@ -898,7 +898,7 @@ static void mark_color_as_moved(struct diff_options *o, if (!match) { if (block_length < COLOR_MOVED_MIN_BLOCK_LENGTH && o->color_moved != COLOR_MOVED_PLAIN) { - for (i = 0; i < block_length + 1; i++) { + for (i = 1; i < block_length + 1; i++) { l = &o->emitted_symbols->buf[n - i]; l->flags &= ~DIFF_SYMBOL_MOVED_LINE; }
-- 2.14.1.480.gb18f417b89-goog