]> git.ipfire.org Git - thirdparty/git.git/commit
csum-file: fix -Wsign-compare warning on 32-bit platform
authorJunio C Hamano <gitster@pobox.com>
Fri, 6 Dec 2024 10:27:22 +0000 (11:27 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 6 Dec 2024 11:20:03 +0000 (20:20 +0900)
commitba8f6018b5bed4fc58f8dfe2f9714d22398b06fe
tree48cabeaead2941a34ff3ca022fa38e8d98a04504
parent47d72a74a737f06791c282a75baf2c573cdf42f6
csum-file: fix -Wsign-compare warning on 32-bit platform

On 32-bit platforms, ssize_t may be "int" while size_t may be
"unsigned int".  At times we compare the number of bytes we read
stored in a ssize_t variable with "unsigned int", but that is done
after we check that we did not get an error return (which is
negative---and that is the whole reason why we used ssize_t and not
size_t), so these comparisons are safe.

But compilers may not realize that.  Cast these to size_t to work
around the false positives.  On platforms with size_t/ssize_t wider
than a normal int, this won't be an issue.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
csum-file.c