Re: [PATCH 4/4] ALLOC_GROW: avoid -Wsign-compare warnings
From: Junio C Hamano <hidden>
Date: 2017-09-22 04:20:52
Ramsay Jones [off-list ref] writes:
Signed-off-by: Ramsay Jones <redacted> --- builtin/pack-objects.c | 4 ++-- config.c | 2 +- diff.c | 2 +- line-log.c | 18 +++++++++--------- line-log.h | 2 +- revision.c | 2 +- tree-walk.c | 3 +-- 7 files changed, 16 insertions(+), 17 deletions(-)
Thanks. I did not spot any questionable conversion (e.g. "something that used to be signed was because it wanted to store -1 as a sentinel" would be broken if we just change that to unsigned) by going over the places these variables and fields are actually used. A review of a patch like this involves reading through 10x more lines than we see in the above diffstat, and producing it would most likely have taken the same amount of effort, at least. Very much appreciated.
quoted hunk
diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index a57b4f058..a6ee653bf 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c@@ -2563,8 +2563,8 @@ struct in_pack_object { }; struct in_pack { - int alloc; - int nr; + unsigned int alloc; + unsigned int nr;
This is a bit questionable ;-) but it is something I can locally tweak easily.