repack.c: LLP64 compatibility, upcast unity for left shift
Visual Studio reports C4334 "was 64-bit shift intended" warning because of size mismatch. Promote unity to the matching type to fit with the `&` operator. Signed-off-by: Philip Oakley <philipoakley@iee.email> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Philip Oakley committed
Dec 1, 2021 at 00:29 UTC
a43abad1e3bfb20864cd3873a4a8b86f845bae1a
1 file changed
+1
-1
builtin/repack.c
+1
-1
@@ -842,7 +842,7 @@ int cmd_repack(int argc, const char **argv, const char *prefix)
842
fname_old = mkpathdup("%s-%s%s",
843
packtmp, item->string, exts[ext].name);
844
845
- if (((uintptr_t)item->util) & (1 << ext)) {
845
+ if (((uintptr_t)item->util) & ((uintptr_t)1 << ext)) {
846
struct stat statbuffer;
847
if (!stat(fname_old, &statbuffer)) {
848
statbuffer.st_mode &= ~(S_IWUSR | S_IWGRP | S_IWOTH);