[PATCH] Add a subject indicating that the commit message is empty
Nezmer
git at nezmer.info
Thu Aug 11 02:57:32 CEST 2011
> > > + ret->subject = xmalloc(20);
> >
> > xstrdup already allocates memory. No need to invoke malloc here (doing
> > this will result in a memory leak).
Good to know.
>
> Oh, and you obviously also introduce a memory leak by overwriting the
> pointer stored in "ret->subject" before without free'ing it.
I assumed strchr(p, '\n') would return NULL here. I see that my
assumption was wrong now.
> This should be implemented slightly differently.
> The idea sounds good, though. I'll have a look at that.
Thanks.
Would declaring a const char* in cgit.h and pointing all empty subjects
to it be a good choice?
> >
> > > + ret->subject = xstrdup("(no commit message)");
> > > + }
More information about the CGit
mailing list