Re: [PATCH v2] macos: lazily initialize iconv

Ralf Thielow [off-list ref] writes:

On Tue, Jul 31, 2012 at 8:37 PM, Junio C Hamano [off-list ref] wrote:

quoted

+       /* Avoid iconv_open()/iconv_close() if there is nothing to convert */
+       for (i = 0; i < argc; i++) {
+               if (has_utf8(argv[i], (size_t)-1, NULL))
+                       break;
+       }
+       if (argc <= i)
+               return; /* no utf8 found */

sorry, but "argc" can never be smaller than "i", right?

Yeah, but it is idiomatic to have an inverse of the exit condition
of the preceding for loop here to catch an early exit, and writing
it as "if (i == argc)", while technically correct, would break the
pattern.