Junio C Hamano <gits...@pobox.com> writes:

> Steffen Prohaska <proha...@zib.de> writes:
>>>> +  if (limit == -1) {
>>>> +          const char *env = getenv("GIT_MMAP_LIMIT");
>>>> +          limit = env ? atoi(env) * 1024 : 0;
>> ... this should then be changed to atol(env), and ... 
> In the real codepath (not debugging aid like this) we try to avoid
> atoi/atol so that we can catch errors like feeding "123Q" and
> parsing it as 123.

Sorry for hitting <SEND> by mistake before finishing the paragraph,
which should have concluded with:

    But it is OK to be loose in an debugging aid.  If I were doing
    this code, I actually would call git_parse_ulong() and not
    define it in terms of kilobytes, though.

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to