Use #define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0])) Signed-off-by: Junio C Hamano <junkio@cox.net>

diff --git a/apply.c b/apply.c
index 849a8b4..179b3bb 100644
--- a/apply.c
+++ b/apply.c

@@ -651,7 +651,7 @@
 		len = linelen(line, size);
 		if (!len || line[len-1] != '\n')
 			break;
-		for (i = 0; i < sizeof(optable) / sizeof(optable[0]); i++) {
+		for (i = 0; i < ARRAY_SIZE(optable); i++) {
 			const struct opentry *p = optable + i;
 			int oplen = strlen(p->str);
 			if (len < oplen || memcmp(p->str, line, oplen))

diff --git a/date.c b/date.c
index 416ea57..1c1917b 100644
--- a/date.c
+++ b/date.c

@@ -123,8 +123,6 @@
 	{ "IDLE", +12, 0, },	/* International Date Line East */
 };
 
-#define NR_TZ (sizeof(timezone_names) / sizeof(timezone_names[0]))
-	
 static int match_string(const char *date, const char *str)
 {
 	int i = 0;
@@ -173,7 +171,7 @@
 		}
 	}
 
-	for (i = 0; i < NR_TZ; i++) {
+	for (i = 0; i < ARRAY_SIZE(timezone_names); i++) {
 		int match = match_string(date, timezone_names[i].name);
 		if (match >= 3) {
 			int off = timezone_names[i].offset;

diff --git a/exec_cmd.c b/exec_cmd.c
index 96cc212..590e738 100644
--- a/exec_cmd.c
+++ b/exec_cmd.c

@@ -37,7 +37,7 @@
 				getenv("GIT_EXEC_PATH"),
 				builtin_exec_path };
 
-	for (i = 0; i < sizeof(paths)/sizeof(paths[0]); ++i) {
+	for (i = 0; i < ARRAY_SIZE(paths); ++i) {
 		const char *exec_dir = paths[i];
 		const char *tmp;
 

diff --git a/git-compat-util.h b/git-compat-util.h
index f982b8e..5d543d2 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h

@@ -9,6 +9,8 @@
 #endif
 #endif
 
+#define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))
+
 #include <unistd.h>
 #include <stdio.h>
 #include <sys/stat.h>

diff --git a/git.c b/git.c
index 164d3e9..746e2af 100644
--- a/git.c
+++ b/git.c

@@ -323,8 +323,6 @@
 	return 0;
 }
 
-#define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))
-
 static void handle_internal_command(int argc, const char **argv, char **envp)
 {
 	const char *cmd = argv[0];