summaryrefslogtreecommitdiff
path: root/shallow.c
diff options
context:
space:
mode:
authorRené Scharfe <l.s.r@web.de>2016-07-30 20:18:31 +0200
committerJunio C Hamano <gitster@pobox.com>2016-08-01 14:01:03 -0700
commit50492f7b38f84619cbfb443efc3c0c78fc0304dd (patch)
tree051c7b006815c0c55c8c37039cb9877402b56a1b /shallow.c
parentf8f7adce9fc50a11a764d57815602dcb818d1816 (diff)
downloadgit-50492f7b38f84619cbfb443efc3c0c78fc0304dd.tar.gz
pass constants as first argument to st_mult()rs/st-mult
The result of st_mult() is the same no matter the order of its arguments. It invokes the macro unsigned_mult_overflows(), which divides the second parameter by the first one. Pass constants first to allow that division to be done already at compile time. Signed-off-by: Rene Scharfe <l.s.r@web.de> Reviewed-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'shallow.c')
-rw-r--r--shallow.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/shallow.c b/shallow.c
index 4d554caf8d..54e2db7334 100644
--- a/shallow.c
+++ b/shallow.c
@@ -389,7 +389,7 @@ static void paint_down(struct paint_info *info, const unsigned char *sha1,
unsigned int i, nr;
struct commit_list *head = NULL;
int bitmap_nr = (info->nr_bits + 31) / 32;
- size_t bitmap_size = st_mult(bitmap_nr, sizeof(uint32_t));
+ size_t bitmap_size = st_mult(sizeof(uint32_t), bitmap_nr);
uint32_t *tmp = xmalloc(bitmap_size); /* to be freed before return */
uint32_t *bitmap = paint_alloc(info);
struct commit *c = lookup_commit_reference_gently(sha1, 1);