Re: [PATCH v2 2/4] pack-objects, streaming: turn "xx >= big_file_threshold" to ".. > .."
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:53:50
Nguyễn Thái Ngọc Duy [off-list ref] writes:
This is because all other places do "xx > big_file_threshold" Signed-off-by: Nguyễn Thái Ngọc Duy <redacted> --- Without two comparisons I added recently in archive-*.c, it's still inconsistent (two xx >= big_file and one xx > big_file). I have no preference between > and >=. If some of you vote >=, I'll update the patch.
Thanks; I do not have huge preference between the two, but "I want special treatment for anything bigger than 2MiB" is probably more natural than "I want special treatment for things that are exactly 2MiB and also things bigger than that threashold".
quoted hunk
builtin/pack-objects.c | 2 +- streaming.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index 1861093..b2e0940 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c@@ -1327,7 +1327,7 @@ static void get_object_details(void) for (i = 0; i < nr_objects; i++) { struct object_entry *entry = sorted_by_offset[i]; check_object(entry); - if (big_file_threshold <= entry->size) + if (big_file_threshold < entry->size) entry->no_try_delta = 1; }diff --git a/streaming.c b/streaming.c index 38b39cd..829ce7d 100644 --- a/streaming.c +++ b/streaming.c@@ -121,7 +121,7 @@ static enum input_source istream_source(const unsigned char *sha1, case OI_LOOSE: return loose; case OI_PACKED: - if (!oi->u.packed.is_delta && big_file_threshold <= size) + if (!oi->u.packed.is_delta && big_file_threshold < size) return pack_non_delta; /* fallthru */ default: