Re: yap - Gnome2::About



On Thu, 2003-04-17 at 20:45, Chas Owens wrote:
Memory leak! bad Chas, no cookie for you!
...
+       if (a) free(a);

a very, very minor point, but since we're using glib anyway you might
want to go ahead and use g_free(a); (and other g_ functions where
appropriate.) as a side benefit it does the if not NULL check for you.
don't say i didn't warn you that it was minor.

-rm




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]