CVS update: gnucash/src/gnome

David Hampton hampton at linas.org
Sat Jun 28 12:05:52 CDT 2003


On Sat, 2003-06-28 at 10:32, Linas Vepstas wrote:
> On Sat, Jun 28, 2003 at 01:29:19AM -0500, David Hampton was heard to remark:
> > 
> > Log Message:
> > Clean up usage of guid_to_string() function. There is no longer any
> > need to release the pointer returned by this function.
> 
> OK, but what motivated this change?

I was looking for memory leaks and discovered that half of the callers
of this function freed the return argument and the other half dropped it
on the floor.  Given that most of the usage was in one of the two forms:

	ENTER("%s", guid_to_string())
	printf("%s", guid_to_string())

it made more sense to fix this so the return argument was a static
buffer from the callers point of view.

David



More information about the gnucash-patches mailing list