r15098 - gnucash/trunk/src/gnome-utils - gnc_history_get_last() can return NULL.

Derek Atkins warlord at MIT.EDU
Thu Nov 9 13:58:31 EST 2006


David Hampton <hampton-gnucash at rainbolthampton.net> writes:

> On Thu, 2006-11-09 at 08:14 +0100, Andreas Köhler wrote:
>
>> g_free can handle NULL pointers just fine, now we are double-checking
>> again. There is even a bug about these checks in gtk+:
>> 
>> http://bugzilla.gnome.org/show_bug.cgi?id=369666
>> 
>> I do not suggest to fix all of them in the source, but I will whenever I
>> am editing the surrounding code block anyway.
>> 
>> Or is there some reason for these checks?
>
> Because I come from a time and place when free() didn't check for
> NULL. :-(  Thanks for the education.

free() still doesn't check, but g_free() does.

> David

-derek

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available



More information about the gnucash-devel mailing list