From: Tay Ray Chuan <rctay89@gmail.com>
Date: Mon, 1 Aug 2011 04:20:10 +0000 (+0800)
Subject: xdiff/xhistogram: drop need for additional variable
X-Git-Tag: v1.7.7-rc0~25^2
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=6486a84cb8e69ddf03b2139db3d968400e05f5b3;p=git-core%2Fgit.git

xdiff/xhistogram: drop need for additional variable

Having an additional variable (ptr) instead of changing line(1|2) and
count(1|2) was for debugging purposes.

Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/xdiff/xhistogram.c b/xdiff/xhistogram.c
index 130ceeed8..18f6f997c 100644
--- a/xdiff/xhistogram.c
+++ b/xdiff/xhistogram.c
@@ -323,11 +323,10 @@ static int histogram_diff(xpparam_t const *xpp, xdfenv_t *env,
 		result = fall_back_to_classic_diff(&index, line1, count1, line2, count2);
 	else {
 		if (lcs.begin1 == 0 && lcs.begin2 == 0) {
-			int ptr;
-			for (ptr = 0; ptr < count1; ptr++)
-				env->xdf1.rchg[line1 + ptr - 1] = 1;
-			for (ptr = 0; ptr < count2; ptr++)
-				env->xdf2.rchg[line2 + ptr - 1] = 1;
+			while (count1--)
+				env->xdf1.rchg[line1++ - 1] = 1;
+			while (count2--)
+				env->xdf2.rchg[line2++ - 1] = 1;
 			result = 0;
 		} else {
 			result = histogram_diff(xpp, env,