unpack-trees.c: work around run-time array initialization flaw on IRIX 6.5

drafnel · gitster · commit 0039ba7e5e63 · 2009-07-10T23:50:29.000-07:00
The c99 MIPSpro Compiler version 7.4.4m on IRIX 6.5 does not properly
initialize run-time initialized arrays.  An array which is initialized with
fewer elements than the length of the array should have the unitialized
elements initialized to zero.  This compiler only initializes the remaining
elements when the last element is a static parameter.  So work around it
by adding a "NULL" initialization parameter.

Signed-off-by: Brandon Casey &lt;casey@nrlssc.navy.mil&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/unpack-trees.c b/unpack-trees.c
@@ -128,7 +128,7 @@ static inline int call_unpack_fn(struct cache_entry **src, struct unpack_trees_o
 
 static int unpack_index_entry(struct cache_entry *ce, struct unpack_trees_options *o)
 {
-	struct cache_entry *src[5] = { ce, };
+	struct cache_entry *src[5] = { ce, NULL, };
 
 	o->pos++;
 	if (ce_stage(ce)) {

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ static inline int call_unpack_fn(struct cache_entry **src, struct unpack_trees_o`
`128`	`128`
`129`	`129`	`static int unpack_index_entry(struct cache_entry ce, struct unpack_trees_options o)`
`130`	`130`	`{`
`131`		`- struct cache_entry *src[5] = { ce, };`
	`131`	`+ struct cache_entry *src[5] = { ce, NULL, };`
`132`	`132`
`133`	`133`	`o->pos++;`
`134`	`134`	`if (ce_stage(ce)) {`