On Sat 12 Apr 2014 08:59, Dmitry Bogatov <kact...@gnu.org> writes:
> * Dmitry Bogatov <kact...@gnu.org> [2014-04-10 20:52:43+0400]
>> > > + {
>> > > + free (nargv);
>> > > return 0L;
>> > > + }
>> > > else
>> > > nargv[nargi++] = narg;
>> > I don't understand this. In both cases, free() will be called with an
>> > argument of 0, which does nothing.
>> My bad. It does not fix problem, that if realloc fails, it does not free
>> memory, but we blindly assign, losing pointer to previous memory
>> chunk.
>
> Attached new version. Please, take a look.
Applied, thanks. I altered the commit log; please take a look and
submit in that format in the future. The "magit" emacs mode makes it
easy to produce this format.
Cheers,
Andy
--
http://wingolog.org/
