Another portability fix for tzcode2016g update.

clang points out that SIZE_MAX wouldn't fit into an int, which means
this comparison is pretty useless.  Per report from Thomas Munro.

diff --git a/src/timezone/zic.c b/src/timezone/zic.c
index 04f4df27ceccb7abe05262bc3350a0e66bf894c4..3f714ef46cb980a300493ccc1ce3fd0364d0b8ae 100644 (file)
--- a/src/timezone/zic.c
+++ b/src/timezone/zic.c
@@ -424,9 +424,8 @@ growalloc(void *ptr, size_t itemsize, int nitems, int *nitems_alloc)
        else
        {
                int                     nitems_max = INT_MAX - WORK_AROUND_QTBUG_53071;
-               int                     amax = nitems_max < SIZE_MAX ? nitems_max : SIZE_MAX;
 
-               if ((amax - 1) / 3 * 2 < *nitems_alloc)
+               if ((nitems_max - 1) / 3 * 2 < *nitems_alloc)
                        memory_exhausted(_("int overflow"));
                *nitems_alloc = *nitems_alloc + (*nitems_alloc >> 1) + 1;
                return erealloc(ptr, size_product(*nitems_alloc, itemsize));