Hi,
On Wed, 9 Aug 2006, Junio C Hamano wrote:
> +
> + /* NUL terminate the result */
> + if (desc.alloc <= desc.size) {
> + desc.buffer = xrealloc(desc.buffer, desc.size + 1);
> + desc.alloc++;
While in this particular case, desc.alloc cannot be smaller than desc.size
(and therefore the condition in the if() is misleading), it might be
better to write "desc.alloc = desc.size + 1;" for others to understand...
Ciao,
Dscho
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html