path: root/decorate.c
diff options
authorLinus Torvalds <>2007-04-21 01:30:06 (GMT)
committerJunio C Hamano <>2007-04-21 02:16:12 (GMT)
commit2a1a3dce3333950d16f78caf34b3bb2ac46ab4d4 (patch)
treeecff3deffa96e5b0971e0fc0926ff97efab2acdf /decorate.c
parent2c9750cc8b902a55669183e05533207dd7ec71fd (diff)
Fix a copy-n-paste bug in the object decorator code.
Duh. When I did the object decorator thing, I made the "loop over the hash" function use the same logic for updating the hash, ie made them use if (++j >= size) j = 0; for both the hash update for both "insert" and "lookup" HOWEVER. For some inexplicable reason I had an extraneous j++; in the insert path (probably just from the fact that the old code there used j++; if (j >= size) j = 0; and when I made them use the same logic I just didn't remove the old extraneous line properly. This fixes it. Signed-off-by: Linus Torvalds <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'decorate.c')
1 files changed, 0 insertions, 1 deletions
diff --git a/decorate.c b/decorate.c
index 396b4131..23f6b00 100644
--- a/decorate.c
+++ b/decorate.c
@@ -24,7 +24,6 @@ static void *insert_decoration(struct decoration *n, struct object *base, void *
hash[j].decoration = decoration;
return old;
- j++;
if (++j >= size)
j = 0;