Junio C Hamano <gitster@xxxxxxxxx> writes: > Marat Radchenko <marat@xxxxxxxxxxxxxxxx> writes: > >> Signed-off-by: Marat Radchenko <marat@xxxxxxxxxxxxxxxx> >> --- >> git-compat-util.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/git-compat-util.h b/git-compat-util.h >> index b338277..101c9d7 100644 >> --- a/git-compat-util.h >> +++ b/git-compat-util.h >> @@ -474,7 +474,7 @@ extern int git_munmap(void *start, size_t length); >> #endif >> >> #define DEFAULT_PACKED_GIT_LIMIT \ >> - ((1024L * 1024L) * (sizeof(void*) >= 8 ? 8192 : 256)) >> + ((size_t)(1024L * 1024L) * (sizeof(void*) >= 8 ? 8192 : 256)) > > 1024 * 1024 * 8192 overflows 32-bit unsigned, but is size_t always > large enough? Just checking. Heh, I was being silly. This gives the default value for a variable whose type is size_t, so it would better fit. So please throw 13 in the list of changes I found sensible in the other message. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html